Check integration test exit code before output

ramosbugs · ramosbugs · commit 82c88926e9cc · 2023-07-11T22:51:55.000-07:00
Certain Windows test runs appear to end abruptly
with truncated output. Hopefully this will help
identify the underlying cause.
diff --git a/packages/test-common/src/spawn.ts b/packages/test-common/src/spawn.ts
@@ -138,10 +138,10 @@ export const spawnTestWithTimeout = async (
       throw asyncTestError.error;
     }
 
-    await verifyOutput(stdoutLines, stderrLines);
-
     expect(signal).toBe(null);
     expect(code).toBe(expectedExitCode);
+
+    await verifyOutput(stdoutLines, stderrLines);
   } catch (e: unknown) {
     // Jest doesn't have a built-in setting for printing console logs only for failed tests, so we
     // just defer the output until this catch block and attach it to the error. See
