Add messages for browser test processes (#53)

* Add messages for browser test processes

* Exclude tests from build

Author: mmarkelov

src/bin/testProcess.ts

@@ -1,6 +1,6 @@
 import { spawn, spawnSync, SpawnSyncOptions } from 'child_process'
 import { readConfig } from '../utils'
-import { checkBrowsers } from './utils'
+import { checkBrowsers, getResultByStatus } from './utils'
 import { PARALLEL, BrowserType } from '../constants'
 
 const getSpawnOptions = (browser: BrowserType): SpawnSyncOptions => ({
@@ -21,12 +21,19 @@ const exec = ({
   browser: BrowserType
   params: string[]
 }): void => {
-  // TODO Add messages for browser process
   const options = getSpawnOptions(browser)
   if (sequence === PARALLEL) {
-    spawn('node', [`node_modules/.bin/jest ${params}`], options)
+    const process = spawn('node', [`node_modules/.bin/jest ${params}`], options)
+    process.on('close', status => {
+      console.log(`${getResultByStatus(status)} tests for ${browser}\n\n`)
+    })
   } else {
-    spawnSync('node', [`node_modules/.bin/jest ${params}`], options)
+    const { status } = spawnSync(
+      'node',
+      [`node_modules/.bin/jest ${params}`],
+      options,
+    )
+    console.log(`${getResultByStatus(status)} tests for ${browser}`)
   }
 }
 

src/bin/utils.test.ts

@@ -1,4 +1,4 @@
-import { checkBrowsers } from './utils'
+import { checkBrowsers, getResultByStatus } from './utils'
 import { BrowserType } from '../constants'
 
 describe('checkBrowsers', () => {
@@ -18,3 +18,17 @@ describe('checkBrowsers', () => {
     ).toThrow()
   })
 })
+
+describe('getResultByStatus', () => {
+  it('should return "Failed" if passed null', () => {
+    expect(getResultByStatus(null)).toBe('Failed')
+  })
+
+  it('should return "Failed" if passed code 1', () => {
+    expect(getResultByStatus(1)).toBe('Failed')
+  })
+
+  it('should return "Passed" if passed code 0', () => {
+    expect(getResultByStatus(0)).toBe('Passed')
+  })
+})

src/bin/utils.ts

@@ -9,3 +9,7 @@ export const checkBrowsers = (browsers?: BrowserType[]): void => {
   }
   browsers.forEach(checkBrowserEnv)
 }
+
+export const getResultByStatus = (status: number | null): string => {
+  return status !== 0 ? 'Failed' : 'Passed'
+}

tsconfig.json

@@ -8,9 +8,13 @@
     "forceConsistentCasingInFileNames": true
   },
   "files": [
-    "types/global.d.ts",
+    "types/global.d.ts"
   ],
   "include": [
-    "src/**/*",
+    "src/**/*"
   ],
-}
+  "exclude": [
+    "node_modules",
+    "**/*.test.ts"
+  ]
+}