Try removing isolate.kill()

Originally isolates were killed when the loader was closed as the runner
is shutting down. There was no comment in the review about the specific
reason for `isolate.kill()`. https://codereview.chromium.org//920703006

Later the kill was moved to when the test suite is done. There was no
specific comment about the `isolate.kill()` changes.
https://codereview.chromium.org//1206033004

No internal tests seem to be relying on this behavior, and nothing times
out when it is removed.

Check whether there are any tests on CI that validate this behavior.
1 file changed
tree: 13ccb9ee8772eedf5ea27b5af7e4a8b3b2a0697d
  1. .github/
  2. integration_tests/
  3. pkgs/
  4. tool/
  5. .gitattributes
  6. .gitignore
  7. analysis_options.yaml
  8. CONTRIBUTING.md
  9. mono_repo.yaml
  10. README.md
README.md

Dart CI OpenSSF Scorecard

What's here?

Welcome! package:test is the standard testing library for Dart and Flutter. If you have questions about Dart testing, please see the docs for package:test. package:test_api and package:test_core are implementation details and generally not user-facing.

package:checks is a relatively new library for expressing test expectations. It's a more modern version of package:matcher and features a literate API.

Packages

PackageDescriptionVersion
checksA framework for checking values against expectations and building custom expectations.pub package
testA full featured library for writing and running Dart tests across platforms.pub package
test_apipub package
test_corepub package