Skip to content

Commit

Permalink
test: stop running placeholder el test in IE and Safari to prevent er…
Browse files Browse the repository at this point in the history
…rors (#7769)
  • Loading branch information
misteroneill committed May 20, 2022
1 parent 1908422 commit 50ffd57
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion test/unit/video.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -580,7 +580,15 @@ QUnit.test('adds video-js class name with the video-js embed', function(assert)
assert.ok(player2.hasClass('video-js'), 'video-js class was preserved to the second embed');
});

QUnit.test('stores placeholder el and restores on dispose', function(assert) {
let testOrSkip = 'test';

// The following test uses some DocumentFragment properties that are not
// available in IE or older Safaris, so we skip it.
if (videojs.browser.IE_VERSION || videojs.browser.IS_ANY_SAFARI) {
testOrSkip = 'skip';
}

QUnit[testOrSkip]('stores placeholder el and restores on dispose', function(assert) {
const fixture = document.getElementById('qunit-fixture');

const embeds = [
Expand Down

0 comments on commit 50ffd57

Please sign in to comment.