replace constant generator algoritm

add unit test
add phantom and node environment test

Author: Jogis

.travis.yml

@@ -3,4 +3,5 @@ node_js:
   - "5"
 script:
   - npm run lint
-  - npm test
+  - npm test
+  - npm run test-node

karma.conf.js

@@ -0,0 +1,24 @@
+var webpack = require('webpack');
+var devConfig = require('./webpack.config.base');
+
+module.exports = function (config) {
+  config.set({
+
+    // browsers: [ process.env.CONTINUOUS_INTEGRATION ? 'Firefox' : 'Chrome' ],
+    browsers: ['PhantomJS'],
+
+    frameworks: [ 'mocha', 'phantomjs-shim' ],
+
+    reporters: [ 'dots' ],
+
+    files: [
+      "test/**/*.js"
+    ],
+
+    preprocessors: {
+      'test/**/*.js': [ 'webpack', 'sourcemap' ]
+    },
+
+    webpack: devConfig,
+  });
+};

log.txt

@@ -1,6 +1,6 @@
 {
   "name": "react-constant",
-  "version": "1.1.2",
+  "version": "1.2.0",
   "description": "Fuck off constants.js and golobal constants in react/flux/redux/wateva.",
   "main": "lib/index.js",
   "files": [
@@ -11,8 +11,10 @@
   "scripts": {
     "clean": "rimraf lib dist coverage",
     "lint": "eslint src test examples",
-    "test": "mocha --compilers js:babel-core/register --recursive",
-    "test:watch": "npm test -- --watch",
+    "test":"karma start --single-run",
+    "test-node": "mocha --compilers js:babel-core/register --recursive",
+    "test:watch": "karma start karma.conf.js --auto-watch",
+    "test-node:watch": "mocha --compilers js:babel-core/register --recursive --watch",
     "check": "npm run lint && npm run test",
     "build:lib": "babel src --out-dir lib",
     "build:umd": "webpack src/index.js dist/constant.js --config webpack.config.development.js",
@@ -45,9 +47,15 @@
     "babel-preset-stage-0": "^6.1.18",
     "eslint": "^1.10.1",
     "eslint-config-rackt": "^1.1.1",
+    "karma": "^0.13.15",
+    "karma-mocha": "^0.2.1",
+    "karma-phantomjs-launcher": "^0.2.1",
+    "karma-phantomjs-shim": "^1.0.0",
+    "karma-sourcemap-loader": "^0.3.5",
+    "karma-webpack": "^1.7.0",
     "mocha": "^2.3.4",
+    "phantomjs": "^1.9.19",
     "rimraf": "^2.4.4",
-    "should": "^7.1.1",
-    "webpack": "^1.12.8"
+    "should": "^7.1.1"
   }
 }

package.json

@@ -1,4 +1,4 @@
-import uuidGenerator from './uuidGenerator';
+import {randomBase64} from './fnUtils';
 
 /**
 * generate and return a constant
@@ -15,7 +15,7 @@ export default class Constants {
     if(typeof this[key] !== 'undefined') {
       return this[key];
     }
-    this[key] = `${this.namespace}-${key}-${uuidGenerator()}`;
+    this[key] = `${this.namespace}-${key}-${randomBase64()}`;
     return this[key];
   }
 }

src/utils/Constants.js

@@ -0,0 +1,29 @@
+/**
+ * Generate random base 64 which length
+ * determinate by param , default to 5;
+ * @param  {...[number, number]} arg  slice start index
+ *  and end index
+ * @return {String}        encoded string
+ */
+export function randomBase64(...arg){
+  let start = typeof arg[0] === 'number'?arg[0]:-5;
+  let end = typeof arg[1] === 'number'?arg[1]:undefined;
+  let codedString ;
+  if('object' === typeof process && Object.prototype.toString.call(process) === '[object process]'){
+    codedString = new Buffer(`${Math.random()}`.slice(2))
+                      .toString('base64')
+                      .replace(/\=/g,'')
+                      .slice(start, end);
+  }else if('undefined' !==typeof btoa){
+    codedString = btoa(Math.random())
+                  .replace('=')
+                  .slice(start, end);
+  }else{
+    codedString = '';
+  }
+
+  if(codedString.length>20){
+    codedString = codedString.slice(0,20)
+  }
+  return codedString;
+}

src/utils/fnUtils.js

@@ -0,0 +1,29 @@
+import {randomBase64} from '../../src/utils/fnUtils';
+import should from 'should';
+
+describe('fnUtils', () => {
+  it('get different value in every invoke', () => {
+    let tmp = [];
+    const base64Encoded = randomBase64();
+    for(let i=0; i<10;i++){
+      tmp.push(randomBase64())
+    }
+    let index = tmp.indexOf(base64Encoded);
+    should(index).lessThan(0);
+  });
+
+  it('fnUtils should return 5 chars default', () => {
+    const base64Encoded = randomBase64();
+    should(base64Encoded.length).equal(5);
+  });
+
+  it('should return spec number by spec param', () => {
+    const base64Encoded = randomBase64(0, 6);
+    should(base64Encoded.length).equal(6);
+  })
+
+  it('should return no more than 20 chars', () => {
+    const base64Encoded = randomBase64(0);
+    should(base64Encoded.length).belowOrEqual(20);
+  })
+})

test/utils/constantHub.js test/utils/constantHub.spec.jstest/utils/constantHub.js renamed to test/utils/constantHub.spec.js

@@ -12,5 +12,6 @@ module.exports = {
   },
   resolve: {
     extensions: ['', '.js']
-  }
+  },
+  devtool: 'source-map',
 };