Skip to content

Commit

Permalink
[Android] Fix the run_debug_test_android device lab test (#117016)
Browse files Browse the repository at this point in the history
* Fix

* Format

* DO NOT MERGE! Test in presubmit

* Revert "DO NOT MERGE! Test in presubmit"

This reverts commit 41c7186c723d844c288cb4e5fab3284a803f82ae.
  • Loading branch information
loic-sharma committed Dec 14, 2022
1 parent f07db40 commit 028c6e2
Showing 1 changed file with 13 additions and 7 deletions.
20 changes: 13 additions & 7 deletions dev/devicelab/lib/tasks/run_tests.dart
Expand Up @@ -102,28 +102,34 @@ class AndroidRunOutputTest extends RunOutputTask {

@override
TaskResult verify(List<String> stdout, List<String> stderr) {
final String gradleTask = release ? 'assembleRelease' : 'assembleDebug';
final String apk = release ? 'app-release.apk' : 'app-debug.apk';

_findNextMatcherInList(
stdout,
(String line) => line.startsWith('Launching lib/main.dart on ') && line.endsWith(' in release mode...'),
(String line) => line.startsWith('Launching lib/main.dart on ') &&
line.endsWith(' in ${release ? 'release' : 'debug'} mode...'),
'Launching lib/main.dart on',
);

_findNextMatcherInList(
stdout,
(String line) => line.startsWith("Running Gradle task 'assembleRelease'..."),
"Running Gradle task 'assembleRelease'...",
(String line) => line.startsWith("Running Gradle task '$gradleTask'..."),
"Running Gradle task '$gradleTask'...",
);

// Size information is only included in release builds.
_findNextMatcherInList(
stdout,
(String line) => line.contains('Built build/app/outputs/flutter-apk/app-release.apk (') && line.contains('MB).'),
'Built build/app/outputs/flutter-apk/app-release.apk',
(String line) => line.contains('Built build/app/outputs/flutter-apk/$apk') &&
(!release || line.contains('MB).')),
'Built build/app/outputs/flutter-apk/$apk',
);

_findNextMatcherInList(
stdout,
(String line) => line.startsWith('Installing build/app/outputs/flutter-apk/app-release.apk...'),
'Installing build/app/outputs/flutter-apk/app-release.apk...',
(String line) => line.startsWith('Installing build/app/outputs/flutter-apk/$apk...'),
'Installing build/app/outputs/flutter-apk/$apk...',
);

_findNextMatcherInList(
Expand Down

0 comments on commit 028c6e2

Please sign in to comment.