[Enhancement]: Improve Instrumented Test Workflow Matrix & Failure Handling

[theMr17]

Summary of Improvement

• Refactor the GitHub Actions workflow for instrumented tests by using a cleaner matrix strategy and ensuring that test failures correctly fail the workflow.
• Also ensure API 33 is added to the matrix to support notification-related tests (introduced by Fix #20, #24: Add Android system notification support with notification channels #23).

Current Implementation

• The current instrumented_tests job uses a hardcoded API level (api-level: [29]) without architecture control.
• It uses reactivecircus/android-emulator-runner@v2 to spin up an emulator and run tests.
• However:
  • There's no matrix for different arch/API combinations—limiting coverage.
  • API 33 is excluded, even though it's required for tests related to notifications.
  • Test command uses:

    ./gradlew connectedAndroidTest --daemon && killall -INT crashpad_handler || true

    The use of || true can silently ignore test failures, resulting in false-positive passes in CI.

Proposed Solution

• Expand the matrix strategy with specific API/arch combinations using include, and drop reliance on a single hardcoded API level:

  strategy:
    matrix:
      include:
        - api-level: 29
          arch: x86
        - api-level: 33
          arch: x86_64

• Ensure API 33 is part of the matrix to support notification-related test coverage.
• Remove || true from the test command to allow failures to surface properly:

  ./gradlew connectedAndroidTest --daemon

• Optional: switch to connectedDebugAndroidTest instead of the default connectedAndroidTest if that aligns with your Gradle configuration.