fixup! add exitWithError to handle exit codes properly

Author: timmywil

.github/workflows/jquery.yml

@@ -41,10 +41,6 @@ jobs:
         working-directory: test/jquery
         run: npm run test:browser -- -v
 
-      - name: Run jQuery tests in Edge in IE mode
-        working-directory: test/jquery
-        run: npm run test:ie -- -v
-
   ie:
     runs-on: windows-latest
     name: Test jQuery with IEDriver

run.js

@@ -206,6 +206,15 @@ export async function run( {
 		{ wait: EXIT_HOOK_WAIT_TIMEOUT }
 	);
 
+	// There is currently a bug in exit-hook where
+	// gracefulExit(1) is not longer respected.
+	// Work araound by setting process.exitCode instead.
+	// See https://github.com/sindresorhus/exit-hook/issues/42
+	function exitWithError() {
+		process.exitCode = 1;
+		gracefulExit();
+	}
+
 	// Start up BrowserStackLocal
 	let tunnel;
 	if ( browserstack ) {
@@ -242,7 +251,7 @@ export async function run( {
 					console.error(
 						chalk.red( `Browser not found: ${ getBrowserString( browser ) }.` )
 					);
-					gracefulExit( 1 );
+					exitWithError();
 				}
 				return latestMatch;
 			} )
@@ -335,7 +344,7 @@ export async function run( {
 	} catch ( error ) {
 		console.error( error );
 		if ( !debug ) {
-			gracefulExit( 1 );
+			exitWithError();
 		}
 	} finally {
 		console.log();
@@ -354,12 +363,13 @@ export async function run( {
 				}
 			}
 			if ( stop ) {
-				return gracefulExit( 1 );
+				exitWithError();
+				return;
 			}
 			console.log( chalk.green( "All tests passed!" ) );
 
 			if ( !debug || browserstack ) {
-				gracefulExit( 0 );
+				gracefulExit();
 			}
 		} else {
 			console.error(
@@ -381,7 +391,7 @@ export async function run( {
 				}
 				console.log( "Press Ctrl+C to exit." );
 			} else {
-				gracefulExit( 1 );
+				exitWithError();
 			}
 		}
 	}