test: fix tests on older IE (#3800)

src/js/component.js

@@ -1443,7 +1443,13 @@ class Component {
     if (name === 'Player' && Component.components_[name]) {
       const Player = Component.components_[name];
 
-      if (Player.players && Object.keys(Player.players).length > 0) {
+      // If we have players that were disposed, then their name will still be
+      // in Players.players. So, we must loop through and verify that the value
+      // for each item is not null. This allows registration of the Player component
+      // after all players have been disposed or before any were created.
+      if (Player.players &&
+          Object.keys(Player.players).length > 0 &&
+          Object.keys(Player.players).map((playerName) => Player.players[playerName]).every(Boolean)) {
         throw new Error('Can not register Player component after player has been created');
       }
     }

test/unit/player.test.js

@@ -144,6 +144,8 @@ QUnit.test('should get current source from source tag', function(assert) {
 
   assert.ok(player.currentSource().src === 'http://google.com');
   assert.ok(player.currentSource().type === 'video/mp4');
+
+  player.dispose();
 });
 
 QUnit.test('should get current sources from source tag', function(assert) {
@@ -172,6 +174,8 @@ QUnit.test('should get current sources from source tag', function(assert) {
   assert.ok(player.currentSources()[0].src === 'http://google.com');
   assert.ok(player.currentSources()[0].type === undefined);
   assert.ok(player.currentSources()[1] === undefined);
+
+  player.dispose();
 });
 
 QUnit.test('should get current source from src set', function(assert) {
@@ -208,6 +212,7 @@ QUnit.test('should get current source from src set', function(assert) {
 
   assert.ok(player.currentSource().src === 'http://google.com');
   assert.ok(player.currentSource().type === 'video/mp4');
+  player.dispose();
 });
 
 QUnit.test('should get current sources from src set', function(assert) {
@@ -255,6 +260,8 @@ QUnit.test('should get current sources from src set', function(assert) {
   assert.ok(player.currentSources()[0].src === 'http://hugo.com');
   assert.ok(player.currentSources()[0].type === undefined);
   assert.ok(player.currentSources()[1] === undefined);
+
+  player.dispose();
 });
 
 QUnit.test('should asynchronously fire error events during source selection', function(assert) {
@@ -334,6 +341,8 @@ QUnit.test('should default to 16:9 when fluid', function(assert) {
 
   // IE8 rounds 0.5625 up to 0.563
   assert.ok(((ratio >= 0.562) && (ratio <= 0.563)), 'fluid player without dimensions defaults to 16:9');
+
+  player.dispose();
 });
 
 QUnit.test('should set fluid to true if element has vjs-fluid class', function(assert) {
@@ -344,6 +353,8 @@ QUnit.test('should set fluid to true if element has vjs-fluid class', function(a
   const player = TestHelpers.makePlayer({}, tag);
 
   assert.ok(player.fluid(), 'fluid is true with vjs-fluid class');
+
+  player.dispose();
 });
 
 QUnit.test('should use an class name that begins with an alpha character', function(assert) {
@@ -1269,19 +1280,23 @@ QUnit.test('should allow to register custom player when any player has not been
 
   assert.equal(player instanceof CustomPlayer, true, 'player is custom');
   player.dispose();
+
+  // reset the Player to the original value;
+  videojs.registerComponent('Player', Player);
 });
 
 QUnit.test('should not allow to register custom player when any player has been created', function(assert) {
   const tag = TestHelpers.makeTag();
   const player = videojs(tag);
 
   class CustomPlayer extends Player {}
-  try {
+
+  assert.throws(function() {
     videojs.registerComponent('Player', CustomPlayer);
-  } catch (e) {
-    player.dispose();
-    return assert.equal(e.message, 'Can not register Player component after player has been created');
-  }
+  }, 'Can not register Player component after player has been created');
+
+  player.dispose();
 
-  assert.ok(false, 'It should throw Error when any player has been created');
+  // reset the Player to the original value;
+  videojs.registerComponent('Player', Player);
 });

test/unit/test-helpers.js

@@ -139,22 +139,22 @@ const TestHelpers = {
    * @param  {string} eventType
    */
   triggerDomEvent(element, eventType) {
-    let event;
-
-    if (document.createEvent) {
-      event = document.createEvent('HTMLEvents');
-      event.initEvent(eventType, true, true);
-    } else {
-      event = document.createEventObject();
-      event.eventType = eventType;
-    }
-
-    event.eventName = eventType;
-
-    if (document.createEvent) {
-      element.dispatchEvent(event);
-    } else {
-      element.fireEvent('on' + event.eventType, event);
+    let event;
+
+    if (document.createEvent) {
+      event = document.createEvent('HTMLEvents');
+      event.initEvent(eventType, true, true);
+    } else {
+      event = document.createEventObject();
+      event.eventType = eventType;
+    }
+
+    event.eventName = eventType;
+
+    if (document.createEvent) {
+      element.dispatchEvent(event);
+    } else {
+      element.fireEvent('on' + event.eventType, event);
     }
   }
 };

test/unit/tracks/text-track-controls.test.js

@@ -408,7 +408,8 @@ test('chapters menu should use track label as menu title', function() {
   player.controlBar.chaptersButton.update();
 
   const menu = player.controlBar.chaptersButton.menu;
-  const menuTitle = menu.contentEl().firstChild.textContent;
+  const titleEl = menu.contentEl().firstChild;
+  const menuTitle = titleEl.textContent || titleEl.innerText;
 
   equal(menuTitle, 'Test Chapters', 'menu gets track label as title');
 
@@ -435,7 +436,15 @@ test('chapters should be displayed when remote track added and load event fired'
 
   equal(chaptersEl.track.cues.length, 2);
 
-  TestHelpers.triggerDomEvent(chaptersEl, 'load');
+  // Anywhere where we support using native text tracks, we can trigger a custom DOM event.
+  // On IE8 and other places where we have emulated tracks, either we cannot trigger custom
+  // DOM events (like IE8 with the custom DOM element) or we aren't using a DOM element at all.
+  // In those cases just trigger `load` directly on the chaptersEl object.
+  if (player.tech_.featuresNativeTextTracks) {
+    TestHelpers.triggerDomEvent(chaptersEl, 'load');
+  } else {
+    chaptersEl.trigger('load');
+  }
 
   ok(!player.controlBar.chaptersButton.hasClass('vjs-hidden'), 'chapters menu is displayed');