From bce8cf9ce9cf0d9ad91af0e7a69f8670e84eff47 Mon Sep 17 00:00:00 2001
From: AkiraFukushima <h3.poteto@gmail.com>
Date: Thu, 12 Jul 2018 00:46:55 +0900
Subject: [PATCH] refs #434 Add mocha test and run in circleci

---
 .babelrc                                    |  4 +-
 .circleci/config.yml                        |  5 +--
 package-lock.json                           | 38 ++++++++-----------
 package.json                                |  5 ++-
 src/renderer/components/Login/LoginForm.vue |  3 +-
 src/renderer/utils/validator.js             |  1 +
 test/mocha/validator.spec.js                | 42 +++++++++++++++++++++
 test/unit/index.js                          |  2 +-
 test/unit/specs/LandingPage.spec.js         | 22 +++++------
 9 files changed, 80 insertions(+), 42 deletions(-)
 create mode 100644 src/renderer/utils/validator.js
 create mode 100644 test/mocha/validator.spec.js

diff --git a/.babelrc b/.babelrc
index 386f3224..d0598528 100644
--- a/.babelrc
+++ b/.babelrc
@@ -4,7 +4,7 @@
     "test": {
       "presets": [
         ["env", {
-          "targets": { "node": 7 }
+          "targets": { "node": 9 }
         }],
         "stage-0"
       ],
@@ -13,7 +13,7 @@
     "main": {
       "presets": [
         ["env", {
-          "targets": { "node": 7 }
+          "targets": { "node": 9 }
         }],
         "stage-0"
       ]
diff --git a/.circleci/config.yml b/.circleci/config.yml
index 1e31c0f8..bbe05dec 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -19,6 +19,5 @@ jobs:
           paths:
             - node_modules
       - run:
-          name: Eslint
-          command: NODE_ENV=production npm run lint
-
+          name: mocha
+          command: npm run mocha
diff --git a/package-lock.json b/package-lock.json
index f55cbfd2..decfd15b 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -8085,9 +8085,9 @@
       "dev": true
     },
     "growl": {
-      "version": "1.10.3",
-      "resolved": "https://registry.npmjs.org/growl/-/growl-1.10.3.tgz",
-      "integrity": "sha512-hKlsbA5Vu3xsh1Cg3J7jSmX/WaW6A5oBeqzM88oNbCRQFz+zUaXm6yxS4RVytp1scBoJzSYl4YAEOQIt6O8V1Q==",
+      "version": "1.10.5",
+      "resolved": "https://registry.npmjs.org/growl/-/growl-1.10.5.tgz",
+      "integrity": "sha512-qBr4OuELkhPenW6goKVXiv47US3clb3/IbuWF9KNKEijAy9oeHxU9IgzjvJhHkUzhaj7rOUD7+YGWqUjLp5oSA==",
       "dev": true
     },
     "handle-thing": {
@@ -11202,43 +11202,37 @@
       }
     },
     "mocha": {
-      "version": "5.1.1",
-      "resolved": "https://registry.npmjs.org/mocha/-/mocha-5.1.1.tgz",
-      "integrity": "sha512-kKKs/H1KrMMQIEsWNxGmb4/BGsmj0dkeyotEvbrAuQ01FcWRLssUNXCEUZk6SZtyJBi6EE7SL0zDDtItw1rGhw==",
+      "version": "5.2.0",
+      "resolved": "https://registry.npmjs.org/mocha/-/mocha-5.2.0.tgz",
+      "integrity": "sha512-2IUgKDhc3J7Uug+FxMXuqIyYzH7gJjXECKe/w43IGgQHTSj3InJi+yAA7T24L9bQMRKiUEHxEX37G5JpVUGLcQ==",
       "dev": true,
       "requires": {
         "browser-stdout": "1.3.1",
-        "commander": "2.11.0",
+        "commander": "2.15.1",
         "debug": "3.1.0",
         "diff": "3.5.0",
         "escape-string-regexp": "1.0.5",
         "glob": "7.1.2",
-        "growl": "1.10.3",
+        "growl": "1.10.5",
         "he": "1.1.1",
         "minimatch": "3.0.4",
         "mkdirp": "0.5.1",
-        "supports-color": "4.4.0"
+        "supports-color": "5.4.0"
       },
       "dependencies": {
         "commander": {
-          "version": "2.11.0",
-          "resolved": "https://registry.npmjs.org/commander/-/commander-2.11.0.tgz",
-          "integrity": "sha512-b0553uYA5YAEGgyYIGYROzKQ7X5RAqedkfjiZxwi0kL1g3bOaBNNZfYkzt/CL0umgD5wc9Jec2FbB98CjkMRvQ==",
-          "dev": true
-        },
-        "has-flag": {
-          "version": "2.0.0",
-          "resolved": "https://registry.npmjs.org/has-flag/-/has-flag-2.0.0.tgz",
-          "integrity": "sha1-6CB68cx7MNRGzHC3NLXovhj4jVE=",
+          "version": "2.15.1",
+          "resolved": "https://registry.npmjs.org/commander/-/commander-2.15.1.tgz",
+          "integrity": "sha512-VlfT9F3V0v+jr4yxPc5gg9s62/fIVWsd2Bk2iD435um1NlGMYdVCq+MjcXnhYq2icNOizHr1kK+5TI6H0Hy0ag==",
           "dev": true
         },
         "supports-color": {
-          "version": "4.4.0",
-          "resolved": "https://registry.npmjs.org/supports-color/-/supports-color-4.4.0.tgz",
-          "integrity": "sha512-rKC3+DyXWgK0ZLKwmRsrkyHVZAjNkfzeehuFWdGGcqGDTZFH73+RH6S/RDAAxl9GusSjZSUWYLmT9N5pzXFOXQ==",
+          "version": "5.4.0",
+          "resolved": "https://registry.npmjs.org/supports-color/-/supports-color-5.4.0.tgz",
+          "integrity": "sha512-zjaXglF5nnWpsq470jSv6P9DwPvgLkuapYmfDm3JWOm0vkNTVF2tI4UrN2r6jH1qM/uc/WtxYY1hYoA2dOKj5w==",
           "dev": true,
           "requires": {
-            "has-flag": "2.0.0"
+            "has-flag": "3.0.0"
           }
         }
       }
diff --git a/package.json b/package.json
index 054f705a..8f398362 100644
--- a/package.json
+++ b/package.json
@@ -32,7 +32,8 @@
     "pack:renderer": "cross-env NODE_ENV=production webpack --progress --colors --config .electron-vue/webpack.renderer.config.js",
     "test": "npm run unit && npm run e2e",
     "unit": "karma start test/unit/karma.conf.js",
-    "postinstall": "npm run lint:fix"
+    "postinstall": "npm run lint:fix",
+    "mocha": "BABEL_ENV=test mocha --require babel-register --recursive ./test/mocha/**/*.js"
   },
   "build": {
     "productName": "Whalebird",
@@ -160,7 +161,7 @@
     "karma-sourcemap-loader": "^0.3.7",
     "karma-spec-reporter": "^0.0.31",
     "karma-webpack": "^2.0.1",
-    "mocha": "^5.1.1",
+    "mocha": "^5.2.0",
     "multispinner": "^0.2.1",
     "node-loader": "^0.6.0",
     "node-sass": "^4.9.0",
diff --git a/src/renderer/components/Login/LoginForm.vue b/src/renderer/components/Login/LoginForm.vue
index 7f282ba6..60ba2016 100644
--- a/src/renderer/components/Login/LoginForm.vue
+++ b/src/renderer/components/Login/LoginForm.vue
@@ -29,6 +29,7 @@
 
 <script>
 import { mapState } from 'vuex'
+import { domainFormat } from '../../utils/validator'
 
 export default {
   name: 'login-form',
@@ -45,7 +46,7 @@ export default {
             message: 'Domain name is required'
           },
           {
-            pattern: /^[a-zA-Z0-9][a-zA-Z0-9-]{1,61}[a-zA-Z0-9]\.[a-zA-Z]{2,}$/,
+            pattern: domainFormat,
             trigger: 'change',
             message: 'Please write only domain name'
           }
diff --git a/src/renderer/utils/validator.js b/src/renderer/utils/validator.js
new file mode 100644
index 00000000..87e936e1
--- /dev/null
+++ b/src/renderer/utils/validator.js
@@ -0,0 +1 @@
+export const domainFormat = new RegExp(/^[a-zA-Z0-9][a-zA-Z0-9-]{1,61}[a-zA-Z0-9]\.[a-zA-Z]{2,}$/)
diff --git a/test/mocha/validator.spec.js b/test/mocha/validator.spec.js
new file mode 100644
index 00000000..75ae5bd8
--- /dev/null
+++ b/test/mocha/validator.spec.js
@@ -0,0 +1,42 @@
+var assert = require('assert')
+import { domainFormat } from '../../src/renderer/utils/validator'
+
+describe('validator', () => {
+  describe('domainFormat', () => {
+    context('string contains protocol', () => {
+      const domain = 'https://mastodon.social'
+      it('should not match', () => {
+        const res = domain.search(domainFormat)
+        assert.equal(res, -1)
+      })
+    })
+    context('string contains account name', () => {
+      const domain = 'h3_poteto@mastodon.social'
+      it('should not match', () => {
+        const res = domain.search(domainFormat)
+        assert.equal(res, -1)
+      })
+    })
+    context('string is gTLD domain', () => {
+      const domain = 'mastodon.social'
+      it('should match', () => {
+        const res = domain.search(domainFormat)
+        assert.equal(res, 0)
+      })
+    })
+    context('string is subdomain', () => {
+      const domain = 'music.mastodon.social'
+      it('should match', () => {
+        const res = domain.search(domainFormat)
+        assert.equal(res, 0)
+      })
+    })
+    context('string is jp domain', () => {
+      const domain = 'mstdn.co.jp'
+      it('should match', () => {
+        const res = domain.search(domainFormat)
+        assert.equal(res, 0)
+      })
+    })
+  })
+})
diff --git a/test/unit/index.js b/test/unit/index.js
index f07be986..630193dc 100644
--- a/test/unit/index.js
+++ b/test/unit/index.js
@@ -9,5 +9,5 @@ testsContext.keys().forEach(testsContext)
 // require all src files except main.js for coverage.
 // you can also change this to match only the subset of files that
 // you want coverage for.
-const srcContext = require.context('../../src/renderer', true, /^\.\/(?!main(\.js)?$)/)
+const srcContext = require.context('../../src/renderer', true, /^(?!.*(main\.js|~|\.scss)).*$/)
 srcContext.keys().forEach(srcContext)
diff --git a/test/unit/specs/LandingPage.spec.js b/test/unit/specs/LandingPage.spec.js
index 7cea66ff..7ac2c28d 100644
--- a/test/unit/specs/LandingPage.spec.js
+++ b/test/unit/specs/LandingPage.spec.js
@@ -1,13 +1,13 @@
-import Vue from 'vue'
-import LandingPage from '@/components/LandingPage'
+// import Vue from 'vue'
+// import LandingPage from '@/components/LandingPage'
 
-describe('LandingPage.vue', () => {
-  it('should render correct contents', () => {
-    const vm = new Vue({
-      el: document.createElement('div'),
-      render: h => h(LandingPage)
-    }).$mount()
+// describe('LandingPage.vue', () => {
+//   it('should render correct contents', () => {
+//     const vm = new Vue({
+//       el: document.createElement('div'),
+//       render: h => h(LandingPage)
+//     }).$mount()
 
-    expect(vm.$el.querySelector('.title').textContent).to.contain('Welcome to your new project!')
-  })
-})
+//     expect(vm.$el.querySelector('.title').textContent).to.contain('Welcome to your new project!')
+//   })
+// })