Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
56 commits
Select commit Hold shift + click to select a range
c13cd89
Wrote felt_config file and parsing.
eyebrowsoffire Feb 10, 2023
b60b1b4
Some suite filtering and copy artifacts step refactor.
eyebrowsoffire Feb 14, 2023
8760d77
CompileBundleStep implemented.
eyebrowsoffire Feb 16, 2023
752547d
Suites running, but assets alongside tests are not being served.
eyebrowsoffire Feb 17, 2023
1bdd442
Reduce verbosity and fix serving assets from test directory.
eyebrowsoffire Feb 17, 2023
b6e520e
All filters except file filters are implemented.
eyebrowsoffire Feb 17, 2023
e26e6a3
File filter implemented.
eyebrowsoffire Feb 17, 2023
c2d464d
Move around tests themselves.
eyebrowsoffire Feb 17, 2023
3c9c37d
Test against canvaskit_chromium.
eyebrowsoffire Feb 17, 2023
a7906e8
Added and updated documentation.
eyebrowsoffire Feb 17, 2023
b0cfad9
Started some CI stuff.
eyebrowsoffire Mar 1, 2023
78859af
Add bringup:true to the new web engine step.
eyebrowsoffire Mar 1, 2023
2dd558d
Change felt parameters.
eyebrowsoffire Mar 1, 2023
35163cf
Remove old web engine builders.
eyebrowsoffire Mar 1, 2023
d721f10
Always hard code canvaskit variants for chrome and edge.
eyebrowsoffire Mar 2, 2023
ad81ea6
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 2, 2023
c48e279
improve some error messages
mdebbar Mar 2, 2023
83f67b8
support filtering by canvaskit variant
mdebbar Mar 2, 2023
c5f60d1
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 7, 2023
1687708
Change linux_web_engine to new felt commands.
eyebrowsoffire Mar 7, 2023
d6ec4e7
Skip the legacy build path for license checks.
eyebrowsoffire Mar 8, 2023
77e0119
Update lib/web_ui/dev/steps/compile_bundle_step.dart
eyebrowsoffire Mar 8, 2023
7fcc94b
Revert weird change to README.md
eyebrowsoffire Mar 8, 2023
8294f13
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 8, 2023
581023a
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 10, 2023
28808e7
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 10, 2023
55195f0
Separate web_tests build out.
eyebrowsoffire Mar 10, 2023
dbc98f9
Copy artifacts in a generator step.
eyebrowsoffire Mar 10, 2023
0c6bf80
Do copy artifacts as part of the test run.
eyebrowsoffire Mar 10, 2023
dd3c7b9
Allow testing against wasm_profile artifacts.
eyebrowsoffire Mar 11, 2023
38dd751
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 13, 2023
f390a96
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 16, 2023
db9c9e0
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 23, 2023
1071c87
Remove duplicated build step.
eyebrowsoffire Mar 23, 2023
5f5ff75
Add new fields for the v2 tests.
eyebrowsoffire Mar 23, 2023
0cc002e
Put the test invocation into `tasks`
eyebrowsoffire Mar 23, 2023
d5a9e7a
Name the test.
eyebrowsoffire Mar 23, 2023
d84c677
Try sharding compilation and tests.
eyebrowsoffire Mar 23, 2023
bb45d11
Fix bundle filtering.
eyebrowsoffire Mar 23, 2023
ee89022
Try making copy artifacts a subbuild.
eyebrowsoffire Mar 23, 2023
164483f
The main build should be `web_tests/artifacts`
eyebrowsoffire Mar 23, 2023
c25b4da
test_bundles, not test_bundle
eyebrowsoffire Mar 24, 2023
97f9459
Generate all builder json.
eyebrowsoffire Mar 24, 2023
a8bc5c4
Specify an empty tests list.
eyebrowsoffire Mar 24, 2023
442034d
Generate the builder json again.
eyebrowsoffire Mar 24, 2023
eb1e3ec
script, not scripts.
eyebrowsoffire Mar 24, 2023
d25ca9e
Get rid of extra array layer.
eyebrowsoffire Mar 24, 2023
aed3f1b
No need for empty tests array in build steps
eyebrowsoffire Mar 24, 2023
f47bfd3
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 24, 2023
2271295
It's `script` for tests, but `scripts` for generators.
eyebrowsoffire Mar 24, 2023
d4a7f5a
Mac, not MacOS
eyebrowsoffire Mar 24, 2023
6d5fa74
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 24, 2023
4ed8d43
Yegor's comments.
eyebrowsoffire Mar 27, 2023
d66c188
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 27, 2023
ba83669
Add back screenshot logging under verbose flag.
eyebrowsoffire Mar 27, 2023
9b61f3a
Merge branch 'main' into web_test_reorganization
eyebrowsoffire Mar 27, 2023
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
Prev Previous commit
Next Next commit
Merge branch 'main' into web_test_reorganization
  • Loading branch information
eyebrowsoffire committed Mar 16, 2023
commit f390a9656f4348370f945ced8c4893801f7eec5d
8 changes: 5 additions & 3 deletions .ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ platform_properties:
ios_debug: "false"
ios_profile: "false"
ios_release: "false"
# CIPD flutter_internal/java/openjdk/$platform
# CIPD flutter/java/openjdk/$platform
dependencies: >-
[
{"dependency": "open_jdk", "version": "version:11"},
Expand All @@ -46,7 +46,7 @@ platform_properties:
ios_debug: "false"
ios_profile: "false"
ios_release: "false"
# CIPD flutter_internal/java/openjdk/$platform
# CIPD flutter/java/openjdk/$platform
dependencies: >-
[
{"dependency": "open_jdk", "version": "version:11"}
Expand All @@ -68,7 +68,7 @@ platform_properties:
ios_debug: "false"
ios_profile: "false"
ios_release: "false"
# CIPD flutter_internal/java/openjdk/$platform
# CIPD flutter/java/openjdk/$platform
dependencies: >-
[
{"dependency": "open_jdk", "version": "version:11"}
Expand Down Expand Up @@ -156,6 +156,7 @@ targets:
emulator_arch: "x64"
enable_cso: "true"
timeout: 60
bringup: true

- name: Linux Fuchsia arm64 FEMU
recipe: engine/femu_test
Expand All @@ -168,6 +169,7 @@ targets:
emulator_arch: "arm64"
enable_cso: "true"
timeout: 60
bringup: true

- name: Linux Framework Smoke Tests
recipe: engine/framework_smoke
Expand Down
10 changes: 5 additions & 5 deletions DEPS
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ vars = {
'llvm_git': 'https://llvm.googlesource.com',
# OCMock is for testing only so there is no google clone
'ocmock_git': 'https://github.com/erikdoe/ocmock.git',
'skia_revision': '1c36c5c38ef4742a4e359484ab15a670c06902d9',
'skia_revision': '9b7c0ae5007a1665fefc9a316bfcdfe40763ed89',

# WARNING: DO NOT EDIT canvaskit_cipd_instance MANUALLY
# See `lib/web_ui/README.md` for how to roll CanvasKit to a new version.
Expand Down Expand Up @@ -48,7 +48,7 @@ vars = {
# Dart is: https://github.com/dart-lang/sdk/blob/main/DEPS
# You can use //tools/dart/create_updated_flutter_deps.py to produce
# updated revision list of existing dependencies.
'dart_revision': 'c766fffb626ecac3be237703d4c68178f308e5f3',
'dart_revision': '2cd913e06b8c41c9b9afd139bedff8fc6d0bb232',

# WARNING: DO NOT EDIT MANUALLY
# The lines between blank lines above and below are generated by a script. See create_updated_flutter_deps.py
Expand Down Expand Up @@ -455,7 +455,7 @@ deps = {
Var('dart_git') + '/test_reflective_loader.git@c4c2d5c3f94a96f3fc79e9e28944fba391bc544c',

'src/third_party/dart/third_party/pkg/tools':
Var('dart_git') + '/tools.git@bed358ea8ca10551c710282be96cf6e95620fb24',
Var('dart_git') + '/tools.git@e5353aa3ae4bcaa75cd0ac48a6e47035c57df942',

'src/third_party/dart/third_party/pkg/typed_data':
Var('dart_git') + '/typed_data.git@f858046fb420cf644e7d8cb86b7893f2830d8a6c',
Expand Down Expand Up @@ -832,7 +832,7 @@ deps = {
'packages': [
{
'package': 'fuchsia/sdk/core/mac-amd64',
'version': 'lfirU5p2GigxFWZVA4s5fWx9-lhElkibhnNu2A3LOAgC'
'version': '3NfZ5JD61eNz4jJqBXd_5fmribyv-hlizMbWAs_TvzkC'
}
],
'condition': 'host_os == "mac" and not download_fuchsia_sdk',
Expand All @@ -842,7 +842,7 @@ deps = {
'packages': [
{
'package': 'fuchsia/sdk/core/linux-amd64',
'version': 'VFZ_BtgLOf7FaEr6XYKW1wXqPuy7nQFMxd3P6-d9XwgC'
'version': 'bXql_WHqGrwP5ZIWm0zEgGn21zyirWdpx439qYw4-GIC'
}
],
'condition': 'host_os == "linux" and not download_fuchsia_sdk',
Expand Down
1 change: 1 addition & 0 deletions build/archives/BUILD.gn
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,7 @@ generated_file("dart_sdk_entitlement_config") {
"dart-sdk/bin/dart",
"dart-sdk/bin/dartaotruntime",
"dart-sdk/bin/utils/gen_snapshot",
"dart-sdk/bin/utils/wasm-opt",
]

deps = []
Expand Down
81 changes: 27 additions & 54 deletions ci/bin/format.dart
Original file line number Diff line number Diff line change
Expand Up @@ -304,19 +304,14 @@ abstract class FormatChecker {
/// Checks and formats C++/ObjC/Shader files using clang-format.
class ClangFormatChecker extends FormatChecker {
ClangFormatChecker({
ProcessManager processManager = const LocalProcessManager(),
required String baseGitRef,
required Directory repoDir,
super.processManager,
required super.baseGitRef,
required super.repoDir,
required Directory srcDir,
bool allFiles = false,
MessageCallback? messageCallback,
super.allFiles,
super.messageCallback,
}) : super(
processManager: processManager,
baseGitRef: baseGitRef,
repoDir: repoDir,
srcDir: srcDir,
allFiles: allFiles,
messageCallback: messageCallback,
) {
/*late*/ String clangOs;
if (Platform.isLinux) {
Expand Down Expand Up @@ -449,19 +444,14 @@ class ClangFormatChecker extends FormatChecker {
/// Checks the format of Java files uing the Google Java format checker.
class JavaFormatChecker extends FormatChecker {
JavaFormatChecker({
ProcessManager processManager = const LocalProcessManager(),
required String baseGitRef,
required Directory repoDir,
super.processManager,
required super.baseGitRef,
required super.repoDir,
required Directory srcDir,
bool allFiles = false,
MessageCallback? messageCallback,
super.allFiles,
super.messageCallback,
}) : super(
processManager: processManager,
baseGitRef: baseGitRef,
repoDir: repoDir,
srcDir: srcDir,
allFiles: allFiles,
messageCallback: messageCallback,
) {
googleJavaFormatJar = File(
path.absolute(
Expand Down Expand Up @@ -596,19 +586,14 @@ class JavaFormatChecker extends FormatChecker {
/// Checks the format of any BUILD.gn files using the "gn format" command.
class GnFormatChecker extends FormatChecker {
GnFormatChecker({
ProcessManager processManager = const LocalProcessManager(),
required String baseGitRef,
super.processManager,
required super.baseGitRef,
required Directory repoDir,
required Directory srcDir,
bool allFiles = false,
MessageCallback? messageCallback,
required super.srcDir,
super.allFiles,
super.messageCallback,
}) : super(
processManager: processManager,
baseGitRef: baseGitRef,
repoDir: repoDir,
srcDir: srcDir,
allFiles: allFiles,
messageCallback: messageCallback,
) {
gnBinary = File(
path.join(
Expand Down Expand Up @@ -687,19 +672,14 @@ class GnFormatChecker extends FormatChecker {
/// Checks the format of any .py files using the "yapf" command.
class PythonFormatChecker extends FormatChecker {
PythonFormatChecker({
ProcessManager processManager = const LocalProcessManager(),
required String baseGitRef,
super.processManager,
required super.baseGitRef,
required Directory repoDir,
required Directory srcDir,
bool allFiles = false,
MessageCallback? messageCallback,
required super.srcDir,
super.allFiles,
super.messageCallback,
}) : super(
processManager: processManager,
baseGitRef: baseGitRef,
repoDir: repoDir,
srcDir: srcDir,
allFiles: allFiles,
messageCallback: messageCallback,
) {
yapfBin = File(path.join(
repoDir.absolute.path,
Expand Down Expand Up @@ -787,20 +767,13 @@ class _GrepResult {
/// Checks for trailing whitspace in Dart files.
class WhitespaceFormatChecker extends FormatChecker {
WhitespaceFormatChecker({
ProcessManager processManager = const LocalProcessManager(),
required String baseGitRef,
required Directory repoDir,
required Directory srcDir,
bool allFiles = false,
MessageCallback? messageCallback,
}) : super(
processManager: processManager,
baseGitRef: baseGitRef,
repoDir: repoDir,
srcDir: srcDir,
allFiles: allFiles,
messageCallback: messageCallback,
);
super.processManager,
required super.baseGitRef,
required super.repoDir,
required super.srcDir,
super.allFiles,
super.messageCallback,
});

@override
Future<bool> checkFormatting() async {
Expand Down
6 changes: 4 additions & 2 deletions ci/builders/linux_web_engine.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,15 +14,17 @@
],
"drone_dimensions": [
"device_type=none",
"os=Linux"
"os=Linux",
"cores=32"
],
"gclient_variables": {
"download_android_deps": false,
"download_emsdk": true
},
"gn": [
"--web",
"--runtime-mode=release"
"--runtime-mode=release",
"--no-goma"
],
"name": "wasm_release",
"ninja": {
Expand Down
30 changes: 24 additions & 6 deletions ci/builders/windows_android_aot_engine.json
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,10 @@
"name": "android_profile"
}
],
"drone_dimensions": [],
"drone_dimensions": [
"device_type=none",
"os=Windows-10"
],
"gn": [
"--runtime-mode",
"profile",
Expand All @@ -36,7 +39,10 @@
"name": "android_profile_arm64"
}
],
"drone_dimensions": [],
"drone_dimensions": [
"device_type=none",
"os=Windows-10"
],
"gn": [
"--runtime-mode",
"profile",
Expand All @@ -62,7 +68,10 @@
"name": "android_profile_x64"
}
],
"drone_dimensions": [],
"drone_dimensions": [
"device_type=none",
"os=Windows-10"
],
"gn": [
"--runtime-mode",
"profile",
Expand All @@ -88,7 +97,10 @@
"name": "android_release"
}
],
"drone_dimensions": [],
"drone_dimensions": [
"device_type=none",
"os=Windows-10"
],
"gn": [
"--runtime-mode",
"release",
Expand All @@ -113,7 +125,10 @@
"name": "android_release_arm64"
}
],
"drone_dimensions": [],
"drone_dimensions": [
"device_type=none",
"os=Windows-10"
],
"gn": [
"--runtime-mode",
"release",
Expand All @@ -139,7 +154,10 @@
"name": "android_release_x64"
}
],
"drone_dimensions": [],
"drone_dimensions": [
"device_type=none",
"os=Windows-10"
],
"gn": [
"--runtime-mode",
"release",
Expand Down
2 changes: 2 additions & 0 deletions ci/licenses_golden/excluded_files
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@
../../../flutter/impeller/tessellator/tessellator_unittests.cc
../../../flutter/impeller/tools/build_metal_library.py
../../../flutter/impeller/tools/check_licenses.py
../../../flutter/impeller/tools/malioc_cores.py
../../../flutter/impeller/tools/malioc_diff.py
../../../flutter/impeller/tools/xxd.py
../../../flutter/impeller/typographer/typographer_unittests.cc
Expand Down Expand Up @@ -2747,6 +2748,7 @@
../../../third_party/skia/src/encode/BUILD.bazel
../../../third_party/skia/src/fonts/BUILD.bazel
../../../third_party/skia/src/gpu/BUILD.bazel
../../../third_party/skia/src/gpu/dawn/BUILD.bazel
../../../third_party/skia/src/gpu/ganesh/BUILD.bazel
../../../third_party/skia/src/gpu/ganesh/d3d/BUILD.bazel
../../../third_party/skia/src/gpu/ganesh/dawn/BUILD.bazel
Expand Down
8 changes: 8 additions & 0 deletions ci/licenses_golden/licenses_flutter
Original file line number Diff line number Diff line change
Expand Up @@ -1123,6 +1123,8 @@ ORIGIN: ../../../flutter/impeller/entity/contents/color_source_contents.cc + ../
ORIGIN: ../../../flutter/impeller/entity/contents/color_source_contents.h + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/contents/color_source_text_contents.cc + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/contents/color_source_text_contents.h + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/contents/conical_gradient_contents.cc + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/contents/conical_gradient_contents.h + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/contents/content_context.cc + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/contents/content_context.h + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/contents/contents.cc + ../../../flutter/LICENSE
Expand Down Expand Up @@ -1237,6 +1239,8 @@ ORIGIN: ../../../flutter/impeller/entity/shaders/border_mask_blur.frag + ../../.
ORIGIN: ../../../flutter/impeller/entity/shaders/border_mask_blur.vert + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/shaders/color_matrix_color_filter.frag + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/shaders/color_matrix_color_filter.vert + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/shaders/conical_gradient_fill.frag + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/shaders/conical_gradient_ssbo_fill.frag + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/shaders/gaussian_blur.frag + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/shaders/gaussian_blur.glsl + ../../../flutter/LICENSE
ORIGIN: ../../../flutter/impeller/entity/shaders/gaussian_blur.vert + ../../../flutter/LICENSE
Expand Down Expand Up @@ -3652,6 +3656,8 @@ FILE: ../../../flutter/impeller/entity/contents/color_source_contents.cc
FILE: ../../../flutter/impeller/entity/contents/color_source_contents.h
FILE: ../../../flutter/impeller/entity/contents/color_source_text_contents.cc
FILE: ../../../flutter/impeller/entity/contents/color_source_text_contents.h
FILE: ../../../flutter/impeller/entity/contents/conical_gradient_contents.cc
FILE: ../../../flutter/impeller/entity/contents/conical_gradient_contents.h
FILE: ../../../flutter/impeller/entity/contents/content_context.cc
FILE: ../../../flutter/impeller/entity/contents/content_context.h
FILE: ../../../flutter/impeller/entity/contents/contents.cc
Expand Down Expand Up @@ -3766,6 +3772,8 @@ FILE: ../../../flutter/impeller/entity/shaders/border_mask_blur.frag
FILE: ../../../flutter/impeller/entity/shaders/border_mask_blur.vert
FILE: ../../../flutter/impeller/entity/shaders/color_matrix_color_filter.frag
FILE: ../../../flutter/impeller/entity/shaders/color_matrix_color_filter.vert
FILE: ../../../flutter/impeller/entity/shaders/conical_gradient_fill.frag
FILE: ../../../flutter/impeller/entity/shaders/conical_gradient_ssbo_fill.frag
FILE: ../../../flutter/impeller/entity/shaders/gaussian_blur.frag
FILE: ../../../flutter/impeller/entity/shaders/gaussian_blur.glsl
FILE: ../../../flutter/impeller/entity/shaders/gaussian_blur.vert
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.