@shopbite-de/storefront 1.16.0 → 1.17.0

package/test/nuxt/useTopSellers.test.ts

@@ -1,6 +1,6 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import { mockNuxtImport } from "@nuxt/test-utils/runtime";
-import { useTopSellers } from "../../app/composables/useTopSellers";
+import { useTopSellers } from "~/composables/useTopSellers";
 
 const { mockInvoke } = vi.hoisted(() => ({
   mockInvoke: vi.fn(),
@@ -31,7 +31,7 @@ describe("useTopSellers", () => {
     const result = await loadTopSellers();
 
     expect(mockInvoke).toHaveBeenCalledWith(
-      "getTopSellers post /product",
+      "readProduct post /product",
       expect.any(Object),
     );
     expect(result).toEqual(mockElements);

package/test/nuxt/useWishlistActions.test.ts

@@ -1,6 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import { mockNuxtImport } from "@nuxt/test-utils/runtime";
 import { useWishlistActions } from "../../app/composables/useWishlistActions";
+import type { Schemas } from "#shopware";
 
 const {
   mockAddProducts,
@@ -44,7 +45,7 @@ describe("useWishlistActions", () => {
     id: "prod-1",
     translated: { name: "Test Product" },
     productNumber: "SW123",
-  } as any;
+  } as unknown as Schemas["Product"];
 
   beforeEach(() => {
     vi.clearAllMocks();
@@ -93,7 +94,7 @@ describe("useWishlistActions", () => {
     const products = [
       { id: "p1", translated: { name: "P1" } },
       { id: "p2", translated: { name: "P2" } },
-    ] as any[];
+    ] as unknown as Schemas["Product"][];
     const { addAllItemsToCart, isAddingToCart } = useWishlistActions();
     mockAddProducts.mockResolvedValue({ id: "cart-1" });
 
@@ -119,7 +120,7 @@ describe("useWishlistActions", () => {
     const products = [
       { id: "p1", translated: { name: "P1" } },
       { id: "p2", childCount: 1, translated: { name: "P2" } },
-    ] as any[];
+    ] as unknown as Schemas["Product"][];
     const { addAllItemsToCart } = useWishlistActions();
     mockAddProducts.mockResolvedValue({ id: "cart-1" });
 

package/test/unit/useCategorySeo.spec.ts

@@ -1,6 +1,9 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import { ref } from "vue";
 
+// Re-import the mocks for assertions
+import { useHead, useSeoMeta } from "#imports";
+
 // Create shared mocks to be exported by both '#imports' and '#app' in a hoisted-safe way
 const shared = vi.hoisted(() => ({
   useHead: vi.fn(),
@@ -41,24 +44,25 @@ vi.mock("#app", async () => {
   };
 });
 
-// Re-import the mocks for assertions
-import { useHead, useSeoMeta } from "#imports";
-
 // Target under test will be dynamically imported after setting up globals
-let useCategorySeo: (arg: any) => any;
+let useCategorySeo: (
+  arg: unknown,
+) => ReturnType<
+  (typeof import("../../app/composables/useCategorySeo"))["useCategorySeo"]
+>;
 
 describe("useCategorySeo", () => {
   beforeEach(async () => {
     vi.clearAllMocks();
     // Provide globals for auto-imported functions (when not transformed in unit env)
     const vue = await import("vue");
-    (globalThis as any).computed = vue.computed;
-    (globalThis as any).ref = vue.ref;
-    (globalThis as any).useRuntimeConfig = () => ({
+    (globalThis as Record<string, unknown>).computed = vue.computed;
+    (globalThis as Record<string, unknown>).ref = vue.ref;
+    (globalThis as Record<string, unknown>).useRuntimeConfig = () => ({
       public: { site: { name: "My Store" }, storeUrl: "https://example.com" },
     });
-    (globalThis as any).useHead = useHead;
-    (globalThis as any).useSeoMeta = useSeoMeta;
+    (globalThis as Record<string, unknown>).useHead = useHead;
+    (globalThis as Record<string, unknown>).useSeoMeta = useSeoMeta;
 
     // Dynamic import after globals are ready
     useCategorySeo = (await import("../../app/composables/useCategorySeo"))
@@ -66,7 +70,17 @@ describe("useCategorySeo", () => {
   });
 
   it("computes core SEO refs and injects head tags", () => {
-    const category = ref<any>({
+    const category = ref<{
+      translated?: {
+        metaTitle?: string;
+        metaDescription?: string;
+        breadcrumb?: string[];
+        name?: string;
+      };
+      seoUrl?: string;
+      active?: boolean;
+      media?: { url?: string };
+    }>({
       translated: {
         metaTitle: "Pizza & Pasta",
         metaDescription: "Leckere Pizza und Pasta bestellen",
@@ -96,7 +110,7 @@ describe("useCategorySeo", () => {
       useHead as unknown as ReturnType<typeof vi.fn>,
     ).toHaveBeenCalledTimes(1);
     const headArg = (useHead as unknown as ReturnType<typeof vi.fn>).mock
-      .calls[0][0];
+      .calls[0]![0]!;
 
     // Canonical link
     const link = headArg.link?.[0];
@@ -116,7 +130,7 @@ describe("useCategorySeo", () => {
   });
 
   it("sets robots to noindex when category is inactive", () => {
-    const category = ref<any>({
+    const category = ref({
       translated: { name: "Salate" },
       active: false,
       seoUrl: "/c/salate",

package/tsconfig.json

@@ -1,4 +1,24 @@
 {
-  // https://v3.nuxtjs.org/concepts/typescript
   "extends": "./.nuxt/tsconfig.json",
+  "include": [
+    ".nuxt/nuxt.d.ts",
+    ".nuxt/nuxt.node.d.ts",
+    "app/**/*",
+    "server/**/*",
+    "shared/**/*",
+    "modules/**/*",
+    "layers/**/*",
+    "test/**/*",
+    "tests/**/*",
+    "*.d.ts",
+    "nuxt.config.*",
+    ".config/nuxt.*",
+    "content.config.*"
+  ],
+  "exclude": [
+    "node_modules",
+  ],
+  "compilerOptions": {
+    "typeRoots": ["./api-types", "./node_modules/@types", "./node_modules"]
+  }
 }

package/app/middleware/trailing-slash.global.ts

@@ -1,19 +0,0 @@
-import { defineNuxtRouteMiddleware, navigateTo } from "#app";
-
-export default defineNuxtRouteMiddleware((to) => {
-  // Only handle category pages
-  if (!to.path.startsWith("/c/")) return;
-
-  // Skip if it already has a trailing slash or it's just "/c/"
-  if (to.path.endsWith("/") || to.path === "/c/") return;
-
-  // Preserve query and hash while adding the trailing slash
-  return navigateTo(
-    {
-      path: `${to.path}/`,
-      query: to.query,
-      hash: to.hash,
-    },
-    { redirectCode: 301 },
-  );
-});

package/server/utils/shopware/adminApiClient.ts

@@ -1,24 +0,0 @@
-import { createAdminAPIClient } from "@shopware/api-client";
-import type { operations } from "@shopware/api-client/admin-api-types";
-
-/**
- * Creates a Shopware Admin API client
- */
-export function createAdminApiClient(baseURL: string, accessToken: string) {
-  return createAdminAPIClient<operations>({
-    baseURL,
-    accessToken,
-  });
-}
-
-/**
- * Fetches sales channels using the Admin API client
- */
-export async function getSalesChannels(client: any) {
-  const response = await client.invoke({
-    method: "GET",
-    path: "/api/v3/sales-channel",
-  });
-
-  return response.data;
-}

package/test/unit/sales-channels.test.ts

@@ -1,66 +0,0 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
-import {
-  createAdminApiClient,
-  getSalesChannels,
-} from "../../server/utils/shopware/adminApiClient";
-
-describe("Shopware Admin API Client", () => {
-  describe("createAdminApiClient", () => {
-    it("should create a client with correct configuration", () => {
-      const endpoint = "https://shopware.example.com/api";
-      const accessToken = "test-token";
-
-      const client = createAdminApiClient(endpoint, accessToken);
-
-      expect(client).toBeDefined();
-      // The client should have the invoke method
-      expect(client).toHaveProperty("invoke");
-      expect(typeof client.invoke).toBe("function");
-    });
-  });
-
-  describe("getSalesChannels", () => {
-    it("should fetch sales channels successfully", async () => {
-      // Mock the client
-      const mockClient = {
-        invoke: vi.fn().mockResolvedValue({
-          data: [
-            { id: "1", name: "Storefront", typeId: "storefront" },
-            { id: "2", name: "Headless", typeId: "headless" },
-          ],
-        }),
-      };
-
-      const result = await getSalesChannels(mockClient);
-
-      expect(result).toEqual([
-        { id: "1", name: "Storefront", typeId: "storefront" },
-        { id: "2", name: "Headless", typeId: "headless" },
-      ]);
-
-      expect(mockClient.invoke).toHaveBeenCalledWith({
-        method: "GET",
-        path: "/api/v3/sales-channel",
-      });
-    });
-
-    it("should handle errors when fetching sales channels", async () => {
-      const mockClient = {
-        invoke: vi.fn().mockRejectedValue(new Error("API error")),
-      };
-
-      await expect(getSalesChannels(mockClient)).rejects.toThrow("API error");
-    });
-
-    it("should handle empty response gracefully", async () => {
-      const mockClient = {
-        invoke: vi.fn().mockResolvedValue({
-          data: [],
-        }),
-      };
-
-      const result = await getSalesChannels(mockClient);
-      expect(result).toEqual([]);
-    });
-  });
-});