Bring over a commit that brings jest testing to all packages

package.json

@@ -26,11 +26,29 @@
   },
   "scripts": {
     "clean": "lerna clean",
-    "test": "lerna run test",
+    "test": "jest",
     "lint": "npm run lint:js && npm run lint:style",
     "lint:js": "eslint 'packages/**/*.{js,jsx}'",
     "lint:style": "stylelint 'packages/**/*.{css,scss}'",
     "precommit": "lint-staged"
   },
+  "jest": {
+    "collectCoverage": true,
+    "collectCoverageFrom": [
+      "packages/cli/src/*.js",
+      "packages/cli/cli/*.js",
+      "client/src/**/*.{js,jsx}",
+      "components/**/*.{js,jsx}",
+      "db-manager/src/**/*.js",
+      "logger/src/**/*.js",
+      "server/src/**/*.js",
+      "sse/lib/**/*.js",
+      "theme-plugin/*.js",
+      "ui/src/**/*.{js,jsx}",
+      "!**/*.test.{js,jsx}",
+      "!**/node_modules/**"
+    ],
+    "projects": ["packages/*"]
+  },
   "workspaces": ["packages/*", "packages/components/packages/*"]
 }

packages/cli/package.json

@@ -43,13 +43,14 @@
     "nsp": "^2.8.1"
   },
   "jest": {
-    "testMatch": ["**/test/*.js", "**/test/cli/*.js"],
+    "testMatch": ["<rootDir>/test/*.test.js", "<rootDir>/test/cli/*.test.js"],
     "testPathIgnorePatterns": ["<rootDir>/build/"],
     "collectCoverage": true,
     "collectCoverageFrom": ["src/*.js", "cli/*.js"],
     "modulePaths": ["<rootDir>/node_modules"],
     "testEnvironment": "node",
     "unmockedModulePathPatterns": ["/src/models"],
+    "setupTestFrameworkScriptFile": "<rootDir>/test/jest-setup.js",
     "verbose": true
   }
 }

packages/cli/test/cli/add.js → packages/cli/test/cli/add.test.js

@@ -24,9 +24,7 @@ const writeSpy = fs.writeJsonSync
 
 describe('add', () => {
   beforeAll(() => {
-    process.chdir(
-      path.join(__dirname, '..', '..', 'node_modules', '@pubsweet', 'starter'),
-    )
+    process.chdir(path.dirname(require.resolve('@pubsweet/starter')))
   })
 
   beforeEach(() => {

packages/cli/test/cli/remove.js → packages/cli/test/cli/remove.test.js

@@ -24,9 +24,7 @@ const writeSpy = fs.writeJsonSync
 
 describe('remove', () => {
   beforeAll(() => {
-    process.chdir(
-      path.join(__dirname, '..', '..', 'node_modules', '@pubsweet', 'starter'),
-    )
+    process.chdir(path.dirname(require.resolve('@pubsweet/starter')))
   })
 
   beforeEach(() => {

packages/cli/test/helpers/index.js

@@ -3,7 +3,7 @@ const { spawn } = require('child_process')
 const path = require('path')
 const reduce = require('lodash/fp/reduce').convert({ cap: false })
 
-const pubsweet = path.resolve('bin/pubsweet.js')
+const pubsweet = path.resolve(__dirname, '..', '..', 'bin', 'pubsweet.js')
 
 const formatOpts = reduce(
   (acc, value, key) => acc.concat(`--${key}`, value),
@@ -41,6 +41,7 @@ const runCommandAsync = ({ args, options, cwd, nodeConfig, stdio }) =>
     stdio,
     env: getEnvWithConfig(nodeConfig),
     shell: true,
+    detached: true,
   })
 
 module.exports = {

packages/cli/test/integration.js → packages/cli/test/integration.test.js

@@ -18,6 +18,18 @@ const nodeConfig = {
     dbPath,
     adapter: 'leveldb',
   },
+  // TODO: Remove this once version of server that handles
+  // undefined app validations is released.
+  validations: 'path',
+  authsome: {
+    mode: 'authsome/src/modes/blog',
+  },
+  pubsweet: {
+    components: [],
+  },
+  'pubsweet-client': {
+    theme: 'PepperTheme',
+  },
 }
 
 const dbOptions = {
@@ -155,11 +167,17 @@ describe('CLI: integration test', () => {
       app.stdout.on('data', async data => {
         console.log('stdout:', data.toString())
         if (data.toString().includes('App is listening')) {
-          const result = await fetch('http://localhost:4000')
+          const result = await fetch('http://localhost:3000')
           expect(result.status).toBe(200)
-          done()
+          console.log('Killing the app')
+          process.kill(-app.pid)
         }
       })
+
+      app.on('close', (code, signal) => {
+        console.log(`App killed ${signal}`)
+        done()
+      })
     })
   })
 })

packages/cli/test/jest-setup.js

+const path = require('path')
+
+process.env.NODE_CONFIG_DIR = path.resolve(__dirname, '..', 'config')
+
+process.cwd = () => path.resolve(__dirname, '..')

packages/client/package.json

@@ -58,6 +58,7 @@
   "jest": {
     "testRegex": "/test/.+test.jsx?$",
     "collectCoverage": true,
+    "setupTestFrameworkScriptFile": "<rootDir>/test/jest-setup.js",
     "collectCoverageFrom": ["src/**/*.{js,jsx}"],
     "globals": {
       "window": {}

packages/client/test/jest-setup.js

+import Enzyme from 'enzyme'
+import Adapter from 'enzyme-adapter-react-16'
+import path from 'path'
+
+Enzyme.configure({ adapter: new Adapter() })
+
+process.env.NODE_CONFIG_DIR = path.resolve(__dirname, '..', 'config')

packages/components/package.json

@@ -51,6 +51,7 @@
       "\\.s?css$": "identity-obj-proxy"
     },
     "transformIgnorePatterns": ["/node_modules/(?!pubsweet)"],
-    "testPathIgnorePatterns": ["/node_modules", "config/"]
+    "testPathIgnorePatterns": ["/node_modules", "config/"],
+    "setupTestFrameworkScriptFile": "<rootDir>/test/jest-setup.js"
   }
 }

packages/components/test/jest-setup.js

+import Enzyme from 'enzyme'
+import Adapter from 'enzyme-adapter-react-16'
+import path from 'path'
+
+Enzyme.configure({ adapter: new Adapter() })
+
+process.env.NODE_CONFIG_DIR = path.resolve(__dirname, '..', 'config')

packages/db-manager/package.json

@@ -19,10 +19,10 @@
     "npm": ">=3.0.0"
   },
   "jest": {
-    "testMatch": ["**/test/**/*.js"],
+    "testMatch": ["**/test/**/*.test.js"],
     "collectCoverage": true,
     "collectCoverageFrom": ["src/**/*.js"],
-    "modulePaths": ["<rootDir>/node_modules"],
+    "setupTestFrameworkScriptFile": "<rootDir>/test/jest-setup.js",
     "testEnvironment": "node",
     "verbose": true
   },

packages/db-manager/test/jest-setup.js

+const path = require('path')
+
+process.env.NODE_CONFIG_DIR = path.resolve(__dirname, '..', 'config')

packages/logger/package.json

@@ -25,12 +25,13 @@
     "winston": "^2.3.1"
   },
   "jest": {
-    "testMatch": ["**/test/**/*.js"],
+    "testMatch": ["**/test/**/*test.js"],
     "collectCoverage": true,
     "collectCoverageFrom": ["src/**/*.js"],
     "modulePaths": ["<rootDir>/node_modules"],
     "testEnvironment": "node",
-    "verbose": true
+    "verbose": true,
+    "setupTestFrameworkScriptFile": "<rootDir>/test/jest-setup.js"
   },
   "dependencies": {
     "config": "^1.26.2",

packages/logger/test/jest-setup.js

+const path = require('path')
+
+process.env.NODE_CONFIG_DIR = path.resolve(__dirname, '..', 'config')

packages/server/package.json

@@ -65,6 +65,7 @@
     "testEnvironment": "node",
     "testRegex": "/test/.*_test.js$",
     "collectCoverage": true,
-    "collectCoverageFrom": ["src/**/*.js"]
+    "collectCoverageFrom": ["src/**/*.js"],
+    "setupTestFrameworkScriptFile": "<rootDir>/test/jest-setup.js"
   }
 }