Merge pull request #11 from BiYuqi/improve-test-cases

Improve test cases

Author: BiYuqi

karma.conf.js

@@ -66,14 +66,6 @@ module.exports = function(config) {
     // level of logging
     // possible values: config.LOG_DISABLE || config.LOG_ERROR || config.LOG_WARN || config.LOG_INFO || config.LOG_DEBUG
     logLevel: config.LOG_INFO,
-    
-    // 设置此项 报错信息不提示
-    // client: {
-    //   captureConsole: true,
-    //   mocha: {
-    //     bail: true
-    //   }
-    // },
 
     // enable / disable watching file and executing tests whenever any file changes
     autoWatch: true,
@@ -101,6 +93,11 @@ module.exports = function(config) {
               loader: 'babel-loader'
             }
           },
+          {
+            test: /\.ejs$/,
+            include: resolve('src'),
+            loader: 'ejs-loader'
+          },
           {
             test: /\.(sa|sc|c)ss$/,
             use: [
@@ -114,22 +111,47 @@ module.exports = function(config) {
                 loader: 'sass-loader'
               }
             ]
+          },
+          {
+            test: /\.(png|jpe?g|gif|svg)(\?.*)?$/,
+            loader: 'url-loader',
+            options: {
+              limit: 10000,
+              name: 'image/[name].[hash:7].[ext]'
+            }
+          },
+          {
+            test: /\.(mp4|webm|ogg|mp3|wav|flac|aac)(\?.*)?$/,
+            loader: 'url-loader',
+            options: {
+              limit: 10000,
+              name: 'media/[name].[hash:7].[ext]'
+            }
+          },
+          {
+            test: /\.(woff2?|eot|ttf|otf)(\?.*)?$/,
+            loader: 'url-loader',
+            options: {
+              limit: 10000,
+              name: 'fonts/[name].[hash:7].[ext]'
+            }
           }
         ]
       },
       resolve: {
-        extensions: ['.js'],
+        extensions: ['.js', '.json'],
+        // 配置项目文件别名
         alias: {
-          '@': resolve('./src'),
-          utils: resolve('./src/utils')
+          '@': resolve('src'),
+          assets: resolve('src/common/assets'),
+          utils: resolve('src/common/utils'),
+          layout: resolve('src/layout'),
+          build: resolve('build')
         }
       }
     },
     webpackServer: {
       noInfo: true
     }
-    // webpackMiddleware: {
-    //   stats: 'errors-only'
-    // }
   })
 }

src/common/utils/tools.js

@@ -8,3 +8,14 @@ export const isEmptyObject = obj => {
   }
   return true
 }
+
+export const pickBy = object => {
+  if (!object) return
+  const result = {}
+  Object.keys(object).forEach(key => {
+    if (object[key]) {
+      result[key] = object[key]
+    }
+  })
+  return result
+}

src/templates/__test__/index.test-test.js

@@ -0,0 +1,29 @@
+import { expect } from 'chai'
+import EjsTest from '@/templates/index.test.ejs'
+
+describe('Test Ejs Template', () => {
+  const data = {
+    index: [
+      {
+        link: 'html/test-demo.html',
+        key: '测试页面',
+        icon: 'iconfont icon-rocket'
+      }
+    ]
+  }
+  const html = EjsTest(data)
+  const classReg = /class="([^"]+)"/gm
+  const hrefReg = /href="([^"]+)"/
+
+  it('should render correct className ejs-dynamic-item', () => {
+    expect(classReg.exec(html)[1]).to.equal('ejs-dynamic-item')
+  })
+
+  it('should render correct icon className iconfont icon-rocket', () => {
+    expect(classReg.exec(html)[1]).to.equal('iconfont icon-rocket')
+  })
+
+  it('should render correct href link', () => {
+    expect(hrefReg.exec(html)[1]).to.equal('html/test-demo.html')
+  })
+})

test/utils-test.js

@@ -1,12 +1,37 @@
 import { expect } from 'chai'
-import { isEmptyObject } from '@/common/utils/tools.js'
+import { isEmptyObject, pickBy } from '@/common/utils/tools.js'
 
-describe('Test project tools\'s method', () => {
-  it('isEmptyObject\'s param  empty object should be return true', () => {
-    expect(isEmptyObject({})).to.equal(true)
+describe('Test utils', () => {
+  describe('isEmptyObject', () => {
+    it('should return true', () => {
+      expect(isEmptyObject({})).to.equal(true)
+    })
+
+    it('should return false', () => {
+      expect(isEmptyObject({ a: 1 })).to.equal(false)
+    })
   })
 
-  it('isEmptyObject\'s param not empty object should be return false', () => {
-    expect(isEmptyObject({a: 1})).to.equal(false)
+  describe('pickBy', () => {
+    it('should return correct value', () => {
+      const testCaseOne = {
+        testA: 'testA',
+        testB: null,
+        testC: false
+      }
+      expect(pickBy(testCaseOne)).to.deep.equal({
+        testA: 'testA'
+      })
+    })
+
+    it('should return correct value', () => {
+      const testCaseTwo = {
+        testD: 'testD',
+        testE: NaN
+      }
+      expect(pickBy(testCaseTwo)).to.deep.equal({
+        testD: 'testD'
+      })
+    })
   })
-})
+})