ci: disable system js tests (angular#49332)

alan-agius4 · atscott · commit 9c3f8d52d899 · 2023-03-22T14:00:20.000-07:00
These are currently broken due to the APF v16 changes which dropped support for ES2015. PR Close angular#49332
diff --git a/aio/content/examples/upgrade-module/BUILD.bazel b/aio/content/examples/upgrade-module/BUILD.bazel
@@ -8,5 +8,9 @@ docs_example(
     flaky = True,  # TODO: figure out why this is flaky or times out
     # This example downloads and inlines resources
     test_exec_properties = ENABLE_NETWORK,
-    test_tags = ["requires-network"],
+    test_tags = [
+        "requires-network",
+        # This is currently broken due to usage of ESM2022 which does not work out-of-the-box with system.js.
+        "broken",
+    ],
 )
diff --git a/aio/content/examples/upgrade-phonecat-2-hybrid/BUILD.bazel b/aio/content/examples/upgrade-phonecat-2-hybrid/BUILD.bazel
@@ -8,5 +8,9 @@ docs_example(
     flaky = True,  # TODO: figure out why this is flaky or times out
     # This example downloads and inlines resources
     test_exec_properties = ENABLE_NETWORK,
-    test_tags = ["requires-network"],
+    test_tags = [
+        "requires-network",
+        # This is currently broken due to usage of ESM2022 which does not work out-of-the-box with system.js.
+        "broken",
+    ],
 )
diff --git a/aio/content/examples/upgrade-phonecat-3-final/BUILD.bazel b/aio/content/examples/upgrade-phonecat-3-final/BUILD.bazel
@@ -5,4 +5,5 @@ package(default_visibility = ["//visibility:public"])
 docs_example(
     name = "upgrade-phonecat-3-final",
     flaky = True,  # TODO: figure out why this is flaky or times out
+    test_tags = ["broken"],  # This is currently broken due to usage of ESM2022 which does not work out-of-the-box with system.js.
 )
diff --git a/aio/package.json b/aio/package.json
@@ -12,10 +12,10 @@
     "build-prod": "yarn build --config=release",
     "build-local": "yarn build --config=aio_local_deps",
     "lint": "yarn check-env && yarn docs-lint && ng lint && yarn example-lint && yarn tools-lint && yarn security-lint",
-    "test": "bazel test //aio:test --test_output=streamed",
-    "test-and-watch": "ibazel test //aio:test-and-watch --test_output=streamed",
+    "test": "bazel test //aio:test --test_output=streamed --test_tag_filters=-broken",
+    "test-and-watch": "ibazel test //aio:test-and-watch --test_output=streamed --test_tag_filters=-broken",
     "test-local": "yarn test --config=aio_local_deps",
-    "test:ci": "bazel test //aio/...",
+    "test:ci": "bazel test //aio/... --test_tag_filters=-broken",
     "test-local:ci": "yarn test:ci --config=aio_local_deps",
     "e2e": "bazel test //aio:e2e",
     "e2e-local": "yarn e2e --config=aio_local_deps",

Original file line number	Diff line number	Diff line change
`@@ -5,4 +5,5 @@ package(default_visibility = ["//visibility:public"])`
`5`	`5`	`docs_example(`
`6`	`6`	`name = "upgrade-phonecat-3-final",`
`7`	`7`	`flaky = True, # TODO: figure out why this is flaky or times out`
	`8`	`+ test_tags = ["broken"], # This is currently broken due to usage of ESM2022 which does not work out-of-the-box with system.js.`
`8`	`9`	`)`