diff --git a/frontend/src/hooks/tools/merge/useMergeParameters.test.ts b/frontend/src/hooks/tools/merge/useMergeParameters.test.ts index c2ef59178..8294cdf6e 100644 --- a/frontend/src/hooks/tools/merge/useMergeParameters.test.ts +++ b/frontend/src/hooks/tools/merge/useMergeParameters.test.ts @@ -1,12 +1,12 @@ import { describe, expect, test } from 'vitest'; import { renderHook, act } from '@testing-library/react'; -import { useMergeParameters, defaultMergeParameters } from './useMergeParameters'; +import { useMergeParameters, defaultParameters } from './useMergeParameters'; describe('useMergeParameters', () => { test('should initialize with default parameters', () => { const { result } = renderHook(() => useMergeParameters()); - expect(result.current.parameters).toStrictEqual(defaultMergeParameters); + expect(result.current.parameters).toStrictEqual(defaultParameters); }); test.each([ @@ -41,7 +41,7 @@ describe('useMergeParameters', () => { result.current.resetParameters(); }); - expect(result.current.parameters).toStrictEqual(defaultMergeParameters); + expect(result.current.parameters).toStrictEqual(defaultParameters); }); test('should validate parameters correctly - always returns true', () => { diff --git a/frontend/src/hooks/tools/merge/useMergeParameters.ts b/frontend/src/hooks/tools/merge/useMergeParameters.ts index c777f1489..2abc416ca 100644 --- a/frontend/src/hooks/tools/merge/useMergeParameters.ts +++ b/frontend/src/hooks/tools/merge/useMergeParameters.ts @@ -1,37 +1,21 @@ -import { useState, useCallback } from 'react'; +import { BaseParameters } from '../../../types/parameters'; +import { BaseParametersHook, useBaseParameters } from '../shared/useBaseParameters'; -export interface MergeParameters { +export interface MergeParameters extends BaseParameters { removeDigitalSignature: boolean; generateTableOfContents: boolean; -} +}; -export const defaultMergeParameters: MergeParameters = { +export const defaultParameters: MergeParameters = { removeDigitalSignature: false, generateTableOfContents: false, }; -export const useMergeParameters = () => { - const [parameters, setParameters] = useState(defaultMergeParameters); +export type MergeParametersHook = BaseParametersHook; - const updateParameter = useCallback(( - key: K, - value: MergeParameters[K] - ) => { - setParameters(prev => ({ ...prev, [key]: value })); - }, []); - - const validateParameters = useCallback((): boolean => { - return true; // Merge has no required parameters - }, []); - - const resetParameters = useCallback(() => { - setParameters(defaultMergeParameters); - }, []); - - return { - parameters, - updateParameter, - validateParameters, - resetParameters, - }; +export const useMergeParameters = (): MergeParametersHook => { + return useBaseParameters({ + defaultParameters, + endpointName: "merge-pdfs", + }); };