Unverified Commit b865e23a authored by Danny Tuppeny's avatar Danny Tuppeny Committed by GitHub

Don't treat stderr output as failures during DAP test teardown (#94335)

parent 90f93a5a
...@@ -73,13 +73,16 @@ class OutOfProcessDapTestServer extends DapTestServer { ...@@ -73,13 +73,16 @@ class OutOfProcessDapTestServer extends DapTestServer {
Logger? logger, Logger? logger,
) { ) {
// Treat anything written to stderr as the DAP crashing and fail the test // Treat anything written to stderr as the DAP crashing and fail the test
// unless it's "Waiting for another flutter command to release the startup lock". // unless it's "Waiting for another flutter command to release the startup
// lock" or we're tearing down.
_process.stderr _process.stderr
.transform(utf8.decoder) .transform(utf8.decoder)
.where((String error) => !error.contains('Waiting for another flutter command to release the startup lock')) .where((String error) => !error.contains('Waiting for another flutter command to release the startup lock'))
.listen((String error) { .listen((String error) {
logger?.call(error); logger?.call(error);
throw error; if (!_isShuttingDown) {
throw error;
}
}); });
unawaited(_process.exitCode.then((int code) { unawaited(_process.exitCode.then((int code) {
final String message = 'Out-of-process DAP server terminated with code $code'; final String message = 'Out-of-process DAP server terminated with code $code';
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment