Skip to content
This repository was archived by the owner on Jun 20, 2023. It is now read-only.

Conversation

@d4rken
Copy link
Member

@d4rken d4rken commented Oct 28, 2020

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

  • Perform TAN submission
  • Perform QR submission
  • Check Calendar UI behavior

@d4rken d4rken added maintainers Tag pull requests created by maintainers feature request A new feature proposal for the app labels Oct 28, 2020
…tion-screenshots-v2

# Conflicts:
#	Corona-Warn-App/build.gradle
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/main/MainActivityModule.kt
@d4rken d4rken changed the title Submission instrumentation screenshots v2 Introduce skeleton classes for UI tests and screenshots (EXPOSUREAPP-2997) Oct 28, 2020
@BMItr
Copy link
Contributor

BMItr commented Oct 29, 2020

Checked on WRU-XD,

  • QR-Code behaviour (invalid, pending, positve, negativ), all passed.
  • TAN behaviour (3 TANS), all passed

@d4rken d4rken added enhancement Improvement of an existing feature and removed feature request A new feature proposal for the app labels Oct 29, 2020
Copy link
Contributor

@AlexanderAlferov AlexanderAlferov left a 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

Copy link
Member

@AndroidMedaGalaxy AndroidMedaGalaxy left a 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 “ヽ(´▽`)ノ”

Copy link
Contributor

@Kolpa Kolpa left a 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

Copy link
Contributor

@kolyaopahle kolyaopahle left a 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 '^^

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 2 Code Smells

7.4% 7.4% Coverage
1.8% 1.8% Duplication

@harambasicluka harambasicluka merged commit bc38a84 into release/1.7.x Oct 30, 2020
@harambasicluka harambasicluka deleted the feature/2997-submission-instrumentation-screenshots-v2 branch October 30, 2020 13:49
@harambasicluka harambasicluka added this to the 1.7.0 milestone Nov 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement Improvement of an existing feature maintainers Tag pull requests created by maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants