From 67981f6181b32d80b501b97f895f7f78634ad9e7 Mon Sep 17 00:00:00 2001 From: Jakob Scheid Date: Tue, 28 Jul 2026 12:20:17 +0200 Subject: [PATCH] test(settings): refactor getWrapper function in settings input test Updated the getWrapper function to use object destructuring. --- .../components/__tests__/SettingsInput.test.js | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/features/settings/components/__tests__/SettingsInput.test.js b/src/features/settings/components/__tests__/SettingsInput.test.js index c08f92d..808c5c2 100644 --- a/src/features/settings/components/__tests__/SettingsInput.test.js +++ b/src/features/settings/components/__tests__/SettingsInput.test.js @@ -20,7 +20,7 @@ import { useSettingsStore } from '../../stores/settingsStore.js'; import { describe, expect, test } from 'vitest'; import { nextTick } from 'vue'; -const getWrapper = function getWrapper(type = 'string', i18n = 'input', piniaOptions = {}) { +const getWrapper = function getWrapper({ type = 'string', i18n = 'input', piniaOptions = {} } = {}) { return mountComponent(SettingsInput, { attrs: { setting: { @@ -40,7 +40,7 @@ describe('SettingsInput', () => { const stringInputElement = stringSettingWrapper.find('input'); expect(stringInputElement.attributes('type')).toBe('text'); - const numberSettingWrapper = getWrapper('number'); + const numberSettingWrapper = getWrapper({ type: 'number' }); const numberInputElement = numberSettingWrapper.find('input'); expect(numberInputElement.attributes('type')).toBe('number'); @@ -55,10 +55,14 @@ describe('SettingsInput', () => { test('shows value from store', () => { const value = 'example value'; - const stringSettingWrapper = getWrapper('string', 'input', { - setupStores: () => { - const store = useSettingsStore(); - store.settings.input = value; + const stringSettingWrapper = getWrapper({ + type: 'string', + i18n: 'input', + piniaOptions: { + setupStores: () => { + const store = useSettingsStore(); + store.settings.input = value; + } } }); const inputElement = stringSettingWrapper.find('input');