nextcloud / nextcloud/android

Improving Stability in Screenshot Tests: Replacing Deprecated APIs and Removing Flaky Delays

Open
#13,618 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Kotlin
Stars
5.6k
Forks
2k
Avg merge
2d 18h
Merged PRs (30d)
92

Description

**Problem:**

IntentsTestRule is a deprecated API and is used for almost all screenshot tests. Additionally, usage of Thread.sleep() leads to flakiness and unreliable screenshot tests.

**Solution:**

To improve stability and reliability, we should replace Thread.sleep() with EspressoIdlingResource. Furthermore, the deprecated IntentsTestRule should be replaced with ActivityScenario.

TrashbinActivityIT can be used as an example.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.