fix error-on-fail test

Author: ds300

integration-tests/error-on-fail/error-on-fail.sh

@@ -6,7 +6,7 @@ yarn add $1
 alias patch-package=./node_modules/.bin/patch-package
 
 export NODE_ENV="development"
-export CI=""
+export CI="true"
 
 (>&2 echo "SNAPSHOT: at dev time patch-package fails but returns 0")
 if ! patch-package;

integration-tests/runIntegrationTest.ts

@@ -40,6 +40,10 @@ export function runIntegrationTest({
       {
         cwd: tmpDir.name,
         throwOnError: false,
+        env: {
+          ...process.env,
+          PATCH_PACKAGE_INTEGRATION_TEST: "1",
+        },
         shell: true,
       },
     )

src/index.ts

@@ -75,10 +75,12 @@ if (argv.version || argv.v) {
   } else {
     console.log("Applying patches...")
     const reverse = !!argv["reverse"]
-    // don't want to exit(1) on postinsall locally.
+    // don't want to exit(1) on postinstall locally.
     // see https://github.com/ds300/patch-package/issues/86
     const shouldExitWithError =
-      !!argv["error-on-fail"] || isCI || process.env.NODE_ENV === "test"
+      !!argv["error-on-fail"] ||
+      (isCI && !process.env.PATCH_PACKAGE_INTEGRATION_TEST) ||
+      process.env.NODE_ENV === "test"
 
     const shouldExitWithWarning = !!argv["error-on-warn"]