[WIP] Patrol POC local Android by tddang-linagora · Pull Request #4469 · linagora/tmail-flutter

tddang-linagora · 2026-04-23T09:50:59Z

Description

Remove the dependency of Firebase Test Lab
Use https://github.com/ReactiveCircus/android-emulator-runner
Reset back-end after every test, ensuring isolation
Fix some flaky tests (there maybe more)

Replace GCloud auth, Firebase Test Lab, and ngrok with a local Android AVD via reactivecircus/android-emulator-runner on CI. The backend runs in Docker and is reachable from the emulator at the QEMU host alias 10.0.2.2. CI / scripts: - Remove Google Cloud auth, Firebase run step, and ngrok tunnel - Add KVM, Gradle, and AVD cache setup; shard tests across 4 parallel runners - Scripts: drop ngrok, point jmap.properties to 10.0.2.2, manage cleanup via trap - prebuild.sh: parallelize module builds, suppress noisy output - provisioning.sh: parallelize james-cli/curl calls, fix hardcoded container IP Backend reset for test isolation: - Add backend-reset-server.py: HTTP server that restarts tmail-backend and re-runs provisioning on POST /reset, so each test starts with a clean state - Add BackendResetClient.dart: Dart client called in patrolTearDown - Reduce visibleTimeout 30s → 10s; allow insecure connections in debug mode

Replace fixed pumpAndSettle delays with waitUntilVisible/waitUntilExists so tests react to actual UI state rather than arbitrary timeouts. - Add PatrolFileExtensions.toFileInfo() and use it in composer robot/scenario utils - Guard MobileEditorWidget.dispose() under isIntegrationTesting to prevent teardown crash - Fix scroll direction (AxisDirection.down) in calendar attendee button scroll - Add scrollTo() before tap in label modal robot and edit-tag scenario - Wait on EmailTileBuilder (not label name) after email provisioning — more reliable - Inline image: wait for HtmlContentViewer to contain base64 data before asserting - Use native platform wait for inline image in upload scenario - Remove unnecessary Future.delayed and extra pumpAndSettle calls - Pre-select English before switching language to ensure deterministic starting state - Fix app_grid back-press sequence

coderabbitai · 2026-04-23T09:51:07Z

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7118ace8-1609-4746-92bd-2c12f345634a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch patrol/local-docker-android

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

github-actions · 2026-04-23T10:10:19Z

This PR has been deployed to https://linagora.github.io/tmail-flutter/4469.

codescene-delta-analysis

Gates Passed
3 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: The Bare Minimum
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

tddang-linagora added 2 commits April 23, 2026 16:48

tddang-linagora requested review from dab246 and hoangdat April 23, 2026 09:51

tddang-linagora self-assigned this Apr 23, 2026

tddang-linagora temporarily deployed to PR-4469 April 23, 2026 09:51 — with GitHub Actions Inactive

github-advanced-security AI found potential problems Apr 23, 2026

View reviewed changes

Comment thread .github/workflows/patrol-integration-test.yaml Fixed

Comment thread .github/workflows/patrol-integration-test.yaml Fixed