do not override Dom methods

brandonocasey · brandonocasey · commit 9a0b6fcf6658 · 2019-01-08T12:50:17.000-05:00
diff --git a/test/unit/video.test.js b/test/unit/video.test.js
@@ -227,13 +227,6 @@ QUnit.test('should expose options and players properties for backward-compatibil
 });
 
 QUnit.test('should expose DOM functions', function(assert) {
-  const origWarnLog = log.warn;
-  const warnLogs = [];
-
-  log.warn = (args) => {
-    warnLogs.push(args);
-  };
-
   const methods = [
     'isEl',
     'isTextNode',
@@ -252,24 +245,7 @@ QUnit.test('should expose DOM functions', function(assert) {
   methods.forEach(name => {
     assert.strictEqual(typeof videojs[name], 'function', `function videojs.${name}`);
     assert.strictEqual(typeof Dom[name], 'function', `Dom.${name} function exists`);
-
-    const oldMethod = Dom[name];
-    let domCalls = 0;
-
-    Dom[name] = () => domCalls++;
-
-    videojs[name]();
-
-    assert.equal(domCalls, 1, `Dom.${name} was called when videojs.${name} is run.`);
-    assert.equal(warnLogs.length, 1, `videojs.${name} logs a deprecation warning`);
-
-    // reset
-    warnLogs.length = 0;
-    Dom[name] = oldMethod;
   });
-
-  // reset log
-  log.warn = origWarnLog;
 });
 
 QUnit.test('ingest player div if data-vjs-player attribute is present on video parentNode', function(assert) {
