test: 'Tapable'

Author: ulivz

Diff for: package.json

@@ -6,10 +6,6 @@
     "packages/docs"
   ],
   "description": "Minimalistic doc generator with Vue component based layout system",
-  "main": "lib/index.js",
-  "bin": {
-    "vuepress": "bin/vuepress.js"
-  },
   "scripts": {
     "boot": "node scripts/bootstrap.js",
     "dev": "lerna run --scope docs dev",
@@ -45,6 +41,8 @@
   },
   "devDependencies": {
     "@vue/test-utils": "^1.0.0-beta.16",
+    "@babel/core": "^7.0.0-beta.47",
+    "@babel/preset-env": "^7.0.0-beta.47",
     "babel-core": "^7.0.0-0",
     "babel-jest": "^23.0.0",
     "conventional-changelog-cli": "^1.3.22",

Diff for: packages/@vuepress/core/__test__/plugin-api/Tapable.spec.js

@@ -1,3 +1,69 @@
-test('two plus two is four', () => {
-  expect(2 + 2).toBe(4)
+// TODO change to ES6 import
+// https://github.com/facebook/jest/issues/6835
+
+const Tapable = require('../../lib/plugin-api/core/Tapable')
+
+describe('Tapable', () => {
+  test('shoould tapable record the key', () => {
+    const tapable = new Tapable('option')
+    expect(tapable.key).toBe('option')
+  })
+
+  test('should \'tap\' work', () => {
+    const tapable = new Tapable('option')
+    tapable.tap('plugin-a', 'a')
+    tapable.tap('plugin-b', 'b')
+    expect(tapable.items).toEqual([
+      { value: 'a', name: 'plugin-a' },
+      { value: 'b', name: 'plugin-b' }
+    ])
+    expect(tapable.values).toEqual(['a', 'b'])
+  })
+
+  test('should \'tap\' resolve array value', () => {
+    const tapable = new Tapable('option')
+    tapable.tap('plugin-a', ['a-1', 'a-2'])
+    tapable.tap('plugin-b', 'b')
+    expect(tapable.items).toEqual([
+      { value: 'a-1', name: 'plugin-a' },
+      { value: 'a-2', name: 'plugin-a' },
+      { value: 'b', name: 'plugin-b' }
+    ])
+    expect(tapable.values).toEqual(['a-1', 'a-2', 'b'])
+  })
+
+  test('should \'run\' work', async () => {
+    const tapable = new Tapable('option')
+    const handler1 = jest.fn()
+    const handler2 = jest.fn()
+
+    tapable.tap('plugin-a', handler1)
+    tapable.tap('plugin-b', handler2)
+
+    await tapable.run(1, 2)
+    expect(handler1.mock.calls).toHaveLength(1)
+    expect(handler2.mock.calls).toHaveLength(1)
+    expect(handler1.mock.calls[0][0]).toBe(1)
+    expect(handler1.mock.calls[0][1]).toBe(2)
+    expect(handler2.mock.calls[0][0]).toBe(1)
+    expect(handler2.mock.calls[0][1]).toBe(2)
+  })
+
+  test('should \'parallelRun\' work', async () => {
+    const tapable = new Tapable('option')
+    const handler1 = jest.fn()
+    const handler2 = jest.fn()
+
+    tapable.tap('plugin-a', handler1)
+    tapable.tap('plugin-b', handler2)
+
+    await tapable.parallelRun(1, 2)
+    expect(handler1.mock.calls).toHaveLength(1)
+    expect(handler2.mock.calls).toHaveLength(1)
+    expect(handler1.mock.calls[0][0]).toBe(1)
+    expect(handler1.mock.calls[0][1]).toBe(2)
+    expect(handler2.mock.calls[0][0]).toBe(1)
+    expect(handler2.mock.calls[0][1]).toBe(2)
+  })
 })
+

Diff for: packages/@vuepress/core/lib/plugin-api/core/Tapable.js

@@ -5,7 +5,7 @@ module.exports = class Tapable {
   }
 
   /**
-   * set value with name.
+   * Set value with name.
    * @param {string} name
    * @param {any} value
    */
@@ -17,15 +17,15 @@ module.exports = class Tapable {
   }
 
   /**
-   * get values.
+   * Get values.
    * @returns {Array<T>}
    */
   get values () {
     return this.items.map(item => item.value)
   }
 
   /**
-   * execute in serial
+   * When T is function, eecute all functions in serial
    * @param args
    * @returns {Promise.<void>}
    */
@@ -36,7 +36,7 @@ module.exports = class Tapable {
   }
 
   /**
-   * execute in parallel
+   * When T is function, eecute all functions in parallel
    * @param args
    * @returns {Promise.<void>}
    */

Diff for: scripts/jest.config.js

@@ -15,8 +15,20 @@ module.exports = {
     'test.js',
     path.resolve(__dirname, '../test')
   ],
+  // projects: [
+  //   '<rootDir>/packages/@vuepress/*',
+  //   {
+  //     displayName: 'test',
+  //     verbose: true,
+  //     runner: 'jest',
+  //     testURL: 'http://localhost/',
+  //     testMatch: [
+  //       '<rootDir>/packages/@vuepress/*.js'
+  //     ]
+  //   }
+  // ],
   moduleNameMapper: {
-    '^@/(.*)$': '<rootDir>/lib/$1'
+    '^@/(.*)$': '<rootDir>/$1'
   },
   transform: {
     '^.+\\.js$': '<rootDir>/node_modules/babel-jest',

Diff for: scripts/test.js

@@ -12,8 +12,6 @@ if (args.p) {
   rawArgs.splice(i, 2)
 }
 
-console.log(regex)
-
 ;(async () => {
   const jestArgs = [
     '--env', 'node',