-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Code coverage #1634
Code coverage #1634
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
…nly wrappers to access the Rust Matrix SDK api, and mappers. It is not really relevant to unit test that: there is no logic to test.
a5841b4
to
02982b6
Compare
Kudos, SonarCloud Quality Gate passed! |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #1634 +/- ##
===========================================
+ Coverage 59.22% 62.99% +3.76%
===========================================
Files 1250 1182 -68
Lines 32327 30383 -1944
Branches 6633 6277 -356
===========================================
- Hits 19147 19140 -7
+ Misses 10306 8371 -1935
+ Partials 2874 2872 -2
☔ View full report in Codecov by Sentry. |
Type of change
Content
Imporve code covergage: add missing unit test and screenshot test.
Also ignore matrix impl lib, since it contains only wrappers and mappers. They are covered by Maestro test.
Motivation and context
Improve code coverage metrics.
Screenshots / GIFs
Tests
Tested devices
Checklist