Skip to content

Commit f30d412

Browse files
refactor(test): migrate storage-utils test to happy-dom and clean up configs
1 parent 7104c28 commit f30d412

2 files changed

Lines changed: 13 additions & 10 deletions

File tree

frontend/src/lib/storage-utils.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -38,17 +38,17 @@ describe("storage-utils", () => {
3838
const consoleSpy = vi
3939
.spyOn(console, "error")
4040
.mockImplementation(() => {});
41-
const originalSetItem = localStorage.setItem;
42-
// Manually overwrite to ensure it throws
43-
localStorage.setItem = () => {
44-
throw new Error("Storage full");
45-
};
41+
const setItemSpy = vi
42+
.spyOn(localStorage, "setItem")
43+
.mockImplementation(() => {
44+
throw new Error("Storage full");
45+
});
4646

4747
try {
4848
setStorageValue(STORAGE_KEYS.FEED_SORT_BY, "title_asc");
4949
expect(consoleSpy).toHaveBeenCalled();
5050
} finally {
51-
localStorage.setItem = originalSetItem;
51+
setItemSpy.mockRestore();
5252
}
5353
});
5454
});

vite.config.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,6 @@ export default defineConfig({
135135
"src/otel.test.ts",
136136
"src/mocks/**/*.test.ts",
137137
"src/lib/block-db.test.ts",
138-
"src/lib/storage-utils.test.ts",
139138
],
140139
setupFiles: ["./src/vitest-setup.ts"],
141140
globals: true,
@@ -147,7 +146,6 @@ export default defineConfig({
147146
extends: true,
148147
test: {
149148
name: "happy-dom",
150-
root: "frontend",
151149
environment: "happy-dom",
152150
isolate: false,
153151
restoreMocks: true,
@@ -159,7 +157,6 @@ export default defineConfig({
159157
"src/otel.test.ts",
160158
"src/mocks/**/*.test.ts",
161159
"src/lib/block-db.test.ts",
162-
"src/lib/storage-utils.test.ts",
163160
"**/node_modules/**",
164161
"**/dist/**",
165162
"**/cypress/**",
@@ -172,13 +169,19 @@ export default defineConfig({
172169
extends: true,
173170
test: {
174171
name: "node",
175-
root: "frontend",
176172
environment: "node",
177173
isolate: false,
178174
restoreMocks: true,
179175
mockReset: true,
180176
globals: true,
181177
include: ["src/**/*.node.test.{ts,tsx}"],
178+
exclude: [
179+
"**/node_modules/**",
180+
"**/dist/**",
181+
"**/cypress/**",
182+
"**/.{idea,git,cache,output,temp}/**",
183+
"**/{karma,rollup,webpack,vite,vitest}.config.*",
184+
],
182185
},
183186
},
184187
],

0 commit comments

Comments
 (0)