Fix test failing with #3063

video-dev · Oct 8, 2020 · a6943fc · a6943fc
1 parent c579fea
commit a6943fc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/controller/subtitle-stream-controller.js b/src/controller/subtitle-stream-controller.js
@@ -264,7 +264,7 @@ export class SubtitleStreamController extends BaseStreamController {
 
   onMediaSeeking () {
     if (this.fragCurrent) {
-      const currentTime = this.media ? this.media.currentTime : null;
+      const currentTime = this.media ? this.media.currentTime : 0;
       const tolerance = this.config.maxFragLookUpTolerance;
       const fragStartOffset = this.fragCurrent.start - tolerance;
       const fragEndOffset = this.fragCurrent.start + this.fragCurrent.duration + tolerance;

diff --git a/tests/unit/controller/subtitle-stream-controller.js b/tests/unit/controller/subtitle-stream-controller.js
@@ -130,7 +130,8 @@ describe('SubtitleStreamController', function () {
   });
 
   describe('onMediaSeeking', function () {
-    it('nulls fragPrevious', function () {
+    it('nulls fragPrevious when seeking away from fragCurrent', function () {
+      subtitleStreamController.fragCurrent = { start: 1000, duration: 10 };
       subtitleStreamController.fragPrevious = {};
       subtitleStreamController.onMediaSeeking();
       expect(subtitleStreamController.fragPrevious).to.not.exist;