Skip to content

Conversation

andremion
Copy link
Contributor

@andremion andremion commented Oct 2, 2025

Goal

Improve coverage by consolidating the module reports

Important

The snapshot tests were not being executed with the current setup.
Now they are executing, so I had to fix some of them.

Implementation

  • Update coverage and sonar Gradle scripts
  • Downgrade Kover and Paparazzi dependencies to fix the error during the test execution
  • Update the outdated snapshots
  • Ignore some snapshot tests that are falling and require changes in their implementation, so they can be fixed at a later time

Testing

Unit tests should succeed

@andremion andremion added pr:dependencies Dependency updates pr:internal Internal or infra-only changes pr:test Testing-related changes labels Oct 2, 2025
Copy link
Contributor

github-actions bot commented Oct 2, 2025

PR checklist ✅

All required conditions are satisfied:

  • Title length is OK (or ignored by label).
  • At least one pr: label exists.
  • Sections ### Goal, ### Implementation, and ### Testing are filled.

🎉 Great job! This PR is ready for review.

Copy link
Contributor

github-actions bot commented Oct 2, 2025

SDK Size Comparison 📏

SDK Before After Difference Status
stream-video-android-core 11.46 MB 11.46 MB 0.00 MB 🟢
stream-video-android-ui-xml 5.68 MB 5.68 MB 0.00 MB 🟢
stream-video-android-ui-compose 5.88 MB 5.88 MB 0.00 MB 🟢

@andremion andremion changed the title Improve testing code coverage Improve the testing code coverage Oct 2, 2025
@andremion andremion force-pushed the AND-783-improve-testing-code-coverage-of-video branch 2 times, most recently from b343cde to c7f6beb Compare October 3, 2025 10:30
@andremion andremion changed the title Improve the testing code coverage Improve the testing coverage Oct 3, 2025
@andremion andremion marked this pull request as ready for review October 3, 2025 14:23
@andremion andremion requested a review from a team as a code owner October 3, 2025 14:23
Copy link

sonarqubecloud bot commented Oct 3, 2025

@andremion andremion changed the title Improve the testing coverage Improve the testing code coverage Oct 3, 2025
@andremion andremion force-pushed the AND-783-improve-testing-code-coverage-of-video branch from 3d1c822 to 3d99e95 Compare October 3, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:dependencies Dependency updates pr:internal Internal or infra-only changes pr:test Testing-related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant