Skip to content

Commit

Permalink
Manual roll Flutter from 6bba08cbcc51 to 0975e612c04a (1 revision) (#…
Browse files Browse the repository at this point in the history
…7814)

Manual roll requested by stuartmorgan@google.com

flutter/flutter@6bba08c...0975e61

2024-10-01 51940183+Sameri11@users.noreply.github.com [tool][android]
Allow --target-platform work properly with --debug mode
(flutter/flutter#154476)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC camillesimon@google.com,stuartmorgan@google.com on the revert
to ensure that a human
is aware of the problem.

To file a bug in Packages:
https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

---------

Co-authored-by: Stuart Morgan <stuartmorgan@google.com>
  • Loading branch information
engine-flutter-autoroll and stuartmorgan authored Oct 8, 2024
1 parent 1c16d29 commit cebc06c
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .ci/flutter_master.version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
6bba08cbcc516b599602673686813c6f3aa9a372
0975e612c04a5ffe1e840908a56cb4e1dd6f6470
11 changes: 11 additions & 0 deletions script/tool/lib/src/native_test_command.dart
Original file line number Diff line number Diff line change
Expand Up @@ -367,10 +367,21 @@ this command.
'notAnnotation=io.flutter.plugins.DartIntegrationTest';

print('Running integration tests...');
// Explicitly request all ABIs, as Flutter would if being called
// without a specific target (see
// https://github.com/flutter/flutter/pull/154476) to ensure it can
// run on any architecture emulator.
const List<String> abis = <String>[
'android-arm',
'android-arm64',
'android-x64',
'android-x86'
];
final int exitCode = await project.runCommand(
'app:connectedAndroidTest',
arguments: <String>[
'-Pandroid.testInstrumentationRunnerArguments.$filter',
'-Ptarget-platform=${abis.join(',')}',
],
);
if (exitCode != 0) {
Expand Down
14 changes: 10 additions & 4 deletions script/tool/test/native_test_command_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ import 'package:test/test.dart';
import 'mocks.dart';
import 'util.dart';

const String _allAbiFlag =
'-Ptarget-platform=android-arm,android-arm64,android-x64,android-x86';

const String _androidIntegrationTestFilter =
'-Pandroid.testInstrumentationRunnerArguments.'
'notAnnotation=io.flutter.plugins.DartIntegrationTest';
Expand Down Expand Up @@ -563,6 +566,7 @@ void main() {
const <String>[
'app:connectedAndroidTest',
_androidIntegrationTestFilter,
_allAbiFlag,
],
androidFolder.path,
),
Expand Down Expand Up @@ -697,6 +701,7 @@ public class FlutterActivityTest {
const <String>[
'app:connectedAndroidTest',
_androidIntegrationTestFilter,
_allAbiFlag,
],
androidFolder.path,
),
Expand Down Expand Up @@ -734,6 +739,7 @@ public class FlutterActivityTest {
const <String>[
'app:connectedAndroidTest',
_androidIntegrationTestFilter,
_allAbiFlag,
],
androidFolder.path,
),
Expand Down Expand Up @@ -1469,12 +1475,12 @@ public class FlutterActivityTest {
processRunner.mockProcessesForExecutable['xcrun'] = <FakeProcessInfo>[
getMockXcodebuildListProcess(
<String>['RunnerTests', 'RunnerUITests']), // iOS list
FakeProcessInfo(
MockProcess(), <String>['xcodebuild', 'clean', 'test']), // iOS run
FakeProcessInfo(MockProcess(),
<String>['xcodebuild', 'clean', 'test']), // iOS run
getMockXcodebuildListProcess(
<String>['RunnerTests', 'RunnerUITests']), // macOS list
FakeProcessInfo(
MockProcess(), <String>['xcodebuild', 'clean', 'test']), // macOS run
FakeProcessInfo(MockProcess(),
<String>['xcodebuild', 'clean', 'test']), // macOS run
];

final List<String> output = await runCapturingPrint(runner, <String>[
Expand Down

0 comments on commit cebc06c

Please sign in to comment.