-
Notifications
You must be signed in to change notification settings - Fork 487
Introduce skeleton classes for UI tests and screenshots (EXPOSUREAPP-2997) #1496
Introduce skeleton classes for UI tests and screenshots (EXPOSUREAPP-2997) #1496
Conversation
Trimming down SubmissionViewModel.kt
…spatcher and Tan complete
…and screenshots to follow
…tion-screenshots-v2 # Conflicts: # Corona-Warn-App/build.gradle # Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/main/MainActivityModule.kt
such that they don't get included in production builds.
…tion-screenshots-v2
|
Checked on WRU-XD,
|
AlexanderAlferov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
App tested. Calendar working fine. Lgtm
AndroidMedaGalaxy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QR and TAN submission flow tests were successful.
LGTM “ヽ(´▽`)ノ”
Kolpa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, still some more refactoring needed but too much to put into this pr as well
kolyaopahle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now also approved using the right github account '^^
…tion-screenshots-v2
|
Kudos, SonarCloud Quality Gate passed!
|
Submission refactoring and many skeletons for UI tests, and some fastlane screenshot logic.
Getting this merge ready such that it does not become too stale too merge.
Replaces #1467
Test