Skip to content

Commit 5efd759

Browse files
authored
Remove now unused apk-health-tests. Can always re-add in the future. (#159349)
flutter/flutter#159000 resolved (for now).
1 parent 5e2a62a commit 5efd759

12 files changed

+0
-73
lines changed

.ci.yaml

Lines changed: 0 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1297,29 +1297,6 @@ targets:
12971297
- bin/**
12981298
- .ci.yaml
12991299

1300-
- name: Linux flutter_build_apk_health_tests
1301-
recipe: flutter/flutter_drone
1302-
# WARNING: Do *NOT* enable. This intended for testing on CI only.
1303-
# Ask matanlurey@ or @gmackall if you have any questions.
1304-
bringup: true
1305-
timeout: 60
1306-
properties:
1307-
add_recipes_cq: "true"
1308-
dependencies: >-
1309-
[
1310-
{"dependency": "android_sdk", "version": "version:35v1"},
1311-
{"dependency": "chrome_and_driver", "version": "version:125.0.6422.141"},
1312-
{"dependency": "clang", "version": "git_revision:5d5aba78dbbee75508f01bcaa69aedb2ab79065a"},
1313-
{"dependency": "cmake", "version": "build_id:8787856497187628321"},
1314-
{"dependency": "goldctl", "version": "git_revision:2387d6fff449587eecbb7e45b2692ca0710b63b9"},
1315-
{"dependency": "ninja", "version": "version:1.9.0"},
1316-
{"dependency": "open_jdk", "version": "version:17"}
1317-
]
1318-
shard: flutter_build_apk_health_tests
1319-
tags: >
1320-
["framework", "hostonly", "shard", "linux"]
1321-
test_timeout_secs: "2700"
1322-
13231300
- name: Linux android_preview_tool_integration_tests
13241301
recipe: flutter/flutter_drone
13251302
timeout: 60

TESTOWNERS

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,6 @@
327327
# coverage @goderbauer @flutter/infra
328328
# customer_testing @Piinks @flutter/framework
329329
# docs @Piinks @flutter/framework
330-
# flutter_build_apk_health_tests @matanlurey @gmackall
331330
# flutter_driver_android_test @matanlurey @johnmccutchan
332331
# flutter_packaging @christopherfujino @flutter/infra
333332
# flutter_plugins @stuartmorgan @flutter/plugin

dev/bots/test.dart

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -136,7 +136,6 @@ Future<void> main(List<String> args) async {
136136
'tool_tests': _runToolTests,
137137
'web_tool_tests': _runWebToolTests,
138138
'tool_integration_tests': _runIntegrationToolTests,
139-
'flutter_build_apk_health_tests': _runFlutterBuildApkHealthTests,
140139
'android_preview_tool_integration_tests': androidPreviewIntegrationToolTestsRunner,
141140
'android_java11_tool_integration_tests': androidJava11IntegrationToolTestsRunner,
142141
'tool_host_cross_arch_tests': _runToolHostCrossArchTests,
@@ -239,19 +238,6 @@ Future<void> _runIntegrationToolTests() async {
239238
);
240239
}
241240

242-
Future<void> _runFlutterBuildApkHealthTests() async {
243-
await runDartTest(
244-
_toolsPath,
245-
forceSingleCore: true,
246-
testPaths: <String>[
247-
path.join(_toolsPath, 'test', 'integration.shard'),
248-
],
249-
collectMetrics: true,
250-
runSkipped: true,
251-
tags: <String>['flutter-build-apk'],
252-
);
253-
}
254-
255241
Future<void> _runToolTests() async {
256242
await selectSubshard(<String, ShardRunner>{
257243
'general': _runGeneralToolTests,

packages/flutter_tools/test/integration.shard/android_e2e_api_test.dart

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
// TODO(matanlurey): Remove after debugging https://github.com/flutter/flutter/issues/159000.
6-
@Tags(<String>['flutter-build-apk'])
7-
library;
8-
95
import 'package:file/file.dart';
106
import 'package:flutter_tools/src/base/io.dart';
117

packages/flutter_tools/test/integration.shard/android_gradle_deprecated_plugin_apply_test.dart

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
// TODO(matanlurey): Remove after debugging https://github.com/flutter/flutter/issues/159000.
6-
@Tags(<String>['flutter-build-apk'])
7-
library;
8-
95
import 'dart:convert';
106

117
import 'package:file/file.dart';

packages/flutter_tools/test/integration.shard/android_gradle_legacy_flutter_plugins_strings_in_comments_test.dart

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
// TODO(matanlurey): Remove after debugging https://github.com/flutter/flutter/issues/159000.
6-
@Tags(<String>['flutter-build-apk'])
7-
library;
8-
95
// This test can be removed once https://github.com/flutter/flutter/issues/155484 is resolved.
106

117
import 'package:flutter_tools/src/base/file_system.dart';

packages/flutter_tools/test/integration.shard/android_gradle_print_build_variants_test.dart

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
// TODO(matanlurey): Remove after debugging https://github.com/flutter/flutter/issues/159000.
6-
@Tags(<String>['flutter-build-apk'])
7-
library;
8-
95
import 'dart:convert';
106

117
import 'package:collection/collection.dart';

packages/flutter_tools/test/integration.shard/android_plugin_example_app_build_test.dart

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
// TODO(matanlurey): Remove after debugging https://github.com/flutter/flutter/issues/159000.
6-
@Tags(<String>['flutter-build-apk'])
7-
library;
8-
95
import 'package:file_testing/file_testing.dart';
106
import 'package:flutter_tools/src/base/file_system.dart';
117
import 'package:flutter_tools/src/base/io.dart';

packages/flutter_tools/test/integration.shard/flutter_build_with_compilation_error_test.dart

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
// TODO(matanlurey): Remove after debugging https://github.com/flutter/flutter/issues/159000.
6-
@Tags(<String>['flutter-build-apk'])
7-
library;
8-
95
import 'package:file/file.dart';
106
import 'package:flutter_tools/src/base/io.dart';
117

packages/flutter_tools/test/integration.shard/isolated/native_assets_agp_version_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,6 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5-
// TODO(matanlurey): Remove after debugging https://github.com/flutter/flutter/issues/159000.
6-
@Tags(<String>['flutter-build-apk'])
7-
@Timeout(Duration(minutes: 10))
8-
library;
9-
105
import 'dart:io';
116

127
import 'package:file/file.dart';

0 commit comments

Comments
 (0)