rx-player 3.33.0 → 3.33.1

package/src/parsers/manifest/utils/__tests__/update_segment_timeline.test.ts

@@ -1,546 +0,0 @@
-/**
- * Copyright 2015 CANAL+ Group
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
-/* eslint-disable @typescript-eslint/no-var-requires */
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
-/* eslint-disable @typescript-eslint/no-unsafe-return */
-
-import { IIndexSegment } from "../index_helpers";
-
-describe("Manifest Parsers utils - updateSegmentTimeline", () => {
-  let mockLogWarn : jest.MockInstance<void, unknown[]> | undefined;
-  let updateSegmentTimeline : ((a : IIndexSegment[],
-                                b : IIndexSegment[]) => boolean) | undefined;
-  beforeEach(() => {
-    jest.resetModules();
-
-    mockLogWarn = jest.spyOn(jest.requireActual("../../../../log").default, "warn");
-    updateSegmentTimeline = jest.requireActual("../update_segment_timeline").default;
-  });
-
-  afterEach(() => {
-    mockLogWarn?.mockRestore();
-  });
-
-  /* eslint-disable max-len */
-  it("should just replace with the new timeline if the old was empty", () => {
-  /* eslint-enable max-len */
-    const oldTimeline : IIndexSegment[] = [];
-    const newTimeline1 = [{ start: 0, duration: 1000, repeatCount: 10 }];
-    const newTimeline2 : IIndexSegment[] = [];
-    expect(updateSegmentTimeline?.(oldTimeline, newTimeline1)).toEqual(true);
-    expect(oldTimeline).toEqual(newTimeline1);
-
-    oldTimeline.length = 0; // reset
-    updateSegmentTimeline?.(oldTimeline, newTimeline2);
-    expect(oldTimeline).toEqual(newTimeline2);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  it("should not do anything if the new timeline is empty", () => {
-    const oldTimeline = [{ start: 0, duration: 1000, repeatCount: 10 }];
-    const newTimeline : IIndexSegment[] = [];
-    const oldTimelineCloned = oldTimeline.slice();
-    expect(updateSegmentTimeline?.(oldTimeline, newTimeline)).toEqual(false);
-    expect(oldTimeline).toEqual(oldTimelineCloned);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  it("should not do anything if the timelines are equal", () => {
-    const oldTimeline1 = [{ start: 0, duration: 1000, repeatCount: 10 }];
-    const newTimeline1 = [{ start: 0, duration: 1000, repeatCount: 10 }];
-    const oldTimeline1Cloned = oldTimeline1.slice();
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual(oldTimeline1Cloned);
-
-    const oldTimeline2 = [ { start: 0, duration: 1000, repeatCount: 10 },
-                           { start: 11000, duration: 1000, repeatCount: 0 },
-                           { start: 12000, duration: 1000, repeatCount: 1 } ];
-    const newTimeline2 = [ { start: 0, duration: 1000, repeatCount: 10 },
-                           { start: 11000, duration: 1000, repeatCount: 0 },
-                           { start: 12000, duration: 1000, repeatCount: 1 } ];
-    const oldTimeline2Cloned = oldTimeline2.slice();
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual(oldTimeline2Cloned);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  /* eslint-disable max-len */
-  it("should throw if the new timeline begin long after the old one", () => {
-  /* eslint-enable max-len */
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 10 },
-                           { start: 11000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 13000, duration: 1000, repeatCount: 10 },
-                           { start: 24000, duration: 1000, repeatCount: 0 },
-                           { start: 25000, duration: 1000, repeatCount: 1 } ];
-    const oldTimeline1Cloned = oldTimeline1.slice();
-
-    let err = null;
-    try {
-      expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).not.toHaveReturned();
-    } catch (e) {
-      err = e;
-    }
-
-    expect(err).toBeInstanceOf(Error);
-
-    // Impossible check to shut-up TypeScript
-    if (!(err instanceof Error)) {
-      throw new Error("Impossible: already checked it was an Error instance");
-    }
-    expect((err as { type? : string }).type).toEqual("MEDIA_ERROR");
-    expect((err as { code? : string }).code).toEqual("MANIFEST_UPDATE_ERROR");
-    expect(err.message)
-      .toEqual("MediaError (MANIFEST_UPDATE_ERROR) Cannot perform " +
-               "partial update: not enough data");
-    expect(oldTimeline1).toEqual(oldTimeline1Cloned);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  it("should merge consecutive timelines", () => {
-    const oldTimeline1 = [ { start: 0, duration: 500, repeatCount: 20 },
-                           { start: 11000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 12000, duration: 1000, repeatCount: 11 },
-                           { start: 24000, duration: 1100, repeatCount: 0 },
-                           { start: 25100, duration: 1000, repeatCount: 1 } ];
-    const oldTimeline1Cloned = oldTimeline1.slice();
-    const newTimeline1Cloned = newTimeline1.slice();
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([...oldTimeline1Cloned, ...newTimeline1Cloned]);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-
-    // With repeats
-
-    const oldTimeline2 = [ { start: 0, duration: 500, repeatCount: 20 },
-                           { start: 11000, duration: 1000, repeatCount: 8 } ];
-    const newTimeline2 = [ { start: 20000, duration: 1000, repeatCount: 3 },
-                           { start: 24000, duration: 1100, repeatCount: 0 },
-                           { start: 25100, duration: 1000, repeatCount: 1 } ];
-    const oldTimeline2Cloned = oldTimeline2.slice();
-    const newTimeline2Cloned = newTimeline2.slice();
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([...oldTimeline2Cloned, ...newTimeline2Cloned]);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  /* eslint-disable max-len */
-  it("should merge consecutive timelines with a repeatCount set to -1 for the old timeline", () => {
-  /* eslint-enable max-len */
-    const oldTimeline1 = [ { start: 0, duration: 500, repeatCount: 20 },
-                           { start: 11000, duration: 1000, repeatCount: -1 } ];
-    const newTimeline1 = [ { start: 12000, duration: 1000, repeatCount: 11 },
-                           { start: 24000, duration: 1100, repeatCount: 0 },
-                           { start: 25100, duration: 1000, repeatCount: 1 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 500, repeatCount: 20 },
-      { start: 11000, duration: 1000, repeatCount: 0 },
-      { start: 12000, duration: 1000, repeatCount: 11 },
-      { start: 24000, duration: 1100, repeatCount: 0 },
-      { start: 25100, duration: 1000, repeatCount: 1 },
-    ]);
-
-    const oldTimeline2 = [ { start: 0, duration: 500, repeatCount: 20 },
-                           { start: 11000, duration: 1000, repeatCount: -1 } ];
-    const newTimeline2 = [ { start: 20000, duration: 1000, repeatCount: 3 },
-                           { start: 24000, duration: 1100, repeatCount: 0 },
-                           { start: 25100, duration: 1000, repeatCount: 1 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([
-      { start: 0, duration: 500, repeatCount: 20 },
-      { start: 11000, duration: 1000, repeatCount: 8 },
-      { start: 20000, duration: 1000, repeatCount: 3 },
-      { start: 24000, duration: 1100, repeatCount: 0 },
-      { start: 25100, duration: 1000, repeatCount: 1 },
-    ]);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  it("should merge perfectly overlapping timelines without repeatCounts", () => {
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 0 },
-                           { start: 1500, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 1000, duration: 500, repeatCount: 0 },
-                           { start: 1500, duration: 1000, repeatCount: 0 },
-                           { start: 2500, duration: 500, repeatCount: 0 },
-                           { start: 3000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 0 },
-      { start: 1500, duration: 1000, repeatCount: 0 },
-      { start: 2500, duration: 500, repeatCount: 0 },
-      { start: 3000, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  it("should merge perfectly overlapping timelines with repeatCounts", () => {
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 2000, duration: 500, repeatCount: 19 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 21 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-
-    const oldTimeline2 = [ { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline2 = [ { start: 2000, duration: 500, repeatCount: 19 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([
-      { start: 1000, duration: 500, repeatCount: 21 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-
-    const oldTimeline3 = [ { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline3 = [ { start: 2000, duration: 500, repeatCount: -1 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline3, newTimeline3)).toEqual(false);
-    expect(oldTimeline3).toEqual([
-      { start: 1000, duration: 500, repeatCount: -1 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  it("should merge even if there are \"holes\" in the old timeline", () => {
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 0 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 2000, duration: 500, repeatCount: 19 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 0 },
-      { start: 2000, duration: 500, repeatCount: 19 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-
-    const oldTimeline2 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 3 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline2 = [ { start: 4000, duration: 500, repeatCount: 15 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 3 },
-      { start: 4000, duration: 500, repeatCount: 15 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  /* eslint-disable max-len */
-  it("should handle cases where the new timeline's start cannot be reached with the old timeline's repeatCount ", () => {
-  /* eslint-enable max-len */
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 2700, duration: 500, repeatCount: 19 },
-                           { start: 12700, duration: 1000, repeatCount: 0 },
-                           { start: 13700, duration: 7000, repeatCount: 0 },
-                           { start: 20700, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 2 },
-      { start: 2700, duration: 500, repeatCount: 19 },
-      { start: 12700, duration: 1000, repeatCount: 0 },
-      { start: 13700, duration: 7000, repeatCount: 0 },
-      { start: 20700, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn).toHaveBeenCalledWith(
-      "RepresentationIndex: Manifest update removed previous segments"
-    );
-    mockLogWarn?.mockClear();
-
-    const oldTimeline2 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 } ];
-    const newTimeline2 = [ { start: 2700, duration: 500, repeatCount: 19 },
-                           { start: 12700, duration: 1000, repeatCount: 0 },
-                           { start: 13700, duration: 7000, repeatCount: 0 },
-                           { start: 20700, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 2 },
-      { start: 2700, duration: 500, repeatCount: 19 },
-      { start: 12700, duration: 1000, repeatCount: 0 },
-      { start: 13700, duration: 7000, repeatCount: 0 },
-      { start: 20700, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn).toHaveBeenCalledWith(
-      "RepresentationIndex: Manifest update removed previous segments"
-    );
-    mockLogWarn?.mockClear();
-
-    const oldTimeline3 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 } ];
-    const newTimeline3 = [ { start: 2700, duration: 500, repeatCount: 19 } ];
-    expect(updateSegmentTimeline?.(oldTimeline3, newTimeline3)).toEqual(false);
-    expect(oldTimeline3).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 2 },
-      { start: 2700, duration: 500, repeatCount: 19 },
-    ]);
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn).toHaveBeenCalledWith(
-      "RepresentationIndex: Manifest update removed previous segments"
-    );
-  });
-
-  /* eslint-disable max-len */
-  it("should handle cases where the repeatCount cannot be incremented due to different durations", () => {
-  /* eslint-enable max-len */
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 2000, duration: 1000, repeatCount: 9 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 1 },
-      { start: 2000, duration: 1000, repeatCount: 9 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn).toHaveBeenCalledWith(
-      "RepresentationIndex: Manifest update removed previous segments"
-    );
-    mockLogWarn?.mockClear();
-
-    const oldTimeline2 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 } ];
-    const newTimeline2 = [ { start: 2000, duration: 1000, repeatCount: 9 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 1 },
-      { start: 2000, duration: 1000, repeatCount: 9 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn).toHaveBeenCalledWith(
-      "RepresentationIndex: Manifest update removed previous segments"
-    );
-    mockLogWarn?.mockClear();
-
-    const oldTimeline3 = [ { start: 1000, duration: 500, repeatCount: 21 } ];
-    const newTimeline3 = [ { start: 2000, duration: 1000, repeatCount: 9 } ];
-    expect(updateSegmentTimeline?.(oldTimeline3, newTimeline3)).toEqual(false);
-    expect(oldTimeline3).toEqual([
-      { start: 1000, duration: 500, repeatCount: 1 },
-      { start: 2000, duration: 1000, repeatCount: 9 },
-    ]);
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn).toHaveBeenCalledWith(
-      "RepresentationIndex: Manifest update removed previous segments"
-    );
-  });
-
-  it("should handle and log when there is a direct overlap (not due to a repeat)", () => {
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 0 },
-                           { start: 1500, duration: 1500, repeatCount: 0 },
-                           { start: 3000, duration: 9000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 2000, duration: 1000, repeatCount: 0 },
-                           { start: 3000, duration: 9000, repeatCount: 0 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(true);
-    expect(oldTimeline1).toEqual([
-      { start: 2000, duration: 1000, repeatCount: 0 },
-      { start: 3000, duration: 9000, repeatCount: 0 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn).toHaveBeenCalledWith(
-      "RepresentationIndex: Manifest update removed all previous segments"
-    );
-  });
-
-  /* eslint-disable max-len */
-  it("should handle the case where the new Timeline just increment the last repeatCount", () => {
-  /* eslint-enable max-len */
-    const oldTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 } ];
-    const newTimeline1 = [ { start: 1000, duration: 500, repeatCount: 51 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 51 },
-    ]);
-
-    const oldTimeline2 = [ { start: 1000, duration: 500, repeatCount: 64 } ];
-    const newTimeline2 = [ { start: 1000, duration: 500, repeatCount: 72 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([ { start: 1000, duration: 500, repeatCount: 72 } ]);
-
-    const oldTimeline3 = [ { start: 1000, duration: 500, repeatCount: 64 } ];
-    const newTimeline3 = [ { start: 1000, duration: 500, repeatCount: 72 },
-                           { start: 37500, duration: 1000, repeatCount: 5 }];
-    expect(updateSegmentTimeline?.(oldTimeline3, newTimeline3)).toEqual(false);
-    expect(oldTimeline3).toEqual([
-      { start: 1000, duration: 500, repeatCount: 72 },
-      { start: 37500, duration: 1000, repeatCount: 5 },
-    ]);
-
-    expect(mockLogWarn).not.toHaveBeenCalled();
-  });
-
-  it("should handle when the newer timeline has more depth than the older one", () => {
-    const oldTimeline1 = [ { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(true);
-    expect(oldTimeline1).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 21 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn)
-      .toHaveBeenCalledWith(
-        "RepresentationIndex: The new index is \"bigger\" than the previous one"
-      );
-
-    mockLogWarn?.mockClear();
-
-    const oldTimeline2 = [ { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: -1 } ];
-    const newTimeline2 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(true);
-    expect(oldTimeline2).toEqual([
-      { start: 0, duration: 1000, repeatCount: 0 },
-      { start: 1000, duration: 500, repeatCount: 21 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn)
-      .toHaveBeenCalledWith(
-        "RepresentationIndex: The new index is \"bigger\" than the previous one"
-      );
-  });
-
-  it("should handle when the newer timeline is actually older than the older one", () => {
-    const oldTimeline1 = [ { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: 0 } ];
-    const newTimeline1 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 } ];
-    expect(updateSegmentTimeline?.(oldTimeline1, newTimeline1)).toEqual(false);
-    expect(oldTimeline1).toEqual([
-      { start: 1000, duration: 500, repeatCount: 21 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: 0 },
-    ]);
-
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn)
-      .toHaveBeenCalledWith(
-        "RepresentationIndex: The new index is older than the previous one"
-      );
-
-    mockLogWarn?.mockClear();
-
-    const oldTimeline2 = [ { start: 1000, duration: 500, repeatCount: 21 },
-                           { start: 12000, duration: 1000, repeatCount: 0 },
-                           { start: 13000, duration: 7000, repeatCount: 0 },
-                           { start: 20000, duration: 5000, repeatCount: -1 } ];
-    const newTimeline2 = [ { start: 0, duration: 1000, repeatCount: 0 },
-                           { start: 1000, duration: 500, repeatCount: 21 } ];
-    expect(updateSegmentTimeline?.(oldTimeline2, newTimeline2)).toEqual(false);
-    expect(oldTimeline2).toEqual([
-      { start: 1000, duration: 500, repeatCount: 21 },
-      { start: 12000, duration: 1000, repeatCount: 0 },
-      { start: 13000, duration: 7000, repeatCount: 0 },
-      { start: 20000, duration: 5000, repeatCount: -1 },
-    ]);
-
-    expect(mockLogWarn).toHaveBeenCalledTimes(1);
-    expect(mockLogWarn)
-      .toHaveBeenCalledWith(
-        "RepresentationIndex: The new index is older than the previous one"
-      );
-
-    mockLogWarn?.mockClear();
-  });
-});

package/src/parsers/manifest/utils/check_manifest_ids.ts

@@ -1,83 +0,0 @@
-/**
- * Copyright 2015 CANAL+ Group
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-import log from "../../../log";
-import arrayIncludes from "../../../utils/array_includes";
-import {
-  IParsedAdaptationType,
-  IParsedManifest,
-} from "../types";
-
-/**
- * Ensure that no two periods, adaptations from the same period and
- * representations from the same adaptation, have the same ID.
- *
- * Log and mutate their ID if not until this is verified.
- *
- * @param {Object} manifest
- */
-export default function checkManifestIDs(
-  manifest : IParsedManifest
-) : void {
-  const periodIDS : string[] = [];
-  manifest.periods.forEach((period) => {
-    const periodID = period.id;
-    if (arrayIncludes(periodIDS, periodID)) {
-      log.warn("Two periods with the same ID found. Updating.");
-      const newID = periodID + "-dup";
-      period.id = newID;
-      checkManifestIDs(manifest);
-      periodIDS.push(newID);
-    } else {
-      periodIDS.push(periodID);
-    }
-    const { adaptations } = period;
-    const adaptationIDs : string[] = [];
-    (Object.keys(adaptations) as IParsedAdaptationType[]).forEach((type) => {
-      const adaptationsForType = adaptations[type];
-      if (adaptationsForType === undefined) {
-        return;
-      }
-      adaptationsForType.forEach(adaptation => {
-        const adaptationID = adaptation.id;
-        if (arrayIncludes(adaptationIDs, adaptationID)) {
-          log.warn("Two adaptations with the same ID found. Updating.",
-                   adaptationID);
-          const newID =  adaptationID + "-dup";
-          adaptation.id = newID;
-          checkManifestIDs(manifest);
-          adaptationIDs.push(newID);
-        } else {
-          adaptationIDs.push(adaptationID);
-        }
-        const representationIDs : Array<number|string> = [];
-        adaptation.representations.forEach(representation => {
-          const representationID = representation.id;
-          if (arrayIncludes(representationIDs, representationID)) {
-            log.warn("Two representations with the same ID found. Updating.",
-                     representationID);
-            const newID = `${representationID}-dup`;
-            representation.id = newID;
-            checkManifestIDs(manifest);
-            representationIDs.push(newID);
-          } else {
-            representationIDs.push(representationID);
-          }
-        });
-      });
-    });
-  });
-}