added more unittests

Author: Matze99

playground/main.ts

@@ -46,12 +46,7 @@ console.log("duration", audioTest.duration.millis);
 
 await audioTrack.add(audioTest);
 await audioTrack.removeSilences();
-
-console.log("clips", audioTrack.clips);
-console.log("duration", audioTrack.clips.at(0)?.start);
-console.log("duration", audioTrack.clips.at(0)?.stop);
-console.log("duration", audioTrack.clips.at(1)?.start);
-console.log("duration", audioTrack.clips.at(1)?.stop);
+audioTrack.stacked(true);
 
 image.animate()
   .rotation(-16).to(14, 5).to(-7, 10).to(24, 7).to(-3, 9).to(19, 7).to(-14, 12).to(5, 9).to(-30, 13)

src/tracks/media/media.spec.ts

@@ -37,58 +37,32 @@ describe('The Media Track Object', () => {
 	});
 
 	it('ignores no silences', async () => {
-		const clip = new MediaClip();
+		const clip = new MockMediaClip(30000, [new Timestamp(10000), new Timestamp(20000)], [], new Audio());
 		clip.duration.frames = 30;
 		await track.add(clip);
 		await track.removeSilences();
 		expect(track.clips.length).toBe(1);
 	});
 
-	// it('ignores not applicable silences', async () => {
-	// 	const clip = new MockMediaClip(30000, [new Timestamp(10000), new Timestamp(20000)], [
-	// 		{
-	// 			start: new Timestamp(0),
-	// 			stop: new Timestamp(500),
-	// 		},
-	// 		{
-	// 			start: new Timestamp(30000),
-	// 			stop: new Timestamp(30500),
-	// 		},
-	// 	], new Audio());
-	// 	await track.add(clip);
-	// 	expect(clip.source).toBeDefined();
-	// 	await track.removeSilences();
-	// 	expect(track.clips.length).toBe(1);
-	// 	expect(track.clips.at(0)).toBe(clip);
-	// });
-
-	it('removes silences', async () => {
+	it('ignores not applicable silences', async () => {
 		const clip = new MockMediaClip(30000, [new Timestamp(10000), new Timestamp(20000)], [
 			{
 				start: new Timestamp(0),
-				stop: new Timestamp(10050),
+				stop: new Timestamp(500),
 			},
 			{
-				start: new Timestamp(11000),
-				stop: new Timestamp(15000),
-			},
-			{
-				start: new Timestamp(19000),
+				start: new Timestamp(30000),
 				stop: new Timestamp(30500),
 			},
 		], new Audio());
 		await track.add(clip);
 		expect(clip.source).toBeDefined();
 		await track.removeSilences();
-		expect(track.clips.length).toBe(2);
-		expect(track.clips.at(0)?.range[0].millis).toBe(10051);
-		expect(track.clips.at(0)?.range[1].millis).toBe(11000);
-		expect(track.clips.at(1)?.range[0].millis).toBe(15001);
-		expect(track.clips.at(1)?.range[1].millis).toBe(19000);
+		expect(track.clips.length).toBe(1);
+		expect(track.clips.at(0)).toBe(clip);
 	});
 
-	it('removes silences stacked', async () => {
-		track.stacked(true);
+	it('removes silences', async () => {
 		const clip = new MockMediaClip(30000, [new Timestamp(10000), new Timestamp(20000)], [
 			{
 				start: new Timestamp(0),
@@ -109,8 +83,8 @@ describe('The Media Track Object', () => {
 		expect(track.clips.length).toBe(2);
 		expect(track.clips.at(0)?.range[0].millis).toBe(10051);
 		expect(track.clips.at(0)?.range[1].millis).toBe(11000);
-		expect(track.clips.at(1)?.range[0].millis).toBe(11001);
-		expect(track.clips.at(1)?.range[1].millis).toBe(15000);
+		expect(track.clips.at(1)?.range[0].millis).toBe(15001);
+		expect(track.clips.at(1)?.range[1].millis).toBe(19000);
 	});
 
 	it('should propagate a seek call', async () => {

src/tracks/media/media.ts

@@ -27,9 +27,6 @@ export class MediaTrack<Clip extends MediaClip> extends Track<MediaClip> {
 	 * @returns Array of silence periods with start and stop times in seconds
 	 */
 	public async removeSilences() {
-        // if (!(this.strategy instanceof StackInsertStrategy)) {
-        //     throw new Error("Cannot remove silences from a non-stacked track");
-        // }
 
 		// Process each clip
 		for (const clip of this.clips) {