Move test dependencies, stop ignoring node_modules

Previously, `node_modules` was in `.gitignore` because there was a lot
of dependencies for the tests that don't need to be committed. However,
npm is also used to manage the primer dependency, which _does_ need to
be committed so GitHub Pages can build the website. At some point,
someone (whose name rhymes with "Landon Creepers") thought it'd be fine
to just type `git add -f node_modules/primer*` to commit the primer
dependencies and leave everything else ignored. This works…until you
want to update primer. Any update requires that the author knows about
this awful `.gitignore` setup, and uses the dreadful `-f` flag.

This commit:

- Moves the dependencies needed for tests into `test/package.json`
- Adds `test/node_modules` to `.gitignore`
- Removes `node_modules` from `.gitignore`

Author: bkeepers

.gitignore

@@ -4,7 +4,7 @@ _site/
 .jekyll-metadata
 .sass-cache/
 bin
-node_modules
+test/node_modules
 /vendor
 Gemfile.lock
 css/main.scss

package.json

@@ -2,36 +2,8 @@
   "name": "open-source-guide",
   "private": true,
   "scripts": {
-    "test": "script/test"
-  },
-  "devDependencies": {
-    "async": "^2.0.0",
-    "download": "^5.0.3",
-    "ebrew": "^0.1.1",
-    "fastmatter": "^1.1.1",
-    "glob": "^7.0.5",
-    "ignore": "^3.1.3",
-    "js-yaml": "^3.6.1",
-    "kindlegen": "^1.1.0",
-    "mkdirp": "^0.5.1",
-    "remark-lint": "^5.0.0",
-    "remark-parse": "^2.0.0",
-    "remark-retext": "^2.0.0",
-    "remark-stringify": "^2.0.0",
-    "retext-english": "^2.0.0",
-    "retext-equality": "^2.3.2",
-    "retext-quotes": "^1.0.0",
-    "retext-readability": "^3.0.0",
-    "retext-repeated-words": "^1.0.0",
-    "retext-sentence-spacing": "^1.0.0",
-    "retext-simplify": "^3.0.0",
-    "retext-words": "bkeepers/retext-words",
-    "slug": "^0.9.1",
-    "to-vfile": "^2.1.0",
-    "unified": "^5.0.0",
-    "unist-util-visit": "^1.1.1",
-    "vfile-reporter": "^3.0.0",
-    "vfile-statistics": "^1.0.0"
+    "test": "script/test",
+    "postinstall": "cd test && npm install"
   },
   "dependencies": {
     "primer-core": "6.0.0",

script/test

@@ -5,4 +5,4 @@ set -e
 script/build --config _config.yml,test/_config.yml
 bundle exec rake
 script/html-proofer
-script/test-prose
+test/prose

test/package.json

@@ -0,0 +1,32 @@
+{
+  "private": true,
+  "devDependencies": {
+    "async": "^2.0.0",
+    "download": "^5.0.3",
+    "ebrew": "^0.1.1",
+    "fastmatter": "^1.1.1",
+    "glob": "^7.0.5",
+    "ignore": "^3.1.3",
+    "js-yaml": "^3.6.1",
+    "kindlegen": "^1.1.0",
+    "mkdirp": "^0.5.1",
+    "remark-lint": "^5.0.0",
+    "remark-parse": "^2.0.0",
+    "remark-retext": "^2.0.0",
+    "remark-stringify": "^2.0.0",
+    "retext-english": "^2.0.0",
+    "retext-equality": "^2.3.2",
+    "retext-quotes": "^1.0.0",
+    "retext-readability": "^3.0.0",
+    "retext-repeated-words": "^1.0.0",
+    "retext-sentence-spacing": "^1.0.0",
+    "retext-simplify": "^3.0.0",
+    "retext-words": "bkeepers/retext-words",
+    "slug": "^0.9.1",
+    "to-vfile": "^2.1.0",
+    "unified": "^5.0.0",
+    "unist-util-visit": "^1.1.1",
+    "vfile-reporter": "^3.0.0",
+    "vfile-statistics": "^1.0.0"
+  }
+}