Skip to content

Increase Source unit test coverage to 100% - #542

Merged
thudugala merged 1 commit into
thudugala:masterfrom
Elvin-Thudugala-2degrees:elvin-thudugala-2degrees-increase-source-coverage
Jul 22, 2026
Merged

Increase Source unit test coverage to 100%#542
thudugala merged 1 commit into
thudugala:masterfrom
Elvin-Thudugala-2degrees:elvin-thudugala-2degrees-increase-source-coverage

Conversation

@Elvin-Thudugala-2degrees

Copy link
Copy Markdown
Contributor

What does this PR do?

Raises unit test coverage for code under Source/ to 100% line coverage from UnitTests/LocalNotification.UnitTests by adding targeted tests for previously uncovered paths and wiring the test project to include Geofence source coverage.

  • Adds focused coverage tests for remaining uncovered Source lines (resource accessors, initialization flow, extension return paths, and TrimImageBinary exception path).
  • Adds Plugin.LocalNotification.Geofence as a test project reference so Source Geofence code is included in coverage.
  • Adds Microsoft.Maui.Controls to the test project so MAUI-linked Source types used by tests can load.
  • Disables unit test parallelization at assembly level to avoid static-state interference in coverage tests.
Why are we doing this? Any context or related work?

The goal was to ensure Source folder unit test coverage is fully complete and stable. Existing tests passed but left a few Source paths uncovered and were susceptible to shared static state collisions when run in parallel. These updates close the remaining gaps and make coverage runs deterministic.

Add focused coverage tests for remaining Source lines, include Geofence project in unit test references, and stabilize tests by disabling parallelization.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@thudugala
thudugala merged commit a984d3d into thudugala:master Jul 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants