Merge pull request #7245 from OmarShehata/bad-specs

Fix broken VideoSynchronizer Specs

Author: mramato

Specs/Core/VideoSynchronizerSpec.js

@@ -160,28 +160,35 @@ defineSuite([
         var epoch = JulianDate.fromIso8601('2015-11-01T00:00:00Z');
         var clock = new Clock();
 
-        var element = loadVideo();
+        // Since Chrome doesn't allow video playback without user
+        // interaction, we use a mock element.
+        var element = jasmine.createSpyObj('MockVideoElement', ['addEventListener', 'removeEventListener', 'play', 'pause']);
+        element.paused = false;
+        element.play.and.callFake(function() {
+            this.paused = false;
+        });
+        element.pause.and.callFake(function() {
+            this.paused = true;
+        });
 
         var videoSynchronizer = new VideoSynchronizer({
             clock : clock,
             element : element,
             epoch : epoch
         });
 
-        return pollToPromise(function() {
-            clock.shouldAnimate = false;
-            clock.tick();
-            return element.paused === true;
-        }).then(function() {
-            clock.shouldAnimate = true;
-            clock.tick();
-            return element.paused === false;
-        }).then(function() {
-            clock.shouldAnimate = false;
-            clock.tick();
-            return element.paused === true;
-        }).then(function() {
-            videoSynchronizer.destroy();
-        });
+        clock.shouldAnimate = false;
+        clock.tick();
+        expect(element.pause.calls.count()).toEqual(1);
+
+        clock.shouldAnimate = true;
+        clock.tick();
+        expect(element.play.calls.count()).toEqual(1);
+
+        clock.shouldAnimate = false;
+        clock.tick();
+        expect(element.pause.calls.count()).toEqual(2);
+
+        videoSynchronizer.destroy();
     });
 });