Skip to content

and <pr_description> are required. <pr_description> should be GitHub markdown description body.#188

Closed
Copilot wants to merge 21 commits intomainfrom
copilot/fix-main-branch-issue
Closed

and <pr_description> are required. <pr_description> should be GitHub markdown description body.#188
Copilot wants to merge 21 commits intomainfrom
copilot/fix-main-branch-issue

Conversation

Copy link

Copilot AI commented Feb 10, 2026

DO NOT CREATE A PULL REQUEST WITHOUT READING THESE INSTRUCTIONS

Instructions

Thanks for submitting a pull request. To accept your pull request we need you do a few things:

If this is your first pull request

Ensure tests pass and code is formatted correctly

  • Run local tests on the DemoDebug variant by running ./gradlew testDemoDebug
  • Fix code formatting: ./gradlew --init-script gradle/init.gradle.kts spotlessApply

Add a description

The instrumentation suite was timing out on navigation flows and failed with back-press focus errors; Spotless flagged formatting in NavigationTest.kt. This change seeds data before tests, hardens back-press handling, and keeps formatting compliant.

What I have done and why

  • Test data readiness

    • Sync topics/news repositories during NavigationTest setup to guarantee content before UI waits.
  • Back-press resilience

    • Centralize expected back-press exceptions without direct internal Espresso imports; assert activity termination deterministically.
  • Runtime dependencies

    • Add AndroidX test core/runner to instrumentation dependencies to resolve ActivityInvoker class loading failures.
runBlocking {
    val synchronizer = TestSynchronizer(niaPreferencesDataSource)
    withTimeout(DATA_LOAD_TIMEOUT_MILLIS) {
        topicsRepository.syncWith(synchronizer)
        newsRepository.syncWith(synchronizer)
    }
}

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • dl.google.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.invoke=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED --add-exports=jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED --add-opens=java.base/java.nio.charset=ALL-UNNAMED --add-opens=java.base/java.net=ALL-UNNAMED --add-opens=java.base/java.util.concurrent.atomic=ALL-UNNAMED --add-opens=java.xml/javax.xml.namespace=ALL-UNNAMED -XX:&#43;UseG1GC -XX:SoftRefLRUPolicyMSPerMB=1 -XX:ReservedCodeCacheSize=256m -XX:&#43;HeapDumpOnOutOfMemoryError -Xms4g -Xmx4g -Dfile.encoding=UTF-8 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 9 commits February 10, 2026 18:57
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate and fix issue in main branch Fix sync delegation and stabilize instrumentation tests Feb 10, 2026
Copilot AI requested a review from lihenggui February 10, 2026 19:10
Copilot AI and others added 3 commits February 10, 2026 19:42
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Copilot AI changed the title Fix sync delegation and stabilize instrumentation tests and <pr_description> are required. <pr_description> should be GitHub markdown description body. Feb 10, 2026
Copilot AI and others added 7 commits February 10, 2026 20:57
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
Co-authored-by: lihenggui <5171312+lihenggui@users.noreply.github.com>
@lihenggui lihenggui closed this Feb 12, 2026
@lihenggui lihenggui deleted the copilot/fix-main-branch-issue branch February 12, 2026 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants