Skip to content

Screenshot tests were failing, JUnit downgrade needed#16578

Open
tobiasKaminsky wants to merge 18 commits into
masterfrom
screenshotTestsDowngradeJunit
Open

Screenshot tests were failing, JUnit downgrade needed#16578
tobiasKaminsky wants to merge 18 commits into
masterfrom
screenshotTestsDowngradeJunit

Conversation

@tobiasKaminsky

Copy link
Copy Markdown
Member

🖼️ Screenshots

🏚️ Before 🏡 After
B A

🏁 Checklist

  • Tests written, or not not needed

@tobiasKaminsky

Copy link
Copy Markdown
Member Author

13:19: There was 46 failure(s).

@tobiasKaminsky
tobiasKaminsky force-pushed the screenshotTestsDowngradeJunit branch from a946c07 to b8a4f52 Compare March 3, 2026 10:46
@github-actions

github-actions Bot commented Mar 5, 2026

Copy link
Copy Markdown

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/16578.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@tobiasKaminsky
tobiasKaminsky force-pushed the screenshotTestsDowngradeJunit branch from e31058c to 95728bb Compare May 7, 2026 08:37
@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown

Codacy

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness6767
Dodgy code216216
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3434
Performance4242
Security1616
Total420420

@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown

APK file: https://github.com/nextcloud/android/actions/runs/25485231490/artifacts/6851071474
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@tobiasKaminsky
tobiasKaminsky force-pushed the screenshotTestsDowngradeJunit branch from 02ce41b to be6f89e Compare July 15, 2026 09:33
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@github-actions

Copy link
Copy Markdown

blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed.

@tobiasKaminsky

Copy link
Copy Markdown
Member Author
> Task :app:compileQaDebugAndroidTestJavaWithJavac
Hinweis: Einige Eingabedateien verwenden oder überschreiben eine veraltete API.
Hinweis: Wiederholen Sie die Kompilierung mit -Xlint:deprecation, um Details zu erhalten.
Finished 1 tests on uiComparison_-_29(AVD) - 10
Shot ADB warning: /storage/emulated/0/Download/screenshots/com.nextcloud.client.test/screenshots-default/: 19 files pulled, 0 skipped. 7.3 MB/s (737828 bytes in 0.097s)
Shot ADB warning: adb: error: failed to stat remote object '/storage/emulated/0/Download/screenshots/com.nextcloud.client.test/screenshots-compose-default/': No such file or directory

> Task :app:genericDebugDownloadScreenshots
⬇️  Pulling screenshots from your connected devices!
Shot ADB warning: We could not pull screenshots from folder: /storage/emulated/0/Download/screenshots/com.nextcloud.client.test/screenshots-compose-default/
Shot ADB warning: rm: /storage/emulated/0/Download/screenshots/com.nextcloud.client.test/screenshots-compose-default/: No such file or directory                                                                                   

> Task :app:genericDebugExecuteScreenshotTests
🔎  Comparing screenshots with previous ones.
❌  Hummmm...the following screenshot tests are broken:

   📱  The size of the screenshot taken has changed for test: com.nextcloud.client.CommunityFragmentIT_open
            💾  Original screenshot dimension: 500x833. You can find the original screenshot here: /home/tobi/github/nextcloud/android2/app/screenshots/generic/debug/com.nextcloud.client.CommunityFragmentIT_open.png
            🆕  New recorded screenshot dimension: 1080x2280. You can find the new recorded screenshot here: /home/tobi/github/nextcloud/android2/app/build/tmp/shot/screenshot//com.nextcloud.client.CommunityFragmentIT_open.png

                                                                                                                                                                                                                                   
🤔 Do you a need a hand with your automated tests or your Android app?
   Maye I can help you! https://github.com/sponsors/pedrovgs
                                                                                                                                                                                                                                   
🤓  You can review the execution report here: /home/tobi/github/nextcloud/android2/app/build/reports/shot/generic/debug/verification/index.html

Works locally…
I will try to test it with the exact same emulator.

@github-actions

Copy link
Copy Markdown

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant