diff --git a/packages/flutter_migrate/test/compute_test.dart b/packages/flutter_migrate/test/compute_test.dart index eea7200a7a985..523162cad38d2 100644 --- a/packages/flutter_migrate/test/compute_test.dart +++ b/packages/flutter_migrate/test/compute_test.dart @@ -251,7 +251,10 @@ void main() { .existsSync(), false); }, timeout: const Timeout(Duration(seconds: 500))); - }); + }, + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: 'TODO: Speed up, or move to another type of test'); group('MigrateRevisions', () { setUp(() async { @@ -486,7 +489,10 @@ migration: .baseRevision, '36427af29421f406ac95ff55ea31d1dc49a45b5f'); }); - }); + }, + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: 'TODO: Speed up, or move to another type of test'); group('project operations', () { setUp(() async { @@ -865,5 +871,8 @@ migration: expect(result.mergeResults[10].hasConflict, false); expect(result.mergeResults[11].hasConflict, false); }, timeout: const Timeout(Duration(seconds: 500))); - }); + }, + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: 'TODO: Speed up, or move to another type of test'); } diff --git a/packages/flutter_migrate/test/migrate_test.dart b/packages/flutter_migrate/test/migrate_test.dart index 10158ea3fd146..beeddb329c860 100644 --- a/packages/flutter_migrate/test/migrate_test.dart +++ b/packages/flutter_migrate/test/migrate_test.dart @@ -106,7 +106,11 @@ Modified files: .existsSync(), true); expect(tempDir.childFile('analysis_options.yaml').existsSync(), true); - }, timeout: const Timeout(Duration(seconds: 500)), skip: isWindows); + }, + timeout: const Timeout(Duration(seconds: 500)), + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: true); // Migrates a clean untouched app generated with flutter create testUsingContext('vanilla migrate builds', () async { @@ -167,7 +171,11 @@ class MyApp extends StatelessWidget { // Skipped due to being flaky, the build completes successfully, but sometimes // Gradle crashes due to resources on the bot. We should fine tune this to // make it stable. - }, timeout: const Timeout(Duration(seconds: 900)), skip: true); + }, + timeout: const Timeout(Duration(seconds: 900)), + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: true); testUsingContext('migrate abandon', () async { // Abandon in an empty dir fails. @@ -209,7 +217,11 @@ class MyApp extends StatelessWidget { ], workingDirectory: tempDir.path); expect(result.exitCode, 0); expect(result.stdout.toString(), contains('Abandon complete')); - }, timeout: const Timeout(Duration(seconds: 300))); + }, + timeout: const Timeout(Duration(seconds: 300)), + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: true); // Migrates a user-modified app testUsingContext('modified migrate process succeeds', () async { @@ -436,5 +448,9 @@ flutter: .existsSync(), true); expect(tempDir.childFile('analysis_options.yaml').existsSync(), true); - }, timeout: const Timeout(Duration(seconds: 500)), skip: isWindows); + }, + timeout: const Timeout(Duration(seconds: 500)), + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: true); } diff --git a/packages/flutter_migrate/test/utils_test.dart b/packages/flutter_migrate/test/utils_test.dart index 35a0bf50e9e76..4c52156d9a688 100644 --- a/packages/flutter_migrate/test/utils_test.dart +++ b/packages/flutter_migrate/test/utils_test.dart @@ -281,7 +281,11 @@ void main() { projectRoot.deleteSync(recursive: true); }); - }, timeout: const Timeout(Duration(seconds: 500))); + }, + timeout: const Timeout(Duration(seconds: 500)), + // TODO(stuartmorgan): These should not be unit tests, see + // https://github.com/flutter/flutter/issues/121257. + skip: 'TODO: Speed up, or move to another type of test'); testWithoutContext('conflictsResolved', () async { expect(utils.conflictsResolved(''), true);