Skip to content

Commit c08259d

Browse files
committed
Try letting the IsolateChannel close the RecievePort
The `IsolateChannel` should close the receive port when we call `outerChannel.sink.close` which is also added to the `cleanupCallbacks`. This was originally added in #1280 to try to fix cases where the test runner could hang after running some tests specifically on the test runner CI when running the test as a subprocess. That PR went through changes closing a number of ports speculatively, and this was one likely not necessary.
1 parent 7724aab commit c08259d

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

pkgs/test_core/lib/src/runner/vm/platform.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,9 +80,7 @@ class VMPlatform extends PlatformPlugin {
8080
rethrow;
8181
}
8282
outerChannel = MultiChannel(IsolateChannel.connectReceive(receivePort));
83-
cleanupCallbacks
84-
..add(receivePort.close)
85-
..add(isolate.kill);
83+
cleanupCallbacks.add(isolate.kill);
8684
}
8785
cleanupCallbacks.add(outerChannel.sink.close);
8886

0 commit comments

Comments
 (0)