fix: async audio only tests (videojs#7673)

alex-barstow · web-flow · commit ebb2d09d17ee · 2022-03-11T12:24:50.000-05:00
diff --git a/test/unit/player.test.js b/test/unit/player.test.js
@@ -2809,11 +2809,13 @@ QUnit.test('playbackRates only accepts arrays of numbers', function(assert) {
 QUnit.test('audioOnlyMode can be set by option', function(assert) {
   assert.expect(4);
 
+  const done = assert.async();
   const player = TestHelpers.makePlayer({audioOnlyMode: true});
 
   player.one('audioonlymodechange', () => {
     assert.equal(player.audioOnlyMode(), true, 'asynchronously set via option');
     assert.equal(player.hasClass('vjs-audio-only-mode'), true, 'class added asynchronously');
+    done();
   });
 
   assert.equal(player.audioOnlyMode(), false, 'defaults to false');
@@ -2827,21 +2829,19 @@ QUnit.test('audioOnlyMode(true) returns Promise when promises are supported', fu
   if (window.Promise) {
     assert.ok(returnValTrue instanceof window.Promise, 'audioOnlyMode(true) returns Promise when supported');
   }
-
-  return returnValTrue;
 });
 
 QUnit.test('audioOnlyMode(false) returns Promise when promises are supported', function(assert) {
+  const done = assert.async();
   const player = TestHelpers.makePlayer({audioOnlyMode: true});
 
   player.one('audioonlymodechange', () => {
     const returnValFalse = player.audioOnlyMode(false);
 
     if (window.Promise) {
       assert.ok(returnValFalse instanceof window.Promise, 'audioOnlyMode(false) returns Promise when supported');
+      done();
     }
-
-    return returnValFalse;
   });
 });
 
