Skip to content

Commit c8c1c28

Browse files
authored
Skip ts_bindings_test and run web_generator tests with VM (#373)
It looks like we were silently skipping scrape_mdn_test.dart before because we never ran the tests in this package with the VM.
1 parent de6b3e4 commit c8c1c28

File tree

4 files changed

+48
-3
lines changed

4 files changed

+48
-3
lines changed

.github/workflows/dart.yml

Lines changed: 41 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -340,6 +340,42 @@ jobs:
340340
- job_004
341341
- job_005
342342
job_010:
343+
name: "unit_test; Dart dev; PKG: web_generator; `dart test --test-randomize-ordering-seed=random`"
344+
runs-on: ubuntu-latest
345+
steps:
346+
- name: Cache Pub hosted dependencies
347+
uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684
348+
with:
349+
path: "~/.pub-cache/hosted"
350+
key: "os:ubuntu-latest;pub-cache-hosted;sdk:dev;packages:web_generator;commands:test_2"
351+
restore-keys: |
352+
os:ubuntu-latest;pub-cache-hosted;sdk:dev;packages:web_generator
353+
os:ubuntu-latest;pub-cache-hosted;sdk:dev
354+
os:ubuntu-latest;pub-cache-hosted
355+
os:ubuntu-latest
356+
- name: Setup Dart SDK
357+
uses: dart-lang/setup-dart@e51d8e571e22473a2ddebf0ef8a2123f0ab2c02c
358+
with:
359+
sdk: dev
360+
- id: checkout
361+
name: Checkout repository
362+
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683
363+
- id: web_generator_pub_upgrade
364+
name: web_generator; dart pub upgrade
365+
run: dart pub upgrade
366+
if: "always() && steps.checkout.conclusion == 'success'"
367+
working-directory: web_generator
368+
- name: "web_generator; dart test --test-randomize-ordering-seed=random"
369+
run: "dart test --test-randomize-ordering-seed=random"
370+
if: "always() && steps.web_generator_pub_upgrade.conclusion == 'success'"
371+
working-directory: web_generator
372+
needs:
373+
- job_001
374+
- job_002
375+
- job_003
376+
- job_004
377+
- job_005
378+
job_011:
343379
name: "generate_and_analyze; Dart dev; PKG: web_generator; `dart pub -C ../web get && dart bin/update_idl_bindings.dart && dart analyze --fatal-infos ../web`"
344380
runs-on: ubuntu-latest
345381
steps:
@@ -379,7 +415,8 @@ jobs:
379415
- job_007
380416
- job_008
381417
- job_009
382-
job_011:
418+
- job_010
419+
job_012:
383420
name: "generate_all_and_analyze; Dart dev; PKG: web_generator; `dart pub -C ../web get && dart bin/update_idl_bindings.dart --generate-all && dart analyze --fatal-infos ../web`"
384421
runs-on: ubuntu-latest
385422
steps:
@@ -420,7 +457,8 @@ jobs:
420457
- job_008
421458
- job_009
422459
- job_010
423-
job_012:
460+
- job_011
461+
job_013:
424462
name: "dart_fixes; Dart main; PKG: web; `dart fix --compare-to-golden test_fixes`"
425463
runs-on: ubuntu-latest
426464
steps:
@@ -462,3 +500,4 @@ jobs:
462500
- job_009
463501
- job_010
464502
- job_011
503+
- job_012

tool/ci.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,10 @@ for PKG in ${PKGS}; do
9191
echo 'dart test --test-randomize-ordering-seed=random -p chrome -c dart2wasm'
9292
dart test --test-randomize-ordering-seed=random -p chrome -c dart2wasm || EXIT_CODE=$?
9393
;;
94+
test_2)
95+
echo 'dart test --test-randomize-ordering-seed=random'
96+
dart test --test-randomize-ordering-seed=random || EXIT_CODE=$?
97+
;;
9498
*)
9599
echo -e "\033[31mUnknown TASK '${TASK}' - TERMINATING JOB\033[0m"
96100
exit 64

web_generator/mono_pkg.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ stages:
88
- unit_test:
99
- test: --test-randomize-ordering-seed=random -p chrome
1010
- test: --test-randomize-ordering-seed=random -p chrome -c dart2wasm
11+
- test: --test-randomize-ordering-seed=random
1112
- generate_and_analyze:
1213
- command:
1314
- dart pub -C ../web get

web_generator/test/ts_bindings_test.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44

55
@TestOn('vm')
66
@Tags(['node'])
7+
@Skip('https://github.com/dart-lang/web/issues/372')
78
library;
89

910
import 'dart:io';
@@ -19,7 +20,7 @@ void main() {
1920
group('Web Generator TS Bindings Integration Test', () {
2021
final inputDir = Directory(testGenDTSFiles);
2122

22-
setUp(() async {
23+
setUpAll(() async {
2324
// set up npm
2425
await runProc('npm', ['install'],
2526
workingDirectory: bindingsGeneratorPath, detached: true);

0 commit comments

Comments
 (0)