From 88937538f11027aebd9db146fe67b4b217a92210 Mon Sep 17 00:00:00 2001 From: Nate Bosch Date: Mon, 31 Mar 2025 17:29:59 +0000 Subject: [PATCH 1/2] Sort reporter options Previous versions of `package:args` alphabetically sorted the `allowedHelp` entries for help output. The latest release removes the sorting and breaks the change-detection tests for the help output. https://github.com/dart-lang/core/pull/852 Manually sort the keys of the reporters map to match the alpha sort of the old output. --- pkgs/test/pubspec.yaml | 2 +- pkgs/test_core/CHANGELOG.md | 2 ++ .../lib/src/runner/configuration/reporters.dart | 12 ++++++------ pkgs/test_core/pubspec.yaml | 2 +- 4 files changed, 10 insertions(+), 8 deletions(-) diff --git a/pkgs/test/pubspec.yaml b/pkgs/test/pubspec.yaml index 7fe0b6d3f..96b7b6be8 100644 --- a/pkgs/test/pubspec.yaml +++ b/pkgs/test/pubspec.yaml @@ -37,7 +37,7 @@ dependencies: # Use an exact version until the test_api and test_core package are stable. test_api: 0.7.4 - test_core: 0.6.8 + test_core: 0.6.9 typed_data: ^1.3.0 web_socket_channel: '>=2.0.0 <4.0.0' diff --git a/pkgs/test_core/CHANGELOG.md b/pkgs/test_core/CHANGELOG.md index 4c305bc61..5e2553d0c 100644 --- a/pkgs/test_core/CHANGELOG.md +++ b/pkgs/test_core/CHANGELOG.md @@ -1,3 +1,5 @@ +## 0.6.9-wip + ## 0.6.8 * Fix hang when running multiple precompiled browser tests. diff --git a/pkgs/test_core/lib/src/runner/configuration/reporters.dart b/pkgs/test_core/lib/src/runner/configuration/reporters.dart index 0f0fe0176..cc1aadf32 100644 --- a/pkgs/test_core/lib/src/runner/configuration/reporters.dart +++ b/pkgs/test_core/lib/src/runner/configuration/reporters.dart @@ -31,17 +31,17 @@ final UnmodifiableMapView allReporters = UnmodifiableMapView(_allReporters); final _allReporters = { - 'expanded': ReporterDetails( - 'A separate line for each update.', - (config, engine, sink) => ExpandedReporter.watch(engine, sink, + 'compact': ReporterDetails( + 'A single line, updated continuously.', + (config, engine, sink) => CompactReporter.watch(engine, sink, color: config.color, printPath: config.testSelections.length > 1 || Directory(config.testSelections.keys.single).existsSync(), printPlatform: config.suiteDefaults.runtimes.length > 1 || config.suiteDefaults.compilerSelections != null)), - 'compact': ReporterDetails( - 'A single line, updated continuously.', - (config, engine, sink) => CompactReporter.watch(engine, sink, + 'expanded': ReporterDetails( + 'A separate line for each update.', + (config, engine, sink) => ExpandedReporter.watch(engine, sink, color: config.color, printPath: config.testSelections.length > 1 || Directory(config.testSelections.keys.single).existsSync(), diff --git a/pkgs/test_core/pubspec.yaml b/pkgs/test_core/pubspec.yaml index ab3f11c41..e5655b974 100644 --- a/pkgs/test_core/pubspec.yaml +++ b/pkgs/test_core/pubspec.yaml @@ -1,5 +1,5 @@ name: test_core -version: 0.6.8 +version: 0.6.9-wip description: A basic library for writing tests and running them on the VM. repository: https://github.com/dart-lang/test/tree/master/pkgs/test_core issue_tracker: https://github.com/dart-lang/test/issues?q=is%3Aissue+is%3Aopen+label%3Apackage%3Atest From d650fc0d2be4e6d5f3b1c580908e390e5128552e Mon Sep 17 00:00:00 2001 From: Nate Bosch Date: Mon, 31 Mar 2025 19:53:01 +0000 Subject: [PATCH 2/2] Use -wip in dependency --- pkgs/test/pubspec.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pkgs/test/pubspec.yaml b/pkgs/test/pubspec.yaml index 96b7b6be8..78dabdeda 100644 --- a/pkgs/test/pubspec.yaml +++ b/pkgs/test/pubspec.yaml @@ -37,7 +37,7 @@ dependencies: # Use an exact version until the test_api and test_core package are stable. test_api: 0.7.4 - test_core: 0.6.9 + test_core: 0.6.9-wip typed_data: ^1.3.0 web_socket_channel: '>=2.0.0 <4.0.0'