Skip to content

Commit 9848181

Browse files
stuartmorganstuartmorgan
authored
Bring up new Windows test targets (flutter#4311)
Uses the new generic plugins recipe to add a number of new Windows tests that have recently gained tool support, but not yet added to the LUCI builds: - New targets: - The "build all plugins" test (master and stable) - UWP build tests (master only, since UWP is not on stable yet) - Tool tests (master only with existing tool tests on Linux) - Replacement versions of the existing builders but with names that match the Cirrus naming to make the parallels in testing easier to understand in the GitHub UI and configs. - Modification of existing targets: - Adds Windows native unit tests to the existing script.
1 parent ae92e62 commit 9848181

11 files changed

+143
-5
lines changed

‎.ci.yaml

+84
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ platform_properties:
3535
os: Windows
3636

3737
targets:
38+
# TODO(stuartmorgan) Remove once the renamed version below has propagated.
3839
- name: Windows Plugins master channel
3940
recipe: plugins/plugins
4041
timeout: 30
@@ -47,6 +48,7 @@ targets:
4748
]
4849
scheduler: luci
4950

51+
# TODO(stuartmorgan) Remove once the renamed version below has propagated.
5052
- name: Windows Plugins stable channel
5153
recipe: plugins/plugins
5254
timeout: 30
@@ -60,6 +62,88 @@ targets:
6062
]
6163
scheduler: luci
6264

65+
- name: Windows win32_build+platform-tests master
66+
recipe: plugins/plugins
67+
bringup: true
68+
timeout: 30
69+
properties:
70+
# TODO(stuartmorgan): Uncomment when removing bringup.
71+
#add_recipes_cq: "true"
72+
target_file: windows_build_and_platform_tests.yaml
73+
dependencies: >
74+
[
75+
{"dependency": "vs_build"}
76+
]
77+
scheduler: luci
78+
79+
- name: Windows win32_build+platform-tests stable
80+
recipe: plugins/plugins
81+
bringup: true
82+
timeout: 30
83+
properties:
84+
# TODO(stuartmorgan): Uncomment when removing bringup.
85+
#add_recipes_cq: "true"
86+
target_file: uwp_build_and_platform_tests.yaml
87+
channel: stable
88+
dependencies: >
89+
[
90+
{"dependency": "vs_build"}
91+
]
92+
scheduler: luci
93+
94+
- name: Windows build_all_plugins master
95+
recipe: plugins/plugins
96+
bringup: true
97+
timeout: 30
98+
properties:
99+
# TODO(stuartmorgan): Uncomment when removing bringup.
100+
#add_recipes_cq: "true"
101+
target_file: build_all_plugins.yaml
102+
dependencies: >
103+
[
104+
{"dependency": "vs_build"}
105+
]
106+
scheduler: luci
107+
108+
- name: Windows build_all_plugins stable
109+
recipe: plugins/plugins
110+
bringup: true
111+
timeout: 30
112+
properties:
113+
# TODO(stuartmorgan): Uncomment when removing bringup.
114+
#add_recipes_cq: "true"
115+
target_file: build_all_plugins.yaml
116+
channel: stable
117+
dependencies: >
118+
[
119+
{"dependency": "vs_build"}
120+
]
121+
scheduler: luci
122+
123+
- name: Windows uwp-build+platform-tests master
124+
recipe: plugins/plugins
125+
bringup: true
126+
timeout: 30
127+
properties:
128+
# TODO(stuartmorgan): Uncomment when removing bringup.
129+
#add_recipes_cq: "true"
130+
target_file: uwp_build_and_platform_tests.yaml
131+
dependencies: >
132+
[
133+
{"dependency": "vs_build"}
134+
]
135+
scheduler: luci
136+
137+
- name: Windows plugin_tools_tests
138+
recipe: plugins/plugins
139+
bringup: true
140+
timeout: 30
141+
properties:
142+
# TODO(stuartmorgan): Uncomment when removing bringup.
143+
#add_recipes_cq: "true"
144+
target_file: plugin_tools_tests.yaml
145+
scheduler: luci
146+
63147
- name: Linux ci_yaml plugins roller
64148
recipe: infra/ci_yaml
65149
timeout: 30

‎.ci/scripts/build_all_plugins.sh

+8
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
#!/bin/bash
2+
# Copyright 2013 The Flutter Authors. All rights reserved.
3+
# Use of this source code is governed by a BSD-style license that can be
4+
# found in the LICENSE file.
5+
6+
cd all_plugins
7+
flutter build windows --debug
8+
flutter build windows --release

‎.ci/scripts/build_examples_uwp.sh

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
#!/bin/bash
2+
# Copyright 2013 The Flutter Authors. All rights reserved.
3+
# Use of this source code is governed by a BSD-style license that can be
4+
# found in the LICENSE file.
5+
6+
dart ./script/tool/bin/flutter_plugin_tools.dart build-examples --winuwp \
7+
--packages-for-branch

‎.ci/scripts/create_all_plugins_app.sh

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
#!/bin/bash
2+
# Copyright 2013 The Flutter Authors. All rights reserved.
3+
# Use of this source code is governed by a BSD-style license that can be
4+
# found in the LICENSE file.
5+
6+
dart ./script/tool/bin/flutter_plugin_tools.dart all-plugins-app \
7+
--output-dir=. --exclude script/configs/exclude_all_plugins_app.yaml

‎.ci/scripts/native_test_win32.sh

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
#!/bin/bash
2+
# Copyright 2013 The Flutter Authors. All rights reserved.
3+
# Use of this source code is governed by a BSD-style license that can be
4+
# found in the LICENSE file.
5+
6+
dart ./script/tool/bin/flutter_plugin_tools.dart native-test --windows \
7+
--no-integration --packages-for-branch

‎.ci/scripts/plugin_tools_tests.sh

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
#!/bin/bash
2+
# Copyright 2013 The Flutter Authors. All rights reserved.
3+
# Use of this source code is governed by a BSD-style license that can be
4+
# found in the LICENSE file.
5+
6+
cd script/tool
7+
dart pub run test

‎.ci/targets/build_all_plugins.yaml

+7
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
tasks:
2+
- name: prepare tool
3+
script: .ci/scripts/prepare_tool.sh
4+
- name: create all_plugins app
5+
script: .ci/scripts/create_all_plugins_app.sh
6+
- name: build all_plugins
7+
script: .ci/scripts/build_all_plugins.sh

‎.ci/targets/plugin_tools_tests.yaml

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
tasks:
2+
- name: prepare tool
3+
script: .ci/scripts/prepare_tool.sh
4+
- name: tool unit tests
5+
script: .ci/scripts/plugin_tools_tests.sh

‎.ci/targets/uwp_build_and_platform_tests.yaml

+5
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
tasks:
2+
- name: prepare tool
3+
script: .ci/scripts/prepare_tool.sh
4+
- name: build examples (UWP)
5+
script: .ci/scripts/build_examples_uwp.sh

‎.ci/targets/windows_build_and_platform_tests.yaml

+5-4
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
1-
21
tasks:
3-
- name: "prepare tool"
2+
- name: prepare tool
43
script: .ci/scripts/prepare_tool.sh
5-
- name: "build examples"
4+
- name: build examples (Win32)
65
script: .ci/scripts/build_examples_win32.sh
7-
- name: "drive examples"
6+
- name: native unit tests (Win32)
7+
script: .ci/scripts/native_test_win32.sh
8+
- name: drive examples (Win32)
89
script: .ci/scripts/drive_examples_win32.sh

‎.cirrus.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ task:
6666
- name: plugin_tools_tests
6767
script:
6868
- cd script/tool
69-
- CIRRUS_BUILD_ID=null pub run test
69+
- dart pub run test
7070
- name: publishable
7171
version_check_script: ./script/tool_runner.sh version-check
7272
publish_check_script: ./script/tool_runner.sh publish-check

0 commit comments

Comments
 (0)