bitwarden / ios

Bitwarden mobile app for iOS.
https://bitwarden.com
GNU General Public License v3.0
209 stars 26 forks source link

PM-12976: Fix test that needs main actor attribute #989

Closed matt-livefront closed 1 month ago

matt-livefront commented 1 month ago

๐ŸŽŸ๏ธ Tracking

PM-12976

๐Ÿ“” Objective

Fixes a test build error because of a missing @MainActor attribute for accessing the MockConfigService.

This is an addition to https://github.com/bitwarden/ios/pull/987 which was missed in https://github.com/bitwarden/ios/pull/988.

โฐ Reminders before review

๐Ÿฆฎ Reviewer guidelines

github-actions[bot] commented 1 month ago

Logo Checkmarx One โ€“ Scan Summary & Details โ€“ f4cfcdca-b642-4f83-b43c-9b39bd4cf024

No New Or Fixed Issues Found

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.82%. Comparing base (60c6c57) to head (fe00dce). Report is 3 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #989 +/- ## ========================================== + Coverage 88.81% 88.82% +0.01% ========================================== Files 641 641 Lines 40215 40219 +4 ========================================== + Hits 35716 35725 +9 + Misses 4499 4494 -5 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.