5728073738

pending completion

Build # 5728073738

Build Type

push

github

Committed by

web-flow

Commit Message

Merge pull request #93 from absidue/no-vjs-extend

refactor: Use ES6 classes instead of videojs.extend

Run Details

0 of 35 branches covered (0.0%)

Branch coverage included in aggregate %.

6 of 6 new or added lines in 2 files covered. (100.0%)

0 of 118 relevant lines covered (0.0%)

0.0 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/src/js/components/QualityOption.js

var _ = require('underscore'),
    events = require('../events');

module.exports = function(videojs) {
   var MenuItem = videojs.getComponent('MenuItem');

   /**
    * A MenuItem to represent a video resolution
    *
    * @class QualityOption
    * @extends videojs.MenuItem
    */
   return class QualityOption extends MenuItem {

      /**
       * @inheritdoc
       */
      constructor(player, options) {
         var source = options.source;

         if (!_.isObject(source)) {
            throw new Error('was not provided a "source" object, but rather: ' + (typeof source));
         }

         options = _.extend({
            selectable: true,
            label: source.label,
         }, options);

         super(player, options);

         this.source = source;
      }

      /**
       * @inheritdoc
       */
      handleClick(event) {
         super.handleClick(event);
         this.player().trigger(events.QUALITY_REQUESTED, this.source);
      }
   };
};

1	var _ = require('underscore'),	×
2	events = require('../events');	×
3
4	module.exports = function(videojs) {	×
5	var MenuItem = videojs.getComponent('MenuItem');	×
6
7	/**
8	* A MenuItem to represent a video resolution
9	*
10	* @class QualityOption
11	* @extends videojs.MenuItem
12	*/
13	return class QualityOption extends MenuItem {	×
14
15	/**
16	* @inheritdoc
17	*/
18	constructor(player, options) {
19	var source = options.source;	×
20
21	if (!_.isObject(source)) {	×
22	throw new Error('was not provided a "source" object, but rather: ' + (typeof source));	×
23	}
24
25	options = _.extend({	×
26	selectable: true,
27	label: source.label,
28	}, options);
29
30	super(player, options);	×
31
32	this.source = source;	×
33	}
34
35	/**
36	* @inheritdoc
37	*/
38	handleClick(event) {
39	super.handleClick(event);	×
40	this.player().trigger(events.QUALITY_REQUESTED, this.source);	×
41	}
42	};
43	};

silvermine / videojs-quality-selector / 5728073738

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous