Files
sdk/pkg/test_runner
Ömer Ağacan 804c6bbd84 [test_runner] Remove duplicate "allowed values" help, use package:args's allowed value help feature
`tools/test.py --help` output before: (only relevant parts)

```
-c, --compiler                    How the Dart code should be compiled or statically processed.
                                  dart2js:              Compile to JavaScript using dart2js.
                                  dart2analyzer:        Perform static analysis on Dart code using the analyzer.
                                  compare_analyzer_cfe: Compare analyzer and common front end representations.
                                  ddc:                  Compile to JavaScript using dartdevc.
                                  app_jitk:             Compile the Dart code into Kernel and then into an app
                                                        snapshot.
                                  dartk:                Compile the Dart code into Kernel before running test.
                                  dartkp:               Compile the Dart code into Kernel and then Kernel into
                                                        AOT snapshot before running the test.
                                  spec_parser:          Parse Dart code using the specification parser.
                                  fasta:                Compile using CFE for errors, but do not run.
                                  [dart2js, dart2analyzer, dart2wasm, ddc, app_jitk, dartk, dartkp, spec_parser, fasta, dart2bytecode]
-r, --runtime                     Where the tests should be run.
                                  vm:               Run Dart code on the standalone Dart VM.
                                  dart_precompiled: Run a precompiled snapshot on the VM without a JIT.
                                  d8:               Run JavaScript from the command line using Chrome's v8.
                                  jsc:              Run JavaScript from the command line using Safari/WebKit's jsc.
                                  jsshell:          Run JavaScript from the command line using Firefox's js-shell.

                                  firefox:
                                  chrome:
                                  safari:
                                  chromeOnAndroid:  Run JavaScript in the specified browser.

                                  none:             No runtime, compile only.
                                  [vm, flutter, dart_precompiled, d8, jsc, jsshell, firefox, chrome, safari, edge, chromeOnAndroid, none]
...
-p, --progress                    Progress indication mode.

                                  Allowed values are:
                                  compact, color, line, verbose, silent, status
                                  [compact (default), color, line, verbose, silent, status]
...
    --nnbd                        Which set of non-nullable type features to use.

                                  Allowed values are: legacy, weak, strong
                                  [legacy, weak, strong (default)]
```

After:

```
-c, --compiler                      How the Dart code should be compiled or statically processed.

          [dart2js]                 Compile to JavaScript using dart2js.
          [dart2analyzer]           Perform static analysis on Dart code using the analyzer.
          [compare_analyzer_cfe]    Compare analyzer and common front end representations.
          [ddc]                     Compile to JavaScript using dartdevc.
          [app_jitk]                Compile the Dart code into Kernel and then into an app snapshot.
          [dartk]                   Compile the Dart code into Kernel before running test.
          [dartkp]                  Compile the Dart code into Kernel and then Kernel into AOT snapshot before running the test.
          [spec_parser]             Parse Dart code using the specification parser.
          [fasta]                   Compile using CFE for errors, but do not run.

-r, --runtime                       Where the tests should be run.

          [vm]                      Run Dart code on the standalone Dart VM.
          [dart_precompiled]        Run a precompiled snapshot on the VM without a JIT.
          [d8]                      Run JavaScript from the command line using Chrome's v8.
          [jsc]                     Run JavaScript from the command line using Safari/WebKit's jsc.
          [jsshell]                 Run JavaScript from the command line using Firefox's js-shell.
          [firefox]                 Run JavaScript in Firefox.
          [chrome]                  Run JavaScript in Chrome.
          [safari]                  Run JavaScript in Safari.
          [chromeOnAndroid]         Run JavaScript in Chrome on Android.
          [none]                    No runtime, compile only.
...
-p, --progress                      Progress indication mode.
                                    [compact (default), color, line, verbose, silent, status]
    --nnbd                          Which set of non-nullable type features to use.
                                    [legacy, weak, strong (default)]
```

Changes:

- Duplicate "allowed values are ..." phrases removed.
- With `package:args`'s `allowedHelp` messages, allowed values documentations are now consistent with the rest of the ecosystem.

Change-Id: I25d5e310e7dd38e0e94622815d9773ec84ab8fb3
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/431860
Reviewed-by: Bob Nystrom <rnystrom@google.com>
Commit-Queue: Ömer Ağacan <omersa@google.com>
2025-05-30 02:09:42 -07:00
..