chore(build): move to rollup over browserify

Author: keithamus

.gitignore

@@ -19,3 +19,5 @@ npm-debug.log
 coverage/
 
 type-detect.js
+type-detect.test.js
+.nyc_output

karma.conf.js

@@ -1,7 +1,6 @@
 /* eslint-disable no-process-env */
 const packageJson = require('./package.json');
 const defaultTimeout = 120000;
-const browserifyIstanbul = require('browserify-istanbul');
 module.exports = function configureKarma(config) {
   const localBrowsers = [
     'PhantomJS',
@@ -42,19 +41,9 @@ module.exports = function configureKarma(config) {
     basePath: '',
     browsers: localBrowsers,
     logLevel: process.env.npm_config_debug ? config.LOG_DEBUG : config.LOG_INFO,
-    frameworks: [ 'browserify', 'mocha' ],
-    files: [ 'test/*.js' ],
+    frameworks: [ 'mocha' ],
+    files: [ 'type-detect.test.js' ],
     exclude: [],
-    preprocessors: {
-      'test/*.js': [ 'browserify' ],
-    },
-    browserify: {
-      debug: true,
-      bare: true,
-      transform: [
-        browserifyIstanbul({ ignore: [ '**/node_modules/**', '**/test/**' ] }),
-      ],
-    },
     reporters: [ 'progress', 'coverage' ],
     coverageReporter: {
       type: 'lcov',

package.json

@@ -24,15 +24,15 @@
   },
   "scripts": {
     "bench": "node bench",
-    "build": "browserify --bare $npm_pakcage_main --standalone typeDetect -o type-detect.js",
+    "build": "rollup -c rollup.conf.js",
     "commit-msg": "validate-commit-msg",
     "lint": "eslint --ignore-path .gitignore .",
-    "prepublish": "npm run build",
+    "prepublish": "NODE_ENV=production npm run build",
     "semantic-release": "semantic-release pre && npm publish && semantic-release post",
-    "pretest": "npm run lint",
+    "pretest": "npm run lint && NODE_ENV=test npm run build",
     "test": "npm run test:node && npm run test:browser && npm run upload-coverage",
     "test:browser": "karma start --singleRun=true",
-    "test:node": "istanbul cover _mocha",
+    "test:node": "nyc mocha type-detect.test.js",
     "upload-coverage": "lcov-result-merger 'coverage/**/lcov.info' | coveralls; exit 0"
   },
   "eslintConfig": {
@@ -51,30 +51,32 @@
       "prefer-rest-params": 0
     }
   },
-  "dependencies": {},
   "devDependencies": {
     "benchmark": "^2.1.0",
-    "browserify": "^14.3.0",
-    "browserify-istanbul": "^2.0.0",
+    "buble": "^0.15.2",
     "coveralls": "2.13.0",
     "eslint": "^3.19.0",
     "eslint-config-strict": "^13.0.0",
-    "eslint-plugin-filenames": "^1.1.0",
-    "husky": "^0.13.3",
-    "istanbul": "^0.4.2",
-    "karma": "^1.1.2",
-    "karma-browserify": "^5.0.2",
-    "karma-coverage": "^1.0.0",
-    "karma-mocha": "^1.0.1",
+    "eslint-plugin-filenames": "^1.2.0",
+    "husky": "^0.13.4",
+    "karma": "^1.7.1",
+    "karma-coverage": "^1.1.1",
+    "karma-mocha": "^1.3.0",
     "karma-phantomjs-launcher": "^1.0.0",
-    "karma-sauce-launcher": "^1.0.0",
+    "karma-sauce-launcher": "^1.2.0",
     "lcov-result-merger": "^1.2.0",
-    "mocha": "^3.0.0",
-    "phantomjs-prebuilt": "^2.1.5",
-    "semantic-release": "^6.3.2",
+    "mocha": "^3.5.1",
+    "nyc": "^11.2.1",
+    "phantomjs-prebuilt": "^2.1.15",
+    "rollup": "^0.49.3",
+    "rollup-plugin-buble": "^0.15.0",
+    "rollup-plugin-commonjs": "^8.2.1",
+    "rollup-plugin-coverage": "^0.1.4",
+    "rollup-plugin-node-resolve": "^3.0.0",
+    "semantic-release": "^6.3.6",
     "simple-assert": "^1.0.0",
-    "travis-after-all": "^1.4.4",
-    "validate-commit-msg": "^2.3.1"
+    "travis-after-all": "^1.4.5",
+    "validate-commit-msg": "^2.14.0"
   },
   "engines": {
     "node": "*"

rollup.conf.js

@@ -0,0 +1,19 @@
+import coverage from 'rollup-plugin-coverage';
+import common from 'rollup-plugin-commonjs';
+import resolve from 'rollup-plugin-node-resolve';
+import transform from 'rollup-plugin-buble';
+const env = process.env.NODE_ENV || 'test'; // eslint-disable-line no-process-env
+export default {
+  input: env === 'test' ? 'test/index.js' : 'index.js',
+  output: {
+    file: env === 'test' ? 'type-detect.test.js' : 'type-detect.js',
+    name: 'typeDetect',
+    format: 'iife',
+  },
+  plugins: [
+    env === 'test' ? coverage({ exclude: 'test/*.js' }) : null,
+    common(),
+    resolve(),
+    transform(),
+  ].filter(Boolean),
+};

test/index.js

@@ -1,5 +1,9 @@
 import assert from 'simple-assert';
 import type from '..';
+import './dom.js';
+import './new-ecmascript-types.js';
+import './node.js';
+import './tostringtag-extras.js';
 describe('Generic', () => {
 
   it('array', () => {
@@ -235,9 +239,10 @@ describe('Generic', () => {
   describe('@@toStringTag Sham', () => {
     const originalObjectToString = Object.prototype.toString;
     before(() => {
-      self.Symbol = self.Symbol || {};
-      if (!self.Symbol.toStringTag) {
-        self.Symbol.toStringTag = '__@@toStringTag__';
+      const globalObject = typeof self === 'object' ? self : global;
+      globalObject.Symbol = globalObject.Symbol || {};
+      if (!Symbol.toStringTag) {
+        Symbol.toStringTag = '__@@toStringTag__';
       }
       const test = {};
       test[Symbol.toStringTag] = function () {