Browserify and webpack tests (#844)

* Browserify and webpack tests

* added tests

@@ -19,7 +19,9 @@
     "build:js:browserify": "browserify . -s videojs-contrib-hls -g browserify-shim -o dist/videojs-contrib-hls.js",
     "build:js:browserify": "browserify . -s videojs-contrib-hls -g browserify-shim -o dist/videojs-contrib-hls.js",
     "build:js:collapse": "bundle-collapser dist/videojs-contrib-hls.js -o dist/videojs-contrib-hls.min.js",
     "build:js:collapse": "bundle-collapser dist/videojs-contrib-hls.js -o dist/videojs-contrib-hls.min.js",
     "build:js:uglify": "uglifyjs dist/videojs-contrib-hls.min.js --comments -m -c -o dist/videojs-contrib-hls.min.js",
     "build:js:uglify": "uglifyjs dist/videojs-contrib-hls.min.js --comments -m -c -o dist/videojs-contrib-hls.min.js",
-    "build:test": "npm-run-all build:test:manifest build:test:js",
+    "build:test:browserify": "browserify test/browserify-test.js -o dist-test/browserify-test.js",
+    "build:test:webpack": "webpack test/webpack-test.js dist-test/webpack-test.js",
+    "build:test": "npm-run-all build:test:manifest build:test:js build:test:browserify build:test:webpack",
     "build:test:js": "node scripts/build-test.js",
     "build:test:js": "node scripts/build-test.js",
     "build:test:manifest": "node -e \"var b=require('./scripts/manifest-data.js'); b.build();\"",
     "build:test:manifest": "node -e \"var b=require('./scripts/manifest-data.js'); b.build();\"",
     "clean": "npm-run-all -p clean:*",
     "clean": "npm-run-all -p clean:*",
@@ -45,7 +47,9 @@
     "watch:js:babel": "npm run build:js:babel -- --watch",
     "watch:js:babel": "npm run build:js:babel -- --watch",
     "watch:js:browserify": "watchify . -v -g browserify-shim -o dist/videojs-contrib-hls.js",
     "watch:js:browserify": "watchify . -v -g browserify-shim -o dist/videojs-contrib-hls.js",
     "watch:js:switcher": "watchify utils/switcher/switcher.js -v -t babelify -g browserify-shim -o dist/switcher.js",
     "watch:js:switcher": "watchify utils/switcher/switcher.js -v -t babelify -g browserify-shim -o dist/switcher.js",
-    "watch:test": "npm-run-all -p watch:test:*",
+    "watch:test:browserify": "watchify test/browserify-test.js -o dist-test/browserify-test.js",
+    "watch:test:webpack": "watchify test/webpack-test.js -o dist-test/webpack-test.js",
+    "watch:test": "npm-run-all -p watch:test:js watch:test:webpack watch:test:browserify",
     "watch:test:js": "node scripts/watch-test.js",
     "watch:test:js": "node scripts/watch-test.js",
     "watch:test:manifest": "node -e \"var b=require('./scripts/manifest-data.js'); b.watch();\"",
     "watch:test:manifest": "node -e \"var b=require('./scripts/manifest-data.js'); b.watch();\"",
     "prepublish": "npm run build"
     "prepublish": "npm run build"
@@ -125,6 +129,7 @@
     "sinon": "1.10.2",
     "sinon": "1.10.2",
     "uglify-js": "^2.5.0",
     "uglify-js": "^2.5.0",
     "videojs-standard": "^4.0.3",
     "videojs-standard": "^4.0.3",
-    "watchify": "^3.6.0"
+    "watchify": "^3.6.0",
+    "webpack": "^1.13.2"
   }
   }
 }
 }

+/* eslint-disable no-var */
+/* eslint-env qunit */
+var hls = require('../es5/videojs-contrib-hls.js');
+var q = window.QUnit;
+
+q.module('Browserify Require');
+q.test('hls should be requirable and bundled via browserify', function(assert) {
+  assert.ok(hls, 'videoj-contrib-hls is required properly');
+});

@@ -12,6 +12,8 @@
   <!-- NOTE in order for test to pass we require sinon 1.10.2 exactly -->
   <!-- NOTE in order for test to pass we require sinon 1.10.2 exactly -->
   <script src="/node_modules/sinon/pkg/sinon.js"></script>
   <script src="/node_modules/sinon/pkg/sinon.js"></script>
   <script src="/node_modules/qunitjs/qunit/qunit.js"></script>
   <script src="/node_modules/qunitjs/qunit/qunit.js"></script>
+  <script src="/dist-test/browserify-test.js"></script>
+  <script src="/dist-test/webpack-test.js"></script>
   <script src="/node_modules/video.js/dist/video.js"></script>
   <script src="/node_modules/video.js/dist/video.js"></script>
   <script src="/dist-test/videojs-contrib-hls.js"></script>
   <script src="/dist-test/videojs-contrib-hls.js"></script>
 
 

@@ -10,7 +10,9 @@ var DEFAULTS = {
     'node_modules/sinon/pkg/sinon-ie.js',
     'node_modules/sinon/pkg/sinon-ie.js',
     'node_modules/video.js/dist/video.js',
     'node_modules/video.js/dist/video.js',
     'node_modules/video.js/dist/video-js.css',
     'node_modules/video.js/dist/video-js.css',
-    'test/**/*.test.js'
+    'test/**/*.test.js',
+    'dist-test/browserify-test.js',
+    'dist-test/webpack-test.js'
   ],
   ],
 
 
   exclude: [],
   exclude: [],
@@ -20,6 +22,11 @@ var DEFAULTS = {
     'karma-qunit'
     'karma-qunit'
   ],
   ],
 
 
+  browserConsoleLogOptions: {
+    level: 'error',
+    terminal: false
+  },
+
   preprocessors: {
   preprocessors: {
     'test/**/*.test.js': ['browserify']
     'test/**/*.test.js': ['browserify']
   },
   },

+let hls = require('../es5/videojs-contrib-hls.js');
+let q = window.QUnit;
+
+q.module('Webpack Require');
+q.test('hls should be requirable and bundled via webpack', function(assert) {
+  assert.ok(hls, 'videojs-contrib-hls is required properly');
+});