Skip to content

Commit b5e8daa

Browse files
Manual roll Flutter from 911aa75 to 043b719 (flutter#8693)
flutter/packages#8692 with fixes for indirect effects of `--explicit-package-dependencies`
1 parent a876ab3 commit b5e8daa

4 files changed

Lines changed: 63 additions & 21 deletions

File tree

.ci/flutter_master.version

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
911aa7547ed732052f98b79f7b1584bd137bda43
1+
043b71954ce73f727f3cb6f5ccd549cc005b1310

packages/flutter_plugin_android_lifecycle/example/android/app/src/main/java/io/flutter/plugins/flutter_plugin_android_lifecycle_example/MainActivity.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
import android.util.Log;
88
import androidx.lifecycle.Lifecycle;
9-
import dev.flutter.plugins.integration_test.IntegrationTestPlugin;
109
import io.flutter.embedding.android.FlutterActivity;
1110
import io.flutter.embedding.engine.FlutterEngine;
1211
import io.flutter.embedding.engine.plugins.FlutterPlugin;
@@ -19,8 +18,8 @@ public class MainActivity extends FlutterActivity {
1918

2019
@Override
2120
public void configureFlutterEngine(FlutterEngine flutterEngine) {
21+
super.configureFlutterEngine(flutterEngine);
2222
flutterEngine.getPlugins().add(new TestPlugin());
23-
flutterEngine.getPlugins().add(new IntegrationTestPlugin());
2423
}
2524

2625
private static class TestPlugin implements FlutterPlugin, ActivityAware {

script/tool/lib/src/firebase_test_lab_command.dart

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -245,22 +245,23 @@ class FirebaseTestLabCommand extends PackageLoopingCommand {
245245
///
246246
/// Returns true if either gradlew was already present, or the build succeeds.
247247
Future<bool> _ensureGradleWrapperExists(GradleProject project) async {
248-
if (!project.isConfigured()) {
249-
print('Running flutter build apk...');
250-
final String experiment = getStringArg(kEnableExperiment);
251-
final int exitCode = await processRunner.runAndStream(
252-
flutterCommand,
253-
<String>[
254-
'build',
255-
'apk',
256-
'--config-only',
257-
if (experiment.isNotEmpty) '--enable-experiment=$experiment',
258-
],
259-
workingDir: project.androidDirectory);
260-
261-
if (exitCode != 0) {
262-
return false;
263-
}
248+
// Unconditionally re-run build with --debug --config-only, to ensure that
249+
// the project is in a debug state even if it was previously configured.
250+
print('Running flutter build apk...');
251+
final String experiment = getStringArg(kEnableExperiment);
252+
final int exitCode = await processRunner.runAndStream(
253+
flutterCommand,
254+
<String>[
255+
'build',
256+
'apk',
257+
'--debug',
258+
'--config-only',
259+
if (experiment.isNotEmpty) '--enable-experiment=$experiment',
260+
],
261+
workingDir: project.androidDirectory);
262+
263+
if (exitCode != 0) {
264+
return false;
264265
}
265266
return true;
266267
}

script/tool/test/firebase_test_lab_command_test.dart

Lines changed: 44 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,14 @@ public class MainActivityTest {
164164
expect(
165165
processRunner.recordedCalls,
166166
orderedEquals(<ProcessCall>[
167+
ProcessCall(
168+
'flutter',
169+
const <String>['build', 'apk', '--debug', '--config-only'],
170+
plugin1
171+
.getExamples()
172+
.first
173+
.platformDirectory(FlutterPlatform.android)
174+
.path),
167175
ProcessCall(
168176
'gcloud',
169177
'auth activate-service-account --key-file=/path/to/key'
@@ -185,6 +193,14 @@ public class MainActivityTest {
185193
'firebase test android run --type instrumentation --app build/app/outputs/apk/debug/app-debug.apk --test build/app/outputs/apk/androidTest/debug/app-debug-androidTest.apk --timeout 7m --results-bucket=gs://a_bucket --results-dir=plugins_android_test/plugin1/buildId/testRunId/example/0/ --device model=redfin,version=30 --device model=seoul,version=26'
186194
.split(' '),
187195
'/packages/plugin1/example'),
196+
ProcessCall(
197+
'flutter',
198+
const <String>['build', 'apk', '--debug', '--config-only'],
199+
plugin2
200+
.getExamples()
201+
.first
202+
.platformDirectory(FlutterPlatform.android)
203+
.path),
188204
ProcessCall(
189205
'/packages/plugin2/example/android/gradlew',
190206
'app:assembleAndroidTest -Pverbose=true'.split(' '),
@@ -245,6 +261,14 @@ public class MainActivityTest {
245261
expect(
246262
processRunner.recordedCalls,
247263
orderedEquals(<ProcessCall>[
264+
ProcessCall(
265+
'flutter',
266+
const <String>['build', 'apk', '--debug', '--config-only'],
267+
plugin
268+
.getExamples()
269+
.first
270+
.platformDirectory(FlutterPlatform.android)
271+
.path),
248272
ProcessCall(
249273
'/packages/plugin/example/android/gradlew',
250274
'app:assembleAndroidTest -Pverbose=true'.split(' '),
@@ -669,8 +693,12 @@ class MainActivityTest {
669693
orderedEquals(<ProcessCall>[
670694
ProcessCall(
671695
'flutter',
672-
'build apk --config-only'.split(' '),
673-
'/packages/plugin/example/android',
696+
'build apk --debug --config-only'.split(' '),
697+
plugin
698+
.getExamples()
699+
.first
700+
.platformDirectory(FlutterPlatform.android)
701+
.path,
674702
),
675703
ProcessCall(
676704
'/packages/plugin/example/android/gradlew',
@@ -841,6 +869,20 @@ class MainActivityTest {
841869
expect(
842870
processRunner.recordedCalls,
843871
orderedEquals(<ProcessCall>[
872+
ProcessCall(
873+
'flutter',
874+
const <String>[
875+
'build',
876+
'apk',
877+
'--debug',
878+
'--config-only',
879+
'--enable-experiment=exp1'
880+
],
881+
plugin
882+
.getExamples()
883+
.first
884+
.platformDirectory(FlutterPlatform.android)
885+
.path),
844886
ProcessCall(
845887
'/packages/plugin/example/android/gradlew',
846888
'app:assembleAndroidTest -Pverbose=true -Pextra-front-end-options=--enable-experiment%3Dexp1 -Pextra-gen-snapshot-options=--enable-experiment%3Dexp1'

0 commit comments

Comments
 (0)