test(legal): add test for the legal notice view

Added tests for the legal notice view (LegalNoticeView.vue).

src/features/legal/services/__tests__/fetchLegalContent.test.js

@@ -14,13 +14,13 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-import { FetchError } from '../errors';
+import { FetchError } from '../../errors';
-import { getLegalNotice } from './fetchLegalContent';
+import { getLegalNotice } from '../fetchLegalContent';
 import { afterEach, beforeEach, describe, expect, test, vi } from 'vitest';
 
 let fetchSpy;
 
-beforeEach(() => {  
+beforeEach(() => {
     fetchSpy = vi.spyOn(globalThis, 'fetch');
 });
 

src/features/legal/views/LegalNoticeView.vue

@@ -27,9 +27,10 @@ const legalNoticeHtml = ref(null);
 onMounted(async () => {
     try {
         legalNoticeHtml.value = await getLegalNotice();
-        loading.value = false;
     } catch {
         legalNoticeError.value = 'legal.notice.loadError';
+    } finally {
+        loading.value = false;
     }
 });
 </script>
@@ -42,7 +43,7 @@ onMounted(async () => {
             </h1>
         </header>
         <main>
-            <div v-if="loading">
+            <div class="legal-notice-loading" v-if="loading">
                 {{ t('loading') }}
             </div>
             <div class="error" v-else-if="legalNoticeError">

src/features/legal/views/__tests__/LegalNoticeView.test.js

@@ -0,0 +1,78 @@
+/*
+Copyright 2026 Seekra
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+import { FetchError } from '../../errors.js';
+import LegalNoticeView from '../LegalNoticeView.vue';
+import { mountComponent } from '@/test-utils/mountComponent';
+import { flushPromises } from '@vue/test-utils';
+import { afterEach, describe, expect, test, vi } from 'vitest';
+
+const getLegalNotice = vi.hoisted(() => vi.fn().mockResolvedValue('legal notice'));
+vi.mock('../../services/fetchLegalContent', async (importOriginal) => {
+    const actual = await importOriginal();
+    return {
+        ...actual,
+        getLegalNotice
+    };
+});
+
+afterEach(() => getLegalNotice.mockRestore());
+
+describe('LegalNoticeView', () => {
+    test('shows loading indicator', () => {
+        const wrapper = mountComponent(LegalNoticeView);
+
+        const legalNoticeLoading = wrapper.find('.legal-notice-loading');
+        const legalNoticeError = wrapper.find('.error');
+        const legalNotice = wrapper.find('.legal-notice');
+
+        expect(legalNoticeLoading.exists()).toBe(true);
+        expect(legalNoticeError.exists()).toBe(false);
+        expect(legalNotice.exists()).toBe(false);
+    });
+
+    test('shows error', async () => {
+        getLegalNotice.mockRejectedValue(new FetchError());
+
+        const wrapper = mountComponent(LegalNoticeView);
+
+        await flushPromises();
+        await wrapper.vm.$nextTick();
+
+        const legalNoticeLoading = wrapper.find('.legal-notice-loading');
+        const legalNoticeError = wrapper.find('.error');
+        const legalNotice = wrapper.find('.legal-notice');
+
+        expect(legalNoticeLoading.exists()).toBe(false);
+        expect(legalNoticeError.exists()).toBe(true);
+        expect(legalNotice.exists()).toBe(false);
+    });
+
+    test('shows legal notice', async () => {
+        const wrapper = mountComponent(LegalNoticeView);
+
+        await flushPromises();
+        await wrapper.vm.$nextTick();
+
+        const legalNoticeLoading = wrapper.find('.legal-notice-loading');
+        const legalNoticeError = wrapper.find('.error');
+        const legalNotice = wrapper.find('.legal-notice');
+
+        expect(legalNoticeLoading.exists()).toBe(false);
+        expect(legalNoticeError.exists()).toBe(false);
+        expect(legalNotice.exists()).toBe(true);
+    });
+});

src/features/settings/composables/useSettings.js

@@ -1,43 +0,0 @@
-/*
-Copyright 2026 Seekra
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-import { useSettingsStore } from '../stores/settingsStore';
-import { getSettingRecursively } from '../utils/getSetting';
-import { loadSettingsConfig } from '../utils/settingsParser';
-
-/**
- * Provides access to the stored settings.
- */
-export const useSettings = function useSettings () {
-    /**
-     * Returns the value of a specific setting.
-     * @param {string[]} key - The settings key.
-     * @return The value of the setting.
-     */
-    const getSetting = async function getSetting (key) {
-        const settingsStore = useSettingsStore();
-
-        const settings = await loadSettingsConfig();
-        return (
-            settingsStore.get(key) ??
-            getSettingRecursively(
-                key, settings.contents ?? []
-            )?.default
-        );
-    };
-
-    return { getSetting };
-};

src/features/sidebar/components/Sidebar.vue

@@ -61,6 +61,10 @@ const toggleSidebar = function toggleSidebar () {
 }
 
 @media (max-width: 48rem) {
+    .sidebar {
+        border: none;
+    }
+
     .sidebar:not(.expanded) .sidebar-expand-button {
         display: none;
     }