src/features/i18n/components/__tests__/LanguageSwitchButton.test.js

@@ -39,7 +39,7 @@ vi.mock('@/features/icons/components/Icon.vue', () => ({
     }
 }));
 
-describe('LanguageSwitchButton.vue', () => {
+describe('LanguageSwitchButton', () => {
     beforeEach(() => {
         vi.clearAllMocks();
         localStorage.clear();

src/features/settings/utils/__tests__/settingsValidator.test.js

@@ -17,7 +17,8 @@ limitations under the License.
 import { describe, test, expect } from 'vitest';
 import { validateSettingsConfig, validateEntry, validateSelectionOptions, assertType, assertString } from '../settingsValidator';
 
-describe('validateSettingsConfig', () => {
+describe('settingsValidator', () => {
+    describe('validateSettingsConfig', () => {
         test.for([
             { raw: false, expected: false },
             { raw: true, expected: false },
@@ -527,9 +528,9 @@ describe('validateSettingsConfig', () => {
             };
             expect(result.valid).toBe(expected);
         });
-});
+    });
 
-describe('validateEntry', () => {
+    describe('validateEntry', () => {
         test.for([
             [{ name: 'enableFeature42', type: 'bool', i18n: 'feature.42.enable' }],
             [{ name: 'enableFeature42', type: 'bool', i18n: 'feature.42.enable', default: true }],
@@ -661,9 +662,9 @@ describe('validateEntry', () => {
         ])('throws an error for the entry %s', ([ entry ]) => {
             expect(() => validateEntry(entry)).throws(Error);
         });
-});
+    });
 
-describe('validateSelectionOptions', () => {
+    describe('validateSelectionOptions', () => {
         test.for([
             [[{ name: 'test', i18n: 'test.label' }]],
             [[{ name: 'test', i18n: 'test.label' }, { name: 'test2', i18n: 'test2.label' }]],
@@ -687,9 +688,9 @@ describe('validateSelectionOptions', () => {
         ])('throws an error for the options %s', ([ options ]) => {
             expect(() => validateSelectionOptions(options)).throws(Error);
         });
-});
+    });
 
-describe('assertType', () => {
+    describe('assertType', () => {
         test.for([
             ['bool'],
             ['number'],
@@ -722,9 +723,9 @@ describe('assertType', () => {
         ])('throws an error for the value %s', ([ value ]) => {
             expect(() => assertType(value)).throw(Error);
         });
-});
+    });
 
-describe('assertString', () => {
+    describe('assertString', () => {
         test.for([
             ['a'],
             ['b'],
@@ -755,4 +756,5 @@ describe('assertString', () => {
         ])('throws an error for the value %s', ([ value ]) => {
             expect(() => assertString(value)).throws(Error);
         });
+    });
 });

src/utils/__tests__/cssDimensions.test.js

@@ -14,10 +14,11 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-import { expect,  test } from 'vitest';
+import { describe, expect,  test } from 'vitest';
 import { ensureUnit } from '../cssDimensions';
 
-test.for([
+describe('cssDimensions', () => {
+    test.for([
         { dimension: null,                       expected: '0px' },
         { dimension: undefined,                  expected: '0px' },
 
@@ -117,6 +118,7 @@ test.for([
         { dimension: 'max(42px,5rem)',           expected: 'max(42px,5rem)' },
         { dimension: 'clamp(42vh,23vw,13cap)',   expected: 'clamp(42vh,23vw,13cap)' },
         { dimension: 'clamp( 42vh,23vw,13cap )', expected: 'clamp( 42vh,23vw,13cap )' }
-])('ensureUnit returns $expected with input $dimension', ({ dimension, expected }) => {
+    ])('ensureUnit returns $expected with input $dimension', ({ dimension, expected }) => {
         expect(ensureUnit(dimension)).toBe(expected);
+    });
 });

src/utils/__tests__/currentLanguage.test.js

@@ -14,10 +14,11 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-import { expect, test } from 'vitest';
+import { describe, expect, test } from 'vitest';
 import getCurrentLanguage from '../currentLanguage';
 
-const locales = [
+describe('currentLanguage', () => {
+    test.for([
         { navigatorLanguage: 'en',                      localStorageLanguage: null, expected: 'en'},
         { navigatorLanguage: 'de',                      localStorageLanguage: null, expected: 'de'},
         { navigatorLanguage: 'fr',                      localStorageLanguage: null, expected: 'fr'},
@@ -45,9 +46,7 @@ const locales = [
         { navigatorLanguage: 'de-de',                   localStorageLanguage: 'en', expected: 'en'},
         { navigatorLanguage: 'zh-Hans-CN',              localStorageLanguage: 'fr', expected: 'fr'},
         { navigatorLanguage: 'en-US-u-ca-gregory',      localStorageLanguage: 'zh', expected: 'zh'}
-];
+    ])('returns the language $expected (navigator: $navigatorLanguage; local storage: $localStorageLanguage)', ({ navigatorLanguage, localStorageLanguage, expected }) => {
-
-test.for(locales)('returns the language $expected (navigator: $navigatorLanguage; local storage: $localStorageLanguage)', ({ navigatorLanguage, localStorageLanguage, expected }) => {
         Object.defineProperty(navigator, 'language', {
             value: navigatorLanguage,
             configurable: true
@@ -58,4 +57,5 @@ test.for(locales)('returns the language $expected (navigator: $navigatorLanguage
         };
 
         expect(getCurrentLanguage()).toBe(expected);
+    });
 });