Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Bring up new Windows test targets #4311

Merged
merged 6 commits into from
Sep 8, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
84 changes: 84 additions & 0 deletions .ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ platform_properties:
os: Windows

targets:
# TODO(stuartmorgan) Remove once the renamed version below has propagated.
- name: Windows Plugins master channel
recipe: plugins/plugins
timeout: 30
Expand All @@ -47,6 +48,7 @@ targets:
]
scheduler: luci

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

- name: Windows win32_build+platform-tests master
recipe: plugins/plugins
bringup: true
timeout: 30
properties:
# TODO(stuartmorgan): Uncomment when removing bringup.
#add_recipes_cq: "true"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this necessary? My expectation is this could land in this PR

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably not for this particular one since it's a copy-paste, but in general I would rather make sure a new target actually works in the wild before adding it to the recipe CQ since I really don't want to bring the recipes repo to a standstill by throwing untested things into its pipeline for landing changes.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a really good insight! I'll have to add it to the documentation (and later as a test)

target_file: windows_build_and_platform_tests.yaml
dependencies: >
[
{"dependency": "vs_build"}
]
scheduler: luci

- name: Windows win32_build+platform-tests stable
recipe: plugins/plugins
bringup: true
timeout: 30
properties:
# TODO(stuartmorgan): Uncomment when removing bringup.
#add_recipes_cq: "true"
target_file: uwp_build_and_platform_tests.yaml
channel: stable
dependencies: >
[
{"dependency": "vs_build"}
]
scheduler: luci

- name: Windows build_all_plugins master
recipe: plugins/plugins
bringup: true
timeout: 30
properties:
# TODO(stuartmorgan): Uncomment when removing bringup.
#add_recipes_cq: "true"
target_file: build_all_plugins.yaml
dependencies: >
[
{"dependency": "vs_build"}
]
scheduler: luci

- name: Windows build_all_plugins stable
recipe: plugins/plugins
bringup: true
timeout: 30
properties:
# TODO(stuartmorgan): Uncomment when removing bringup.
#add_recipes_cq: "true"
target_file: build_all_plugins.yaml
channel: stable
dependencies: >
[
{"dependency": "vs_build"}
]
scheduler: luci

- name: Windows uwp-build+platform-tests master
recipe: plugins/plugins
bringup: true
timeout: 30
properties:
# TODO(stuartmorgan): Uncomment when removing bringup.
#add_recipes_cq: "true"
target_file: uwp_build_and_platform_tests.yaml
dependencies: >
[
{"dependency": "vs_build"}
]
scheduler: luci

- name: Windows plugin_tools_tests
recipe: plugins/plugins
bringup: true
timeout: 30
properties:
# TODO(stuartmorgan): Uncomment when removing bringup.
#add_recipes_cq: "true"
target_file: plugin_tools_tests.yaml
scheduler: luci

- name: Linux ci_yaml plugins roller
recipe: infra/ci_yaml
timeout: 30
Expand Down
8 changes: 8 additions & 0 deletions .ci/scripts/build_all_plugins.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
#!/bin/bash
# Copyright 2013 The Flutter Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

cd all_plugins
flutter build windows --debug
flutter build windows --release
7 changes: 7 additions & 0 deletions .ci/scripts/build_examples_uwp.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
#!/bin/bash
# Copyright 2013 The Flutter Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

dart ./script/tool/bin/flutter_plugin_tools.dart build-examples --winuwp \
--packages-for-branch
7 changes: 7 additions & 0 deletions .ci/scripts/create_all_plugins_app.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
#!/bin/bash
# Copyright 2013 The Flutter Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

dart ./script/tool/bin/flutter_plugin_tools.dart all-plugins-app \
--output-dir=. --exclude script/configs/exclude_all_plugins_app.yaml
7 changes: 7 additions & 0 deletions .ci/scripts/native_test_win32.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
#!/bin/bash
# Copyright 2013 The Flutter Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

dart ./script/tool/bin/flutter_plugin_tools.dart native-test --windows \
--no-integration --packages-for-branch
7 changes: 7 additions & 0 deletions .ci/scripts/plugin_tools_tests.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
#!/bin/bash
# Copyright 2013 The Flutter Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

cd script/tool
dart pub run test
7 changes: 7 additions & 0 deletions .ci/targets/build_all_plugins.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
tasks:
- name: prepare tool
script: .ci/scripts/prepare_tool.sh
- name: create all_plugins app
script: .ci/scripts/create_all_plugins_app.sh
- name: build all_plugins
script: .ci/scripts/build_all_plugins.sh
5 changes: 5 additions & 0 deletions .ci/targets/plugin_tools_tests.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
tasks:
- name: prepare tool
script: .ci/scripts/prepare_tool.sh
- name: tool unit tests
script: .ci/scripts/plugin_tools_tests.sh
5 changes: 5 additions & 0 deletions .ci/targets/uwp_build_and_platform_tests.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
tasks:
- name: prepare tool
script: .ci/scripts/prepare_tool.sh
- name: build examples (UWP)
script: .ci/scripts/build_examples_uwp.sh
9 changes: 5 additions & 4 deletions .ci/targets/windows_build_and_platform_tests.yaml
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@

tasks:
- name: "prepare tool"
- name: prepare tool
script: .ci/scripts/prepare_tool.sh
- name: "build examples"
- name: build examples (Win32)
script: .ci/scripts/build_examples_win32.sh
- name: "drive examples"
- name: native unit tests (Win32)
script: .ci/scripts/native_test_win32.sh
- name: drive examples (Win32)
script: .ci/scripts/drive_examples_win32.sh
2 changes: 1 addition & 1 deletion .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ task:
- name: plugin_tools_tests
script:
- cd script/tool
- CIRRUS_BUILD_ID=null pub run test
- dart pub run test
- name: publishable
version_check_script: ./script/tool_runner.sh version-check
publish_check_script: ./script/tool_runner.sh publish-check
Expand Down