add tests in github actions

.github/workflows/test.yml

@@ -0,0 +1,30 @@
+name: Run tests
+on:
+  workflow_dispatch:
+  push:
+    branches:
+    - 'master'
+
+jobs:
+  test:
+    name: Run karma tests
+    runs-on: ubuntu-latest    
+    steps:
+    - uses: actions/checkout@v2
+    - name: Use Node.js
+      uses: actions/setup-node@v1
+      with:
+        node-version: 10.x
+    - name: Install chromium
+      run:  
+        sudo DEBIAN_FRONTEND=noninteractive apt-get update && 
+        sudo DEBIAN_FRONTEND=noninteractive apt-get install -y chromium-browser
+
+    - name: Install the project
+      run: npm install
+
+    - name: Run lint
+      run: npm run lint
+
+    - name: Run tests
+      run: npm run test:unit

karma.conf.js

@@ -1,31 +1,33 @@
 // Karma configuration
-"use strict";
+'use strict'
-const path = require("path");
+const path = require('path')
-const VueLoaderPlugin = require("vue-loader/lib/plugin");
+const VueLoaderPlugin = require('vue-loader/lib/plugin')
 
-function resolve(dir) {
+function resolve (dir) {
-  return path.join(__dirname, dir);
+  return path.join(__dirname, dir)
 }
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     // base path that will be used to resolve all patterns (eg. files, exclude)
-    basePath: "",
+    basePath: '',
 
     // frameworks to use
     // available frameworks: https://npmjs.org/browse/keyword/karma-adapter
-    frameworks: ["mocha", "sinon-chai"],
+    frameworks: ['mocha', 'sinon-chai'],
 
     // list of files / patterns to load in the browser
     files: [
-      "./karma.files.js",
+      './karma.files.js',
-      { pattern: 'node_modules/sql.js/dist/sql-wasm.wasm',
+      {
+        pattern: 'node_modules/sql.js/dist/sql-wasm.wasm',
         watched: false,
         included: false,
         served: true,
         nocache: false
       },
-      { pattern: 'node_modules/sql.js/dist/worker.sql-wasm.js',
+      {
+        pattern: 'node_modules/sql.js/dist/worker.sql-wasm.js',
         watched: false,
         included: false,
         served: true,
@@ -39,29 +41,29 @@ module.exports = function(config) {
     // preprocess matching files before serving them to the browser
     // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
     preprocessors: {
-      "./karma.files.js": ["webpack"]
+      './karma.files.js': ['webpack']
     },
 
     // test results reporter to use
     // possible values: 'dots', 'progress'
     // available reporters: https://npmjs.org/browse/keyword/karma-reporter
-    reporters: ["spec", "coverage"],
+    reporters: ['spec', 'coverage'],
 
     coverageReporter: {
-      dir: "coverage",
+      dir: 'coverage',
-      reporters: [{ type: "lcov", subdir: "." }, { type: "text-summary" }]
+      reporters: [{ type: 'lcov', subdir: '.' }, { type: 'text-summary' }]
     },
 
     // !!DONOT delete this reporter, or vue-cli-addon-ui-karma doesnot work
     jsonResultReporter: {
-      outputFile: "report/karma-result.json",
+      outputFile: 'report/karma-result.json',
       isSynchronous: true
     },
 
     junitReporter: {
-      outputDir: "report", // results will be saved as $outputDir/$browserName.xml
+      outputDir: 'report', // results will be saved as $outputDir/$browserName.xml
       outputFile: undefined, // if included, results will be saved as $outputDir/$browserName/$outputFile
-      suite: "", // suite will become the package name attribute in xml testsuite element
+      suite: '', // suite will become the package name attribute in xml testsuite element
       useBrowserName: true, // add browser name to report and classes names
       nameFormatter: undefined, // function (browser, result) to customize the name attribute in xml testcase element
       classNameFormatter: undefined, // function (browser, result) to customize the classname attribute in xml testcase element
@@ -83,7 +85,7 @@ module.exports = function(config) {
 
     // start these browsers
     // available browser launchers: https://npmjs.org/browse/keyword/karma-launcher
-    browsers: ["ChromiumHeadless"],
+    browsers: ['ChromiumHeadless'],
 
     // Continuous Integration mode
     // if true, Karma captures browsers, runs the tests and exits
@@ -98,16 +100,16 @@ module.exports = function(config) {
     },
     browserConsoleLogOptions: {
       terminal: true,
-      level: ""
+      level: ''
     },
     webpack: {
-      mode: "development",
+      mode: 'development',
-      entry: "./src/main.js",
+      entry: './src/main.js',
       resolve: {
-        extensions: [".js", ".vue", ".json"],
+        extensions: ['.js', '.vue', '.json'],
         alias: {
-          vue$: "vue/dist/vue.esm.js",
+          vue$: 'vue/dist/vue.esm.js',
-          "@": resolve("src")
+          '@': resolve('src')
         }
       },
       module: {
@@ -117,7 +119,7 @@ module.exports = function(config) {
             exclude: /(node_modules|bower_components)/,
             use: [
               {
-                loader: "babel-loader"
+                loader: 'babel-loader'
               }
             ]
           },
@@ -127,7 +129,7 @@ module.exports = function(config) {
             exclude: /(node_modules|bower_components|\.spec\.js$|\/file)/,
             use: [
               {
-                loader: "istanbul-instrumenter-loader",
+                loader: 'istanbul-instrumenter-loader',
                 options: {
                   esModules: true
                 }
@@ -136,34 +138,34 @@ module.exports = function(config) {
           },
           {
             test: /\.(png|jpe?g|gif|svg)(\?.*)?$/,
-            loader: "url-loader"
+            loader: 'url-loader'
           },
           {
             test: /\.vue$/,
-            loader: "vue-loader",
+            loader: 'vue-loader',
             options: {
               loaders: {
-                js: "babel-loader"
+                js: 'babel-loader'
               },
               postLoaders: {
-                js: "istanbul-instrumenter-loader?esModules=true"
+                js: 'istanbul-instrumenter-loader?esModules=true'
               }
             }
           },
           {
             test: /\.css$/,
-            use: ["vue-style-loader", "css-loader"]
+            use: ['vue-style-loader', 'css-loader']
           },
           {
             test: /\.scss$/,
-            use: ["vue-style-loader", "css-loader", "sass-loader"]
+            use: ['vue-style-loader', 'css-loader', 'sass-loader']
           },
           {
             test: /\.(woff2?|eot|ttf|otf)(\?.*)?$/,
-            loader: "url-loader",
+            loader: 'url-loader',
             options: {
               limit: 10000,
-              name: resolve("fonts/[name].[hash:7].[ext]")
+              name: resolve('fonts/[name].[hash:7].[ext]')
             }
           }
         ]
@@ -174,7 +176,7 @@ module.exports = function(config) {
       }
     },
     proxies: {
-      "/js/": "/base/node_modules/sql.js/dist/"
+      '/js/': '/base/node_modules/sql.js/dist/'
     }
-  });
+  })
-};
+}

karma.files.js

@@ -1,16 +1,16 @@
-import Vue from "vue";
+import Vue from 'vue'
 
-Vue.config.productionTip = false;
+Vue.config.productionTip = false
 
 // require all test files (files that ends with .spec.js)
-const testsContext = require.context("./tests/unit", true, /\.spec.js$/);
+const testsContext = require.context('./tests/unit', true, /\.spec.js$/)
 
 // Read more about why we need to call testContext:
 // https://www.npmjs.com/package/require-context#context-api
-testsContext.keys().forEach(testsContext);
+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", true, /^\.\/(?!main(\.js)?$)/);
+const srcContext = require.context('./src', true, /^\.\/(?!main(\.js)?$)/)
-srcContext.keys().forEach(srcContext);
+srcContext.keys().forEach(srcContext)

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "sqliteviz",
-  "version": "0.6.0",
+  "version": "1.0.0",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "sqliteviz",
-      "version": "0.6.0",
+      "version": "1.0.0",
       "license": "Apache-2.0",
       "dependencies": {
         "codemirror": "^5.57.0",
@@ -34,7 +34,7 @@
         "@vue/cli-plugin-vuex": "^4.4.0",
         "@vue/cli-service": "^4.4.0",
         "@vue/eslint-config-standard": "^5.1.2",
-        "@vue/test-utils": "^1.0.3",
+        "@vue/test-utils": "^1.1.2",
         "babel-eslint": "^10.1.0",
         "chai": "^4.1.2",
         "eslint": "^6.7.2",
@@ -2166,14 +2166,18 @@
       }
     },
     "node_modules/@vue/test-utils": {
-      "version": "1.0.3",
+      "version": "1.1.2",
-      "resolved": "https://registry.npmjs.org/@vue/test-utils/-/test-utils-1.0.3.tgz",
+      "resolved": "https://registry.npmjs.org/@vue/test-utils/-/test-utils-1.1.2.tgz",
-      "integrity": "sha512-mmsKXZSGfvd0bH05l4SNuczZ2MqlJH2DWhiul5wJXFxbf/gRRd2UL4QZgozEMQ30mRi9i4/+p4JJat8S4Js64Q==",
+      "integrity": "sha512-utbIL7zn9c+SjhybPwh48lpWCiluFCbP1yyRNAy1fQsw/6hiNFioaWy05FoVAFIZXC5WwBf+5r4ypfM1j/nI4A==",
       "dev": true,
       "dependencies": {
         "dom-event-types": "^1.0.0",
         "lodash": "^4.17.15",
         "pretty": "^2.0.0"
+      },
+      "peerDependencies": {
+        "vue": "2.x",
+        "vue-template-compiler": "^2.x"
       }
     },
     "node_modules/@vue/web-component-wrapper": {
@@ -24288,9 +24292,9 @@
       "dev": true
     },
     "@vue/test-utils": {
-      "version": "1.0.3",
+      "version": "1.1.2",
-      "resolved": "https://registry.npmjs.org/@vue/test-utils/-/test-utils-1.0.3.tgz",
+      "resolved": "https://registry.npmjs.org/@vue/test-utils/-/test-utils-1.1.2.tgz",
-      "integrity": "sha512-mmsKXZSGfvd0bH05l4SNuczZ2MqlJH2DWhiul5wJXFxbf/gRRd2UL4QZgozEMQ30mRi9i4/+p4JJat8S4Js64Q==",
+      "integrity": "sha512-utbIL7zn9c+SjhybPwh48lpWCiluFCbP1yyRNAy1fQsw/6hiNFioaWy05FoVAFIZXC5WwBf+5r4ypfM1j/nI4A==",
       "dev": true,
       "requires": {
         "dom-event-types": "^1.0.0",

package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "serve": "vue-cli-service serve",
     "build": "NODE_OPTIONS=--max_old_space_size=4096 vue-cli-service build",
-    "test:unit": "vue-cli-service test:unit",
+    "test:unit": "vue-cli-service karma",
     "lint": "vue-cli-service lint"
   },
   "dependencies": {
@@ -35,7 +35,7 @@
     "@vue/cli-plugin-vuex": "^4.4.0",
     "@vue/cli-service": "^4.4.0",
     "@vue/eslint-config-standard": "^5.1.2",
-    "@vue/test-utils": "^1.0.3",
+    "@vue/test-utils": "^1.1.2",
     "babel-eslint": "^10.1.0",
     "chai": "^4.1.2",
     "eslint": "^6.7.2",

tests/unit/database.spec.js

@@ -2,7 +2,7 @@ import { expect } from 'chai'
 import initSqlJs from 'sql.js'
 import db from '@/database.js'
 const config = {
-  locateFile: filename => `js/sql-wasm.wasm`
+  locateFile: filename => 'js/sql-wasm.wasm'
 }
 
 describe('database.js', () => {
@@ -23,7 +23,7 @@ describe('database.js', () => {
         const buffer = new Blob([data])
         return db.loadDb(buffer)
       })
-    .then(({dbName, schema}) => {
+      .then(({ dbName, schema }) => {
         expect(schema).to.have.lengthOf(1)
         expect(schema[0].name).to.equal('test')
         expect(schema[0].columns[0].name).to.equal('col1')
@@ -52,7 +52,7 @@ describe('database.js', () => {
         const buffer = new Blob([data])
         return db.loadDb(buffer)
       })
-    .then(({dbName, schema}) => {
+      .then(({ dbName, schema }) => {
         expect(schema[0].name).to.equal('test_virtual')
         expect(schema[0].columns[0].name).to.equal('col1')
         expect(schema[0].columns[0].type).to.equal('N/A')
@@ -81,7 +81,7 @@ describe('database.js', () => {
         const buffer = new Blob([data])
         return db.loadDb(buffer)
       })
-    .then(({dbName, schema}) => {
+      .then(({ dbName, schema }) => {
         return db.execute('SELECT * from test')
       })
       .then(result => {
@@ -123,7 +123,6 @@ describe('database.js', () => {
         return db.execute('SELECT * from foo')
       })
       .catch(result => {
-      console.log(result)
         expect(result).to.equal('no such table: foo')
       })
   })

tests/unit/store.spec.js

@@ -17,9 +17,12 @@ describe('mutations', () => {
     const state = {}
 
     const schema = [
-      { name: 'table1', columns: [
+      {
+        name: 'table1',
+        columns: [
           { name: 'id', type: 'INTEGER' }
-      ]}
+        ]
+      }
     ]
     saveSchema(state, {
       dbName: 'test',
@@ -323,7 +326,7 @@ describe('mutations', () => {
   it('setCurrentTab', () => {
     // mock state
     const state = {
-      currentTab: { id: 1}
+      currentTab: { id: 1 }
     }
 
     setCurrentTab(state, { id: 2 })
@@ -372,7 +375,6 @@ describe('mutations', () => {
   })
 })
 
-
 describe('actions', () => {
   it('addTab (new)', async () => {
     // mock state
@@ -429,8 +431,8 @@ describe('actions', () => {
 
     // mock state
     const state = {
-      tabs: [ tab1, tab2 ],
+      tabs: [tab1, tab2],
-      untitledLastIndex: 0,
+      untitledLastIndex: 0
     }
 
     await addTab({ state }, tab1)

tests/unit/storedQueries.spec.js

@@ -2,7 +2,7 @@ import { expect } from 'chai'
 import storedQueries from '@/storedQueries.js'
 
 describe('storedQueries.js', () => {
-  beforeEach(()=> {
+  beforeEach(() => {
     localStorage.removeItem('myQueries')
   })
 
@@ -14,7 +14,7 @@ describe('storedQueries.js', () => {
   it('getStoredQueries', () => {
     const data = [
       { id: 1 },
-      { id: 2 },
+      { id: 2 }
     ]
     storedQueries.updateStorage(data)
     const queries = storedQueries.getStoredQueries()
@@ -23,7 +23,7 @@ describe('storedQueries.js', () => {
 
   it('duplicateQuery', () => {
     const now = new Date()
-    const nowPlusMinute = new Date(now.getTime() + 60*1000)
+    const nowPlusMinute = new Date(now.getTime() + 60 * 1000)
     const base = {
       id: 1,
       name: 'foo',
@@ -43,14 +43,14 @@ describe('storedQueries.js', () => {
   })
 
   it('isTabNeedName returns false when the query has a name and is not predefined', () => {
-    let tab = {
+    const tab = {
       initName: 'foo'
     }
     expect(storedQueries.isTabNeedName(tab)).to.be.false
   })
 
   it('isTabNeedName returns true when the query has no name and is not predefined', () => {
-    let tab = {
+    const tab = {
       initName: null,
       tempName: 'Untitled'
     }
@@ -58,7 +58,7 @@ describe('storedQueries.js', () => {
   })
 
   it('isTabNeedName returns true when the qiery is predefined', () => {
-    let tab = {
+    const tab = {
       initName: 'foo',
       isPredefined: true
     }