Skip to content

Commit

Permalink
Try removing isolate.kill()
Browse files Browse the repository at this point in the history
At one point isolates were killed when the loader was closed after tests
are down. 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

Before that the `isolate.kill` on close goes back to the original
implementation of the loader.
https://codereview.chromium.org//920703006

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 4e0db3f
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 4e0db3f

Please sign in to comment.