Skip to content

Commit 14d9a21

Browse files
[ci] Bring up LUCI Linux custom package tests (flutter#4382)
Adds LUCI version of Linux custom package tests. Removes the custom tests for `cross_file` and `google_identity_services_web` since they only existed to run unit tests in browser mode, which is [now standard for all packages that can be run that way](flutter/packages#4378). Part of flutter#114373
1 parent f145bd6 commit 14d9a21

File tree

6 files changed

+41
-116
lines changed

6 files changed

+41
-116
lines changed

.ci.yaml

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,40 @@ targets:
214214
channel: stable
215215
version_file: flutter_stable.version
216216

217+
- name: Linux_android custom_package_tests master
218+
bringup: true # New target
219+
recipe: packages/packages
220+
timeout: 30
221+
properties:
222+
add_recipes_cq: "true"
223+
version_file: flutter_master.version
224+
target_file: linux_custom_package_tests.yaml
225+
cores: "32"
226+
# Pigeon tests need Andoid deps (thus the Linux_android base) and
227+
# clang-format.
228+
dependencies: >-
229+
[
230+
{"dependency": "clang", "version": "git_revision:5d5aba78dbbee75508f01bcaa69aedb2ab79065a"}
231+
]
232+
channel: master
233+
234+
- name: Linux_android custom_package_tests stable
235+
bringup: true # New target
236+
recipe: packages/packages
237+
timeout: 30
238+
properties:
239+
add_recipes_cq: "true"
240+
version_file: flutter_stable.version
241+
target_file: linux_custom_package_tests.yaml
242+
cores: "32"
243+
# Pigeon tests need Android deps (thus the Linux_android base) and
244+
# clang-format.
245+
dependencies: >-
246+
[
247+
{"dependency": "clang", "version": "git_revision:5d5aba78dbbee75508f01bcaa69aedb2ab79065a"}
248+
]
249+
channel: stable
250+
217251
### Android tasks ###
218252
- name: Linux_android android_build_all_packages master
219253
recipe: packages/packages

.ci/scripts/custom_package_tests.sh

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,5 @@
44
# found in the LICENSE file.
55
set -e
66

7-
# Exclusions
8-
#
9-
# script/configs/linux_only_custom_test.yaml
10-
# Custom tests need Chrome. (They run in linux-custom_package_tests)
11-
127
dart ./script/tool/bin/flutter_plugin_tools.dart custom-test \
13-
--packages-for-branch --log-timing \
14-
--exclude=script/configs/linux_only_custom_test.yaml
8+
--packages-for-branch --log-timing
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
tasks:
2+
- name: prepare tool
3+
script: .ci/scripts/prepare_tool.sh
4+
- name: custom tests
5+
script: script/tool_runner.sh
6+
args: ["custom-test"]

packages/cross_file/tool/run_tests.dart

Lines changed: 0 additions & 50 deletions
This file was deleted.

packages/google_identity_services_web/tool/run_tests.dart

Lines changed: 0 additions & 50 deletions
This file was deleted.

script/configs/linux_only_custom_test.yaml

Lines changed: 0 additions & 9 deletions
This file was deleted.

0 commit comments

Comments
 (0)