Skip to content

Commit

Permalink
Try removing isolate.kill()
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
natebosch committed Mar 22, 2024
1 parent 6a4e75a commit ef7c98d
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion pkgs/test_core/lib/src/runner/vm/platform.dart
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ class VMPlatform extends PlatformPlugin {
rethrow;
}
outerChannel = MultiChannel(IsolateChannel.connectReceive(receivePort));
cleanupCallbacks.add(isolate.kill);
}
cleanupCallbacks.add(outerChannel.sink.close);

Expand Down

0 comments on commit ef7c98d

Please sign in to comment.