Skip to content

Commit

Permalink
more test coverage (#238)
Browse files Browse the repository at this point in the history
* update dev dependencies
* add test for audio-video
* add test for destroy/enumerateDevices
* add tests for getDuration/getCurrentTime/pause/resume
* add test for reset
* add stopDevice test
* test against local firefox
* add test for setAudioOutput
* replace deprecated banner script
* rename deprecated npm command (fixes #232)
  • Loading branch information
thijstriemstra authored May 6, 2018
1 parent 02f1b31 commit e30986e
Show file tree
Hide file tree
Showing 15 changed files with 552 additions and 378 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ addons:
chrome: stable
language: node_js
node_js:
- '9'
- node
install:
- npm prune
- npm install
Expand Down
5 changes: 1 addition & 4 deletions build-config/fragments/dev.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,8 @@ const path = require('path');
module.exports = {
mode: 'development',
devtool: 'source-map',
output: {
publicPath: 'localhost:8080/dist/'
},
devServer: {
contentBase: [path.resolve(__dirname, '../', '../')],
contentBase: [path.resolve(__dirname, '..', '..')],
publicPath: 'localhost:8080/dist/',
watchContentBase: true
}
Expand Down
3 changes: 1 addition & 2 deletions build-config/fragments/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@ const path = require('path');
module.exports = {
entry: {
'videojs.record': path.resolve(
__dirname,
'../', '../', 'src', 'js', 'videojs.record.js'
__dirname, '..', '..', 'src', 'js', 'videojs.record.js'
)
},
output: {
Expand Down
6 changes: 3 additions & 3 deletions build-config/fragments/plugins.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@ function buildPluginEntry(plugins) {
plugin =>
(result[plugin.split('-plugin')[0]] = path.resolve(
__dirname,
'../',
'../',
'..',
'..',
'src',
'js',
'plugins',
Expand All @@ -33,7 +33,7 @@ module.exports = {
'recorderjs-plugin'
]),
output: {
path: path.resolve(__dirname, '../', '../', 'dist', 'plugins'),
path: path.resolve(__dirname, '..', '..', 'dist', 'plugins'),
filename: 'videojs.record.[name].js',
library: ['VideojsRecord', '[name]']
}
Expand Down
37 changes: 0 additions & 37 deletions build-config/scripts/add-banner.js

This file was deleted.

3 changes: 0 additions & 3 deletions build-config/scripts/banner.css

This file was deleted.

1 change: 1 addition & 0 deletions karma.conf.js
Original file line number Diff line number Diff line change
Expand Up @@ -125,6 +125,7 @@ module.exports = function(config) {
reporters: ['verbose', 'progress', 'coverage'],
coverageReporter: {
type: 'html',
// specify a common output directory
dir: 'coverage/'
},
webpack: webpackConfig,
Expand Down
Loading

0 comments on commit e30986e

Please sign in to comment.