Skip to content

MS-1345 androidTest usage review#1607

Open
luhmirin-s wants to merge 4 commits intomainfrom
feature/MS-1345-androidTest-review
Open

MS-1345 androidTest usage review#1607
luhmirin-s wants to merge 4 commits intomainfrom
feature/MS-1345-androidTest-review

Conversation

@luhmirin-s
Copy link
Contributor

JIRA ticket
Will be released in: 2026.2.0

Notable changes

  • Removed completely redundant usage of "simprints.testing.android" in multiple modules
  • Removed long-ago commented out event-sync test file, there is reason to keep it
  • Removed androidTest in image module since it has not been kept up-to-date and has been replaced with regular unitTests
  • Removed face capture fragment tests since they were not working, logic was bare-bones and the screen is covered extensively by the Appium tests

The only module with valid androidTest is the enrolment-records/repository.

Testing guidance

  • No business logic changed, only removals of non-functional code.

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@luhmirin-s luhmirin-s requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, meladRaouf and ybourgery and removed request for a team March 5, 2026 14:54
@cla-bot cla-bot bot added the ... label Mar 5, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 5, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants