Merge pull request ds300#170 from ds300/ignore-scripts

Ignore scripts when making patches

Author: ds300

integration-tests/ignores-scripts-when-making-patch/__snapshots__/ignores-scripts-when-making-patch.test.ts.snap

@@ -0,0 +1,31 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Test ignores-scripts-when-making-patch: a patch file got produced 1`] = `
+"SNAPSHOT: a patch file got produced
+diff --git a/node_modules/naughty-package/postinstall.sh b/node_modules/naughty-package/postinstall.sh
+index 3784520..c4af29c 100755
+--- a/node_modules/naughty-package/postinstall.sh
++++ b/node_modules/naughty-package/postinstall.sh
+@@ -1,4 +1,4 @@
+-# postinstall
++# lol
+ 
+ # if we're running in the main repo is ok
+ if ls ../patch-package ;
+END SNAPSHOT"
+`;
+
+exports[`Test ignores-scripts-when-making-patch: the patch creation output should look normal 1`] = `
+"SNAPSHOT: the patch creation output should look normal
+patch-package 0.0.0
+• Creating temporary folder
+• Installing [email protected] with yarn
+• Diffing your files with clean files
+✔ Created file patches/naughty-package+1.0.0.patch
+END SNAPSHOT"
+`;
+
+exports[`Test ignores-scripts-when-making-patch: there should be no stderr 1`] = `
+"SNAPSHOT: there should be no stderr
+END SNAPSHOT"
+`;

integration-tests/ignores-scripts-when-making-patch/ignores-scripts-when-making-patch.sh

@@ -0,0 +1,17 @@
+# make sure errors stop the script
+set -e
+
+echo "add patch-package"
+yarn add $1
+
+npx replace postinstall lol node_modules/naughty-package/postinstall.sh
+
+echo "SNAPSHOT: the patch creation output should look normal"
+(>&2 echo "SNAPSHOT: there should be no stderr")
+npx patch-package naughty-package
+echo "END SNAPSHOT"
+(>&2 echo "END SNAPSHOT")
+
+echo "SNAPSHOT: a patch file got produced"
+cat patches/naughty-package+1.0.0.patch
+echo "END SNAPSHOT"

integration-tests/ignores-scripts-when-making-patch/ignores-scripts-when-making-patch.test.ts

@@ -0,0 +1,5 @@
+import { runIntegrationTest } from "../runIntegrationTest"
+runIntegrationTest({
+  projectName: "ignores-scripts-when-making-patch",
+  shouldProduceSnapshots: true,
+})

integration-tests/ignores-scripts-when-making-patch/naughty-package/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "naughty-package",
+  "version": "1.0.0",
+  "main": "index.js",
+  "license": "MIT",
+  "scripts": {
+    "postinstall": "./postinstall.sh"
+  }
+}

integration-tests/ignores-scripts-when-making-patch/naughty-package/postinstall.sh

@@ -0,0 +1,9 @@
+# postinstall
+
+# if we're running in the main repo is ok
+if ls ../patch-package ;
+then
+  exit 0
+fi
+# if we are installed alone then fail
+exit 1

integration-tests/ignores-scripts-when-making-patch/package.json

@@ -0,0 +1,12 @@
+{
+  "name": "ignores-scripts-when-making-patch",
+  "version": "1.0.0",
+  "description": "integration test for patch-package",
+  "main": "index.js",
+  "author": "",
+  "license": "ISC",
+  "dependencies": {
+    "naughty-package": "file:./naughty-package",
+    "replace": "^1.1.1"
+  }
+}