Merge pull request #170 from jekyll/ci-build

Build the front end in CI

Author: benbalter

package.json

@@ -10,7 +10,8 @@
     "open:src": "babel-node tools/srcServer.js",
     "lint": "esw webpack.config.* src tools",
     "lint:watch": "npm run lint -- --watch",
-    "clean-dist": "npm run remove-dist && mkdir ./lib/jekyll-admin/public",
+    "clean-dist": "npm run remove-dist  && npm run add-dist",
+    "add-dist": "mkdirp ./lib/jekyll-admin/public",
     "remove-dist": "rimraf ./lib/jekyll-admin/public",
     "build:html": "babel-node tools/buildHtml.js",
     "prebuild": "npm run clean-dist && npm run build:html && npm run lint && npm test",
@@ -71,6 +72,7 @@
     "file-loader": "0.8.5",
     "isparta": "4.0.0",
     "jsdom": "^9.2.1",
+    "mkdirp": "^0.5.1",
     "mocha": "2.4.5",
     "nock": "^8.0.0",
     "node-sass": "3.7.0",

script/cibuild-node

@@ -3,4 +3,13 @@
 set -e
 
 echo "Running Node tests..."
-npm test
+
+# We want to run `script/build` on CI to ensure things actually build, but
+# we don't want to have to run `script/build` locally every time we run tests
+#
+# Note: `script/build` runs `npm test` as  part of the build process
+if [ -n "$CI" ]; then
+  script/build
+else
+  npm test
+fi