From ec03192f49b45ccbb38a03164c4f7e2193be5242 Mon Sep 17 00:00:00 2001 From: Liviu Burcusel Date: Mon, 22 Dec 2025 16:10:50 +0100 Subject: [PATCH] Removed ui components from code analysis / coverage and reduced code duplication --- sonar-project.properties | 4 ++-- tests/layouts/default/Sidebar.test.ts | 21 ++++++++------------- 2 files changed, 10 insertions(+), 15 deletions(-) diff --git a/sonar-project.properties b/sonar-project.properties index 1c80e34..7c4df0f 100644 --- a/sonar-project.properties +++ b/sonar-project.properties @@ -7,7 +7,7 @@ sonar.projectVersion=1.0.0 sonar.sourceEncoding=UTF-8 sonar.sources=app, tests sonar.inclusions=app/**/*.ts, app/**/*.js, app/**/*.vue, app/**/*.css, app/**/*.scss, tests/**/*.test.ts -sonar.exclusions=**/node_modules/**, **/coverage/**, *.config.ts -sonar.coverage.exclusions=tests/**, *.config.ts +sonar.exclusions=**/node_modules/**, **/coverage/**, app/components/ui/**, *.config.ts +sonar.coverage.exclusions=tests/**, app/components/ui/**, *.config.ts sonar.javascript.lcov.reportPaths=coverage/lcov.info # sonar.testExecutionReportPaths=coverage/sonar-report.xml diff --git a/tests/layouts/default/Sidebar.test.ts b/tests/layouts/default/Sidebar.test.ts index ba04808..e4e59b7 100644 --- a/tests/layouts/default/Sidebar.test.ts +++ b/tests/layouts/default/Sidebar.test.ts @@ -1,5 +1,5 @@ import { mount } from "@vue/test-utils"; -import { describe, it, expect, vi } from "vitest"; +import { describe, expect, it, vi } from "vitest"; import SidebarLayout from "~/layouts/default/Sidebar.vue"; import { ref } from "vue"; import type * as SidebarUI from "~/components/ui/sidebar"; @@ -124,27 +124,22 @@ describe("SidebarLayout", () => { expect(wrapper.text()).toContain("v1.0.0"); }); - it("renders navigation groups", () => { + it("renders sidebar content correctly", () => { const wrapper = mount(SidebarLayout); const text = wrapper.text(); + + // Navigation groups expect(text).toContain("Playground"); expect(text).toContain("Documentation"); expect(text).toContain("Settings"); - }); - it("renders user information", () => { - const wrapper = mount(SidebarLayout); - expect(wrapper.text()).toContain("Liviu"); - expect(wrapper.text()).toContain("x.liviu@gmail.com"); - }); + // User information + expect(text).toContain("Liviu"); + expect(text).toContain("x.liviu@gmail.com"); - it("renders sub-items in navigation", () => { - const wrapper = mount(SidebarLayout); - const text = wrapper.text(); - // Checking sub-items of Playground + // Sub-items expect(text).toContain("History"); expect(text).toContain("Starred"); - // Checking sub-items of Documentation expect(text).toContain("Introduction"); expect(text).toContain("Get Started"); });