reorganize tests

Author: jquense

.eslintrc

@@ -17,6 +17,7 @@
     "eol-last": 2,
     "dot-notation": [2, { "allowKeywords": true }],
     "semi": [0, "never"],
+    "semi-spacing": 0,
     "curly": 0,
     "eqeqeq": [2, "allow-null"],
     "no-undef": 2,

_test-bootstrap.js

@@ -78,7 +78,7 @@ module.exports = function(config) {
       '_test-bootstrap.js': ['webpack']
     },
 
-    webpack: require('./webpack.configs').test,
+    webpack: require('./build/test.config'),
     webpackServer: {
       noInfo: true
     },

karma.conf-ci.js

@@ -1,9 +1,6 @@
 'use strict';
 
 module.exports = function (config) {
-  
-  
-  //console.log(process.env.TRAVIS_CI)
 
   config.set({
 
@@ -15,7 +12,7 @@ module.exports = function (config) {
       './vendor/phantomjs-shim.js',
       './vendor/sinon-1.10.3.js',
       './vendor/jquery-1.11.2.min.js',
-      '_test-bootstrap.js',
+      './test/index.js'
     ],
 
     reporters: ['mocha'],
@@ -30,12 +27,12 @@ module.exports = function (config) {
     browsers: [ 'PhantomJS'],
 
     preprocessors: {
-      '_test-bootstrap.js': ['webpack', 'sourcemap']
+      'test/index.js': ['webpack', 'sourcemap']
     },
 
-    webpack: require('./webpack.configs').test,
+    webpack: require('./build/test.config'),
     webpackServer: {
       noInfo: true
     }
   });
-};
+};

karma.conf.js

@@ -26,7 +26,7 @@
     "react-component"
   ],
   "scripts": {
-    "test": "npm run lint && mocha -R spec ./test/server.js && node _test-versions.js",
+    "test": "npm run lint && npm run test:all",
     "test:all": "mocha -R spec ./test/server.js && karma start karma.conf.js --single-run=true",
     "tdd": "karma start",
     "lint": "eslint src --ext .jsx --ext .js",
@@ -43,14 +43,14 @@
     "build:lib": "npm run clean:lib && npm run compile:lib && npm run assets:lib",
     "build:docs": "npm run clean:docs && npm run compile:docs && npm run assets:docs",
     "build:dist": "npm run clean:dist && npm run compile:dist && npm run less && npm run assets:dist",
-    "build": "build:lib && build:dist & build:docs",
+    "build": "npm run build:lib && npm run build:dist & npm run build:docs",
     "docs": "npm run clean:docs && webpack-dev-server --config ./build/docs.config.js --hot",
     "dev": "webpack-dev-server --config ./build/dev.config.js --hot",
 
     "major": "release major",
     "minor": "release minor",
     "patch": "release patch",
-    "alpha": "release --preid beta",
+    "alpha": "release --preid alpha",
     "beta": "release --preid beta"
   },
   "homepage": "http://jquense.github.io/react-widgets/docs/",
@@ -112,7 +112,6 @@
     "react-bootstrap": "0.24.5-react-pre.0",
     "react-dom": "^0.14.0-beta3",
     "react-router": "^1.0.0-beta3",
-    "react-widgets-globalize-localizer": "file:../react-widgets-globalize-localizer",
     "release-script": "^0.2.7",
     "rimraf": "^2.4.2",
     "scriptjs": "^2.5.7",

package.json

@@ -276,7 +276,7 @@ var Multiselect = React.createClass({
           <div>
           { shouldRenderPopup && [
               <List ref="list"
-                key='0'
+                key={0}
                 {...listProps}
                 readOnly={!!readOnly}
                 disabled={!!disabled}
@@ -296,7 +296,7 @@ var Multiselect = React.createClass({
                 }}
               />,
               shouldShowCreate &&
-                <ul role='listbox' id={createID} className="rw-list rw-multiselect-create-tag" key='1'>
+                <ul key={1} role='listbox' id={createID} className="rw-list rw-multiselect-create-tag">
                   <li onClick={this._onCreate.bind(null, searchTerm)}
                       role='option'
                       id={createOptionID}

src/Multiselect.jsx

@@ -5,10 +5,7 @@ import CustomPropTypes from './util/propTypes';
 import { instanceId } from './util/widgetHelpers';
 import { dataText } from './util/dataHelpers';
 
-import {
-    move, contains
-  , isDisabled, isReadOnly
-  , isDisabledItem, isReadOnlyItem } from './util/interaction';
+import { isDisabled, isDisabledItem, isReadOnlyItem } from './util/interaction';
 
 let optionId = (id, idx)=> `${id}__option__${idx}`;
 

src/MultiselectTagList.jsx

@@ -3,10 +3,11 @@ import animate from './dom/animate';
 function error(){
   throw new Error(
     '[React Widgets] You are attempting to use a widget that requires localization (DateTimePicker, NumberPicker)' +
-    'but there is no localizer set. IN order to use these widgets please install a localizer')
+    'but there is no localizer set. In order to use these widgets please install a localizer')
 }
 
 let dummy = {
+  formats: error,
   parse: error,
   format:error,
   firstOfWeek: error

src/util/configuration.js

@@ -5,6 +5,7 @@
   "globals": {
     "$": true,
     "expect": true,
+    "sinon": true,
     "__REACT_VERSION__": true
   }
 }

test/.eslintrc

@@ -1,41 +1,42 @@
 'use strict';
-var server = require('karma').server
+var Server = require('karma').Server
   , webpack = require('webpack')
-  , webpackConfig = require('./webpack.configs').test;
+  , webpackConfig = require('../build/test.config');
 
 var plugins = webpackConfig.plugins || [];
 
 webpackConfig.externals = {
-  'react':  'window.React',
-  'react/addons':  'window.React'
+  'react': 'window.React',
+  'react-dom': 'window.React',
+  'react/addons': 'window.React',
+  'react-dom/server': 'window.React'
 }
 
 
-var SUPPORTED_VERSIONS = ["0.12.2", "0.13.0"];
+var SUPPORTED_VERSIONS = ['0.14.0-rc'];
 
 
 series(SUPPORTED_VERSIONS, function(version, idx, next){
   console.log('-------------------------------------------');
   console.log('------- Testing React version: ' + version );
   console.log('-------------------------------------------');
 
-  server.start(
-      config(version)
-    , function(exitCode) {
-        if ( exitCode )
-          process.exit(exitCode)
+  var server = new Server(config(version), function(exitCode) {
+    if ( exitCode )
+      process.exit(exitCode)
 
-        next();
-      });
-}, function(){
-  process.exit(0)
+    next();
+  });
+
+
+  server.start();
 })
 
 function config(version){
 
   webpackConfig.plugins = plugins.concat(
     new webpack.DefinePlugin({
-      '__REACT_VERSION__': JSON.stringify(version),
+      '__REACT_VERSION__': JSON.stringify(version)
     })
   )
 
@@ -50,7 +51,7 @@ function config(version){
       'https://cdnjs.cloudflare.com/ajax/libs/react/' + version + '/react-with-addons.js',
       './vendor/sinon-1.10.3.js',
       './vendor/jquery-1.11.2.min.js',
-      '_test-bootstrap.js'
+      './test/index.js'
     ],
 
     reporters: ['mocha'],
@@ -65,7 +66,7 @@ function config(version){
     browsers: ['PhantomJS'],
 
     preprocessors: {
-      '_test-bootstrap.js': ['webpack']
+      'test/index.js': ['webpack']
     },
 
     webpack: webpackConfig,