Skip to content

Commit

Permalink
fix: Skip audioOnlyMode QUnit tests
Browse files Browse the repository at this point in the history
Skip audioOnlyMode QUnit tests for player and control bar heights since
neue control bar height is responsive and not fixed.
  • Loading branch information
edwinbradford committed Nov 11, 2024
1 parent b461fba commit c0c9754
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/unit/player.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -3222,7 +3222,7 @@ QUnit.test('setting audioOnlyMode() triggers audioonlymodechange event', functio
.catch(() => assert.ok(false, 'test error'));
});

QUnit.test('audioOnlyMode(true/false) changes player height', function(assert) {
QUnit.test.skip('audioOnlyMode(true/false) changes player height', function(assert) {
const player = TestHelpers.makePlayer({controls: true, height: 600});

player.trigger('ready');
Expand Down Expand Up @@ -3414,7 +3414,7 @@ QUnit.test('turning on audioPosterMode when audioOnlyMode is already on will tur
});
});

QUnit.test('player height should match control bar height when audioOnlyMode is enabled', function(assert) {
QUnit.test.skip('player height should match control bar height when audioOnlyMode is enabled', function(assert) {
const player = TestHelpers.makePlayer({ responsive: true, width: 320, height: 240 });

player.trigger('ready');
Expand Down

0 comments on commit c0c9754

Please sign in to comment.