Skip to content

Commit 5777d84

Browse files
fix: clean env vars in validate-env test beforeEach
The tests captured originalEnv at module load time. In CI, env vars like DATABASE_URL may already be set, so beforeEach was restoring them. Tests that assumed vars were missing actually had them present. Now beforeEach explicitly deletes all required env vars to ensure a clean state for each test.
1 parent 22993b0 commit 5777d84

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

‎test/unit/validate-env.spec.ts‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,11 @@ describe('validateEnvironment', () => {
99
beforeEach(() => {
1010
// Reset all mocks
1111
jest.clearAllMocks();
12-
// Reset process.env to original state before each test
12+
// Reset process.env to a clean state without any required vars
1313
process.env = { ...originalEnv };
14+
delete process.env.DATABASE_URL;
15+
delete process.env.JWT_SECRET;
16+
delete process.env.JWT_REFRESH_SECRET;
1417
});
1518

1619
afterAll(() => {

0 commit comments

Comments
 (0)