[Dev Deps] update babel-cli, cheerio, eslint, `eslint-config-ai…

…rbnb`, `eslint-plugin-import`, `eslint-plugin-jsx-a11y`, `eslint-plugin-react`, `mocha`, `mocha-wrap`, `rimraf`, `sinon`

package.json

@@ -50,21 +50,21 @@
   },
   "homepage": "https://github.com/airbnb/hypernova",
   "devDependencies": {
-    "babel-cli": "^6.9.0",
+    "babel-cli": "^6.14.0",
     "babel-plugin-add-module-exports": "^0.2.1",
     "babel-preset-airbnb": "^2.0.0",
     "chai": "^3.5.0",
-    "cheerio": "^0.20.0",
-    "eslint": "^2.11.1",
-    "eslint-config-airbnb": "^9.0.1",
-    "eslint-plugin-import": "^1.8.1",
-    "eslint-plugin-jsx-a11y": "^1.3.0",
-    "eslint-plugin-react": "^5.1.1",
+    "cheerio": "^0.22.0",
+    "eslint": "^3.4.0",
+    "eslint-config-airbnb": "^10.0.1",
+    "eslint-plugin-import": "^1.14.0",
+    "eslint-plugin-jsx-a11y": "^2.2.1",
+    "eslint-plugin-react": "^6.2.0",
     "istanbul": "^1.0.0-alpha.2",
-    "mocha": "^2.5.3",
-    "mocha-wrap": "^2.0.4",
-    "rimraf": "^2.5.2",
-    "sinon": "^1.17.4",
+    "mocha": "^3.0.2",
+    "mocha-wrap": "^2.0.5",
+    "rimraf": "^2.5.4",
+    "sinon": "^1.17.5",
     "sinon-sandbox": "^1.0.2"
   },
   "dependencies": {

src/coordinator.js

@@ -1,6 +1,7 @@
-import './environment';
 import cluster from 'cluster';
 import os from 'os';
+
+import './environment';
 import logger from './utils/logger';
 import { raceTo } from './utils/lifecycle';
 

src/createGetComponent.js

@@ -1,7 +1,8 @@
-import createVM from './createVM';
 import fs from 'fs';
 import has from 'has';
 
+import createVM from './createVM';
+
 // This function takes in an Object of files and an Object that configures the VM. It will return
 // a function that can be used as `getComponent` for Hypernova.
 // The file's object structure is [componentName]: 'AbsolutePath.js'

src/index.js

@@ -1,3 +1,5 @@
+/* globals document */
+
 const LEFT = '<!--';
 const RIGHT = '-->';
 

src/server.js

@@ -1,10 +1,10 @@
-import './environment';
+import cluster from 'cluster';
+import express from 'express';
 
+import './environment';
 import Module from './Module';
-import cluster from 'cluster';
 import coordinator from './coordinator';
 import createGetComponent from './createGetComponent';
-import express from 'express';
 import getFiles from './getFiles';
 import loadModules from './loadModules';
 import logger from './utils/logger';
@@ -36,12 +36,10 @@ export default function hypernova(userConfig, onServer) {
 
   if (config.devMode) {
     worker(app, config, onServer);
+  } else if (cluster.isMaster) {
+    coordinator();
   } else {
-    if (cluster.isMaster) {
-      coordinator();
-    } else {
-      worker(app, config, onServer, cluster.worker.id);
-    }
+    worker(app, config, onServer, cluster.worker.id);
   }
 
   return app;

src/worker.js

@@ -1,5 +1,6 @@
-import './environment';
 import bodyParser from 'body-parser';
+
+import './environment';
 import logger from './utils/logger';
 import renderBatch from './utils/renderBatch';
 import { runAppLifecycle, errorSync, raceTo } from './utils/lifecycle';

test/.eslintrc

@@ -1,5 +1,10 @@
 {
   "env": {
     "mocha": true
+  },
+  "rules": {
+    "import/no-extraneous-dependencies": [2, {
+      "devDependencies": true
+    }],
   }
 }

test/BatchManager-test.js

@@ -1,6 +1,7 @@
 import { assert } from 'chai';
-import { makeJob, COMPONENT_NAME } from './helper';
 import sinon from 'sinon-sandbox';
+
+import { makeJob, COMPONENT_NAME } from './helper';
 import BatchManager from '../lib/utils/BatchManager';
 
 function mockPlugin() {

test/Module-test.js

@@ -1,4 +1,5 @@
 import { assert } from 'chai';
+import has from 'has';
 import { Module } from '../server';
 import mutableArray from './mutableArray';
 
@@ -27,7 +28,7 @@ describe('Module', () => {
 
   it('loads a module and return the instance', () => {
     const module = Module.load('./test/mutableArray.js');
-    assert(module.hasOwnProperty('exports') === true, 'module has exports property');
+    assert(has(module, 'exports') === true, 'module has exports property');
     assert.isArray(module.exports, 'module.exports is our array');
   });
 

test/createGetComponent-test.js

@@ -1,6 +1,6 @@
 import { assert } from 'chai';
-import { createGetComponent } from '../server';
 import path from 'path';
+import { createGetComponent } from '../server';
 
 describe('createGetComponent', () => {
   const files = {

test/getFiles-test.js

@@ -1,7 +1,8 @@
 import { assert } from 'chai';
-import { getFiles } from '../server';
 import path from 'path';
 
+import { getFiles } from '../server';
+
 describe('getFiles', () => {
   it('retrieves files', () => {
     const files = getFiles(path.join('test', 'components'));

test/hypernova-runner-test.js

@@ -1,8 +1,9 @@
 import { assert } from 'chai';
-import hypernova from '..';
 import sinon from 'sinon-sandbox';
 import wrap from 'mocha-wrap';
 
+import hypernova from '../';
+
 describe('the runner', () => {
   it('runs server if window is not defined', () => {
     const server = sinon.spy();

test/init.js

@@ -1,4 +1,5 @@
-require('airbnb-js-shims');
+import 'airbnb-js-shims';
+
 import sinon from 'sinon-sandbox';
 
 afterEach(() => {

test/lifecycle-test.js

@@ -1,7 +1,8 @@
-import '../lib/environment';
 import { assert } from 'chai';
-import { makeJob } from './helper';
 import sinon from 'sinon';
+
+import '../lib/environment';
+import { makeJob } from './helper';
 import * as lifecycle from '../lib/utils/lifecycle';
 import BatchManager from '../lib/utils/BatchManager';
 

test/server-test.js

@@ -1,6 +1,7 @@
-import hypernova from '../server';
 import { assert } from 'chai';
 
+import hypernova from '../server';
+
 describe('Hypernova server', () => {
   it('blows up if hypernova does not get getComponent', () => {
     assert.throws(hypernova, TypeError);