@gsa-tts/graymatter-ui 2.0.2 → 2.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gsa-tts/graymatter-ui",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "type": "module",
   "publishConfig": {
     "access": "public"
@@ -42,26 +42,26 @@
   ],
   "devDependencies": {
     "@sveltejs/vite-plugin-svelte": "^6.2.4",
-    "@testing-library/jest-dom": "^6.9.1",
+    "@testing-library/jest-dom": "^7.0.1",
     "@testing-library/svelte": "^5.4.2",
     "@testing-library/user-event": "^14.6.7",
     "@types/node": "^22.20.1",
     "@uswds/compile": "1.3.2",
-    "@vitest/coverage-istanbul": "^3.2.7",
+    "@vitest/coverage-istanbul": "^5.0.0",
     "eslint": "^9.39.5",
     "gulp": "^5.0.0",
     "svelte": "^5.57.0",
     "typescript": "5.9.3",
     "vite": "^7.3.6",
-    "vitest": "^3.2.7",
+    "vitest": "^5.0.0",
     "@gsa-tts/graymatter-eslint": "0.0.4",
-    "@gsa-tts/graymatter-typescript-config": "0.0.4",
-    "@gsa-tts/graymatter-vitest-config": "0.0.3"
+    "@gsa-tts/graymatter-typescript-config": "0.0.5",
+    "@gsa-tts/graymatter-vitest-config": "0.0.4"
   },
   "dependencies": {
     "@uswds/uswds": "3.14.0",
     "uuid": "^14.0.1",
-    "@gsa-tts/graymatter-style-tokens": "0.6.3"
+    "@gsa-tts/graymatter-style-tokens": "2.0.3"
   },
   "scripts": {
     "build": "gulp update",

package/src/utils/setupMocks.ts

@@ -1,35 +1,42 @@
 import { vi } from 'vitest';
 import { writable, derived } from 'svelte/store';
 
-export function setupCommonMocks() {
-  // navigationStore.js mock
-  vi.mock('../stores/navigationStore.js', () => {
-    const isExpanded = writable(true);
-    const selectedItem = writable('console');
-    const selectedSubNavItemId = writable('');
-    const selectedSubNavItemTitle = writable('');
+// navigationStore.js mock. Vitest 5 requires vi.mock at the top level of a
+// module, so it can no longer live inside setupCommonMocks().
+//
+// Note this mock does not currently take effect: vi.mock is hoisted only within
+// *this* module, so it registers when this module is imported, which is after
+// the importing test file has already resolved its own import of
+// navigationStore.js. The four test files using this helper exercise the real
+// store. Pre-existing behavior, kept as-is here; fixing it is tracked
+// separately since it changes what those tests assert.
+vi.mock('../stores/navigationStore.js', () => {
+  const isExpanded = writable(true);
+  const selectedItem = writable('console');
+  const selectedSubNavItemId = writable('');
+  const selectedSubNavItemTitle = writable('');
 
-    const isMenuButtonDisabled = derived(
-      selectedItem,
-      $selectedItem => $selectedItem === 'discover'
-    );
+  const isMenuButtonDisabled = derived(
+    selectedItem,
+    $selectedItem => $selectedItem === 'discover'
+  );
 
-    return {
-      isExpanded,
-      selectedItem,
-      isMenuButtonDisabled,
-      selectedSubNavItemId,
-      selectedSubNavItemTitle,
-      navigationStore: {
-        setSelectedItem: vi.fn(),
-        setExpanded: (val: boolean) => isExpanded.set(val),
-        toggleExpanded: () =>
-          isExpanded.update((expanded: boolean) => !expanded),
-        handleLayoutTransition: vi.fn(),
-      },
-    };
-  });
+  return {
+    isExpanded,
+    selectedItem,
+    isMenuButtonDisabled,
+    selectedSubNavItemId,
+    selectedSubNavItemTitle,
+    navigationStore: {
+      setSelectedItem: vi.fn(),
+      setExpanded: (val: boolean) => isExpanded.set(val),
+      toggleExpanded: () => isExpanded.update((expanded: boolean) => !expanded),
+      handleLayoutTransition: vi.fn(),
+    },
+  };
+});
 
+export function setupCommonMocks() {
   vi.stubGlobal(
     'window',
     Object.create(window, {