Make integration tests play nice with Jest

Rename /__tests__ --> /integration-tests

integration-tests/__tests__/index.js

@@ -0,0 +1,5 @@
+// This weird trick was used to make sure jest will run
+// build result test from cli test since cli will clean up dist folder
+
+require("../build-result.js")
+require("../cli.js")

__tests__/index.js → integration-tests/build-result.js

@@ -1,11 +1,11 @@
-import test from "ava"
+import test from "jest-ava-api"
 
 import { readFileSync } from "fs"
 import { join } from "path"
 
 import globby from "globby"
 
-const testFolder = "../test-phenomic-theme-base/dist"
+const testFolder = __dirname + "/../test-phenomic-theme-base/dist"
 const files = globby.sync("**/*", {
   cwd: testFolder,
   nodir: true,

integration-tests/cli.js

@@ -0,0 +1,83 @@
+import { join } from "path"
+
+import { exec } from "child_process"
+
+const target = join(__dirname, "..", "test-phenomic-theme-base")
+const execOpts = { cwd: target }
+
+const phenomic = "node ./node_modules/.bin/phenomic"
+const timing = process.env.CI ? 10000 : 5000
+
+describe("Integration: CLI", () => {
+  it("should throw if a CLI flag is NOT recognized", () => {
+    return new Promise((resolve, reject) => {
+      const child = exec(
+        `${ phenomic } start --open=false --lol`, execOpts,
+        (err) => {
+          if (err && !err.killed) {
+            clearTimeout(timeout)
+            expect(err.message.indexOf("Unknown argument") > -1).toBeTruthy()
+            resolve()
+          }
+        }
+      )
+
+      const timeout = setTimeout(() => {
+        child.kill()
+        reject("Test didn't finish before timeout")
+      }, timing)
+    })
+  })
+
+  it("should NOT throw if a CLI flag is recognized", () => {
+    return new Promise((resolve, reject) => {
+      const child = exec(
+        `${ phenomic } start --open=false --devPort=4000`, execOpts,
+
+        // should die quickly...
+        (err) => {
+          if (err && !err.killed) {
+            clearTimeout(timeout)
+            reject(err)
+          }
+        }
+      )
+
+      // ...or be ok quickly
+      // we assume it's ok and kill the process
+      // we don't need the actual build
+      const timeout = setTimeout(() => {
+        child.kill()
+        resolve()
+      }, timing)
+    })
+  })
+
+  fit("should NOT throw if port is used", () => {
+    return new Promise((resolve, reject) => {
+      const app = require("express")()
+      const server = app.listen(8081, (err) => {
+        if (err) {
+          reject(err)
+        }
+
+        const child = exec(
+          `${ phenomic } start --open=false --devPort=8081`, execOpts,
+
+          (err) => {
+            if (err && !err.killed) {
+              clearTimeout(timeout)
+              reject(err)
+            }
+          }
+        )
+
+        const timeout = setTimeout(() => {
+          child.kill()
+          server.close()
+          resolve()
+        }, timing * 2)
+      })
+    })
+  })
+})

package.json

@@ -173,8 +173,7 @@
     "lint:js:flow": "flow check",
     "lint:js": "npm-run-all --parallel lint:js:*",
     "lint": "npm-run-all --parallel lint:*",
-    "#integration-tests": "needs this order (index test the build, and cli is cleaning it)",
-    "integration-tests": "ava __tests__/index.js && ava __tests__/cli.js",
+    "integration-tests": "jest integration-tests",
     "pretest-phenomic-theme-base": "npm run transpile",
     "test-phenomic-theme-base": "babel-node scripts/test-phenomic-theme-base.js",
     "posttest-phenomic-theme-base": "npm run integration-tests",
@@ -224,9 +223,6 @@
     "preprocessorIgnorePatterns": [
       "node_modules"
     ],
-    "testPathDirs": [
-      "src"
-    ],
     "testPathIgnorePatterns": [
       "/fixtures/",
       "/results/",

src/bin/commands/setup/questions.js

@@ -1,7 +1,7 @@
 import validUrl from "valid-url"
 
 export const defaultTestAnswers = {
-  name: "Phenomic",
+  name: "Phenomic_app",
   homepage: "https://phenomic.io/",
   twitter: "Phenomic_app",
   CNAME: false,