From f730c0548e65ffe2437cb7e1f30a8d3ded604d5f Mon Sep 17 00:00:00 2001 From: AshAnand34 Date: Fri, 11 Jul 2025 15:15:39 -0700 Subject: [PATCH] Renaming service function to mergeVideos --- src/pages/tools/video/merge-video/index.tsx | 4 ++-- .../video/merge-video/merge-video.service.test.ts | 10 +++++----- src/pages/tools/video/merge-video/service.ts | 2 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/pages/tools/video/merge-video/index.tsx b/src/pages/tools/video/merge-video/index.tsx index fb35c0a..3866b32 100644 --- a/src/pages/tools/video/merge-video/index.tsx +++ b/src/pages/tools/video/merge-video/index.tsx @@ -6,7 +6,7 @@ import ToolFileResult from '@components/result/ToolFileResult'; import ToolMultipleVideoInput, { MultiVideoInput } from '@components/input/ToolMultipleVideoInput'; -import { main } from './service'; +import { mergeVideos } from './service'; import { InitialValuesType } from './types'; const initialValues: InitialValuesType = {}; @@ -37,7 +37,7 @@ export default function MergeVideo({ 'Files to merge:', files.map((f) => f.name) ); - const mergedBlob = await main(files, initialValues); + const mergedBlob = await mergeVideos(files, initialValues); const mergedFile = new File([mergedBlob], 'merged-video.mp4', { type: 'video/mp4' }); diff --git a/src/pages/tools/video/merge-video/merge-video.service.test.ts b/src/pages/tools/video/merge-video/merge-video.service.test.ts index ee5a8b2..ccc81e6 100644 --- a/src/pages/tools/video/merge-video/merge-video.service.test.ts +++ b/src/pages/tools/video/merge-video/merge-video.service.test.ts @@ -17,7 +17,7 @@ vi.mock('@ffmpeg/util', () => ({ })); // Import after mocking -import { main } from './service'; +import { mergeVideos } from './service'; function createMockFile(name: string, type = 'video/mp4') { return new File([new Uint8Array([0, 1, 2])], name, { type }); @@ -25,17 +25,17 @@ function createMockFile(name: string, type = 'video/mp4') { describe('merge-video', () => { it('throws if less than two files are provided', async () => { - await expect(main([], {})).rejects.toThrow( + await expect(mergeVideos([], {})).rejects.toThrow( 'Please provide at least two video files to merge.' ); - await expect(main([createMockFile('a.mp4')], {})).rejects.toThrow( + await expect(mergeVideos([createMockFile('a.mp4')], {})).rejects.toThrow( 'Please provide at least two video files to merge.' ); }); it('throws if input is not an array', async () => { // @ts-ignore - testing invalid input - await expect(main(null, {})).rejects.toThrow( + await expect(mergeVideos(null, {})).rejects.toThrow( 'Please provide at least two video files to merge.' ); }); @@ -44,7 +44,7 @@ describe('merge-video', () => { const mockFile1 = createMockFile('video1.mp4'); const mockFile2 = createMockFile('video2.mp4'); - const result = await main([mockFile1, mockFile2], {}); + const result = await mergeVideos([mockFile1, mockFile2], {}); expect(result).toBeInstanceOf(Blob); expect(result.type).toBe('video/mp4'); diff --git a/src/pages/tools/video/merge-video/service.ts b/src/pages/tools/video/merge-video/service.ts index 8a374d5..c376681 100644 --- a/src/pages/tools/video/merge-video/service.ts +++ b/src/pages/tools/video/merge-video/service.ts @@ -4,7 +4,7 @@ import { fetchFile } from '@ffmpeg/util'; // This function will use ffmpeg.wasm to merge multiple video files in the browser. // Returns a Promise that resolves to a Blob of the merged video. -export async function main( +export async function mergeVideos( input: MergeVideoInput, options: InitialValuesType ): Promise {