Merge pull request #253 from we3x/feature/testing

Clear test
one-love · Jun 24, 2016 · 02a9971 · 02a9971
2 parents 8ccbffc + c4ca8a5
commit 02a9971
Show file tree

Hide file tree

Showing 50 changed files with 515 additions and 408 deletions.
diff --git a/package.json b/package.json
@@ -5,6 +5,7 @@
   "main": "server.js",
   "scripts": {
     "start": "webpack-dev-server --inline --hot --host 0.0.0.0 --color",
+    "pretest": "eslint src/",
     "test": "NODE_ENV=production node_modules/.bin/babel-node node_modules/.bin/babel-istanbul cover node_modules/.bin/_mocha -- -R xunit-file src/test.js",
     "coverage": "NODE_ENV=production istanbul report cobertura --dir shippable/codecoverage/",
     "build": "webpack -p --config webpack.config.prod.js src/index.js",

diff --git a/src/components/auth/tests/auth.js b/src/components/auth/tests/auth.js
@@ -1,28 +1,31 @@
+/* eslint no-undef: 0 */
+/* eslint no-unused-expressions: 0 */
+/* eslint no-sequences: 0 */
 import expect from 'expect';
 import actions from '../actions';
 import store from '../../../store';
 import { LOGIN } from '../constants';
 
 const loginTest = describe('Testing login', () => {
-   it('get pending state', () => {
-     expect(store.dispatch(actions.begin()))
+  it('get pending state', () => {
+    expect(store.dispatch(actions.begin()))
       .toEqual({
         payload: {
           status: 'pending',
         },
         type: LOGIN,
       });
   }),
-   it('get error state', () => {
-     expect(store.dispatch(actions.fail({ message: 'ErorR'})))
-      .toEqual({
-        payload: {
-          status: 'error',
-          error: 'ErorR',
-        },
-        type: LOGIN,
-      });
-   });
+  it('get error state', () => {
+    expect(store.dispatch(actions.fail({ message: 'ErorR' })))
+     .toEqual({
+       payload: {
+         status: 'error',
+         error: 'ErorR',
+       },
+       type: LOGIN,
+     });
+  });
 });
 
 export default loginTest;
diff --git a/src/components/cluster/components/provider/components/create.js b/src/components/cluster/components/provider/components/create.js
@@ -106,10 +106,10 @@ const ProviderCreate = React.createClass({
               {
                 this.props.plugins.map(
                   plugin =>
-                  <option
-                    key={plugin.type}
-                    value={plugin.type}
-                  > {plugin.type}</option>
+                    <option
+                      key={plugin.type}
+                      value={plugin.type}
+                    > {plugin.type}</option>
                 )
               }
             </select>

diff --git a/src/components/cluster/components/provider/components/detail.js b/src/components/cluster/components/provider/components/detail.js
@@ -60,25 +60,28 @@ const ProviderDetail = React.createClass({
     }
     return (
       <div>
-        <h2>Provider {this.props.provider.name}</h2>
-          <ul className="item__list">
-            <li className="item__heading">
-              Name:
-              <InlineEdit
-                text={this.props.provider.name}
-                change={this.dataChanged}
-                paramName="name"
-              />
-            </li>
-            <li className="item__child">
-              <Link to={
-                `/clusters/${clusterId}/providers/${this.props.provider.name}/hosts/`
-                }
-              >
-                Hosts
-              </Link>
-            </li>
-          </ul>
+        <h2>
+          Provider {this.props.provider.name}
+        </h2>
+        <ul className="item__list">
+          <li className="item__heading">
+            Name:
+            <InlineEdit
+              text={this.props.provider.name}
+              change={this.dataChanged}
+              paramName="name"
+            />
+          </li>
+          <li className="item__child">
+            <Link
+              to={
+              `/clusters/${clusterId}/providers/${this.props.provider.name}/hosts/`
+              }
+            >
+              Hosts
+            </Link>
+          </li>
+        </ul>
         <Link to={`/clusters/${clusterId}/providers/${this.props.provider.name}/remove/`}>
           Remove
         </Link>

diff --git a/src/components/cluster/components/provider/components/host/components/create.js b/src/components/cluster/components/provider/components/host/components/create.js
@@ -3,7 +3,7 @@ import { connect } from 'react-redux';
 import actions from '../actions/create';
 import store from '../../../../../../../store';
 import { history } from '../../../../../../../constants';
-import Spinner from '../../../.../../../../../layout/spinner';
+import Spinner from '../../../../../../layout/spinner';
 
 
 const errorMessages = {

diff --git a/src/components/cluster/components/provider/components/host/index.js b/src/components/cluster/components/provider/components/host/index.js
@@ -48,11 +48,12 @@ const HostList = React.createClass({
                 detailLink += `/hosts/${host.hostname}/`;
                 return (
                   <li key={host.hostname}>
-                  <Link
-                    key={host.hostname}
-                    to={detailLink}
-                    host={host}
-                  > {host.hostname} </Link> </li>
+                    <Link
+                      key={host.hostname}
+                      to={detailLink}
+                      host={host}
+                    > {host.hostname} </Link>
+                  </li>
                 );
               }
             )

diff --git a/src/components/cluster/components/provider/components/host/tests/create.js b/src/components/cluster/components/provider/components/host/tests/create.js
@@ -1,18 +1,21 @@
+/* eslint no-undef: 0 */
+/* eslint no-unused-expressions: 0 */
+/* eslint no-sequences: 0 */
 import expect from 'expect';
 import actions from '../actions/create';
 import store from '../../../../../../../store';
 import { HOST_CREATE } from '../constants';
 
 
 const hostCreateTest = describe('Testing create of host', () => {
-   it('get initial state', () => {
-     expect(store.dispatch(actions.reset()))
+  it('get initial state', () => {
+    expect(store.dispatch(actions.reset()))
       .toEqual({
         payload: {
           status: 'initial',
         },
         type: HOST_CREATE,
-     })
+      });
   }),
    it('get success state', () => {
      expect(store.dispatch(actions.success({})))
@@ -22,17 +25,17 @@ const hostCreateTest = describe('Testing create of host', () => {
           status: 'success',
         },
         type: HOST_CREATE,
-     })
-  }),
+      });
+   }),
    it('get pending state', () => {
      expect(store.dispatch(actions.begin()))
       .toEqual({
         payload: {
           status: 'pending',
         },
         type: HOST_CREATE,
-     })
-  }),
+      });
+   }),
    it('get error state', () => {
      expect(store.dispatch(actions.fail('ErorR')))
       .toEqual({
@@ -41,8 +44,8 @@ const hostCreateTest = describe('Testing create of host', () => {
           error: 'ErorR',
         },
         type: HOST_CREATE,
-     })
-  })
+      });
+   });
 });
 
 export default hostCreateTest;
diff --git a/src/components/cluster/components/provider/components/host/tests/detail.js b/src/components/cluster/components/provider/components/host/tests/detail.js
@@ -1,18 +1,21 @@
+/* eslint no-undef: 0 */
+/* eslint no-unused-expressions: 0 */
+/* eslint no-sequences: 0 */
 import expect from 'expect';
 import actions from '../actions/detail';
 import store from '../../../../../../../store';
 import { HOST_DETAIL } from '../constants';
 
 
 const hostDetailTest = describe('Testing detail of host', () => {
-   it('get initial state', () => {
-     expect(store.dispatch(actions.reset()))
+  it('get initial state', () => {
+    expect(store.dispatch(actions.reset()))
       .toEqual({
         payload: {
           status: 'initial',
         },
         type: HOST_DETAIL,
-     })
+      });
   }),
    it('get success state', () => {
      expect(store.dispatch(actions.success({})))
@@ -22,17 +25,17 @@ const hostDetailTest = describe('Testing detail of host', () => {
           status: 'success',
         },
         type: HOST_DETAIL,
-     })
-  }),
+      });
+   }),
    it('get pending state', () => {
      expect(store.dispatch(actions.begin()))
       .toEqual({
         payload: {
           status: 'pending',
         },
         type: HOST_DETAIL,
-     })
-  }),
+      });
+   }),
    it('get error state', () => {
      expect(store.dispatch(actions.fail('ErorR')))
       .toEqual({
@@ -41,8 +44,8 @@ const hostDetailTest = describe('Testing detail of host', () => {
           error: 'ErorR',
         },
         type: HOST_DETAIL,
-     })
-  })
+      });
+   });
 });
 
 export default hostDetailTest;
diff --git a/src/components/cluster/components/provider/components/host/tests/edit.js b/src/components/cluster/components/provider/components/host/tests/edit.js
@@ -1,18 +1,21 @@
+/* eslint no-undef: 0 */
+/* eslint no-unused-expressions: 0 */
+/* eslint no-sequences: 0 */
 import expect from 'expect';
 import actions from '../actions/edit';
 import store from '../../../../../../../store';
 import { HOST_EDIT } from '../constants';
 
 
 const hostEditTest = describe('Testing edit of host', () => {
-   it('get initial state', () => {
-     expect(store.dispatch(actions.reset()))
+  it('get initial state', () => {
+    expect(store.dispatch(actions.reset()))
       .toEqual({
         payload: {
           status: 'initial',
         },
         type: HOST_EDIT,
-     })
+      });
   }),
    it('get success state', () => {
      expect(store.dispatch(actions.success({})))
@@ -22,17 +25,17 @@ const hostEditTest = describe('Testing edit of host', () => {
           status: 'success',
         },
         type: HOST_EDIT,
-     })
-  }),
+      });
+   }),
    it('get pending state', () => {
      expect(store.dispatch(actions.begin()))
       .toEqual({
         payload: {
           status: 'pending',
         },
         type: HOST_EDIT,
-     })
-  }),
+      });
+   }),
    it('get error state', () => {
      expect(store.dispatch(actions.fail('ErorR')))
       .toEqual({
@@ -41,8 +44,8 @@ const hostEditTest = describe('Testing edit of host', () => {
           error: 'ErorR',
         },
         type: HOST_EDIT,
-     })
-  })
+      });
+   });
 });
 
 export default hostEditTest;
diff --git a/src/components/cluster/components/provider/components/host/tests/list.js b/src/components/cluster/components/provider/components/host/tests/list.js
@@ -1,19 +1,22 @@
+/* eslint no-undef: 0 */
+/* eslint no-unused-expressions: 0 */
+/* eslint no-sequences: 0 */
 import expect from 'expect';
 import actions from '../actions/list';
 import store from '../../../../../../../store';
 import { HOST_LIST } from '../constants';
 
 
 const hostListTest = describe('Testing host list', () => {
-   it('get initial state', () => {
-     expect(store.dispatch(actions.reset()))
+  it('get initial state', () => {
+    expect(store.dispatch(actions.reset()))
       .toEqual({
         payload: {
           status: 'initial',
           hosts: [],
         },
         type: HOST_LIST,
-     })
+      });
   }),
    it('get success state', () => {
      expect(store.dispatch(actions.success({})))
@@ -23,8 +26,8 @@ const hostListTest = describe('Testing host list', () => {
           status: 'success',
         },
         type: HOST_LIST,
-     })
-  }),
+      });
+   }),
    it('get pending state', () => {
      expect(store.dispatch(actions.begin()))
       .toEqual({
@@ -33,19 +36,19 @@ const hostListTest = describe('Testing host list', () => {
           hosts: [],
         },
         type: HOST_LIST,
-     })
-  }),
+      });
+   }),
 
    it('get error state', () => {
      expect(store.dispatch(actions.fail('erroR')))
       .toEqual({
         payload: {
           status: 'error',
-          error: 'erroR'
+          error: 'erroR',
         },
         type: HOST_LIST,
-     })
-  })
+      });
+   });
 });
 
 export default hostListTest;