test(@nestjs/config) reset env correctly

process.env does not reset correctly. assgin copy value of origin process.env in beforeAll and reset process.env with value of envBackup
nestjs · Mar 22, 2024 · 6498d2b · 6498d2b
1 parent 7a7dc4d
commit 6498d2b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/tests/e2e/cache.spec.ts b/tests/e2e/cache.spec.ts
@@ -7,7 +7,9 @@ describe('Cache', () => {
   let app: INestApplication;
   let envBackup: NodeJS.ProcessEnv;
   beforeAll(() => {
-    envBackup = process.env;
+    envBackup = {
+      ...process.env,
+    };
   });
   describe('without cache', () => {
     beforeAll(async () => {
@@ -56,7 +58,9 @@ describe('Cache', () => {
   });
 
   afterEach(async () => {
-    process.env = envBackup;
+    process.env = {
+      ...envBackup,
+    };
     await app.close();
   });
 });
diff --git a/tests/e2e/load-priority.spec.ts b/tests/e2e/load-priority.spec.ts
@@ -8,7 +8,9 @@ describe('Environment variables and .env files', () => {
   let app: INestApplication;
   let envBackup: NodeJS.ProcessEnv;
   beforeAll(() => {
-    envBackup = process.env;
+    envBackup = {
+      ...process.env,
+    };
   });
   describe('without conflicts', () => {
     beforeAll(async () => {
@@ -65,7 +67,9 @@ describe('Environment variables and .env files', () => {
   });
 
   afterEach(async () => {
-    process.env = envBackup;
+    process.env = {
+      ...envBackup,
+    };
     await app.close();
   });
 });