From cec44d90597eea7e2bad98c342a86ec65480fb2e Mon Sep 17 00:00:00 2001 From: Gergo Magyar Date: Thu, 18 Jun 2026 15:55:47 +0000 Subject: [PATCH] test(server): rename shadowed savedHome in server-mapping fallback test The inner savedHome (saves process.env.HOME) shadowed the describe-scope savedHome (saves process.env.GITNEXUS_HOME). Renamed the inner one to savedProcessHome at its declaration and both restore sites in the finally block. Pure rename, no behavior change. Co-Authored-By: Claude Opus 4.8 (1M context) --- gitnexus/test/unit/gitnexus-home-roots.test.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gitnexus/test/unit/gitnexus-home-roots.test.ts b/gitnexus/test/unit/gitnexus-home-roots.test.ts index a8e51030a..a01ddf309 100644 --- a/gitnexus/test/unit/gitnexus-home-roots.test.ts +++ b/gitnexus/test/unit/gitnexus-home-roots.test.ts @@ -80,7 +80,7 @@ describe('GITNEXUS_HOME path roots', () => { // ~/.gitnexus) without writing into the developer's actual // ~/.gitnexus/server-mapping.json. const fakeHome = await fs.mkdtemp(path.join(os.tmpdir(), 'gitnexus-fallback-home-')); - const savedHome = process.env.HOME; + const savedProcessHome = process.env.HOME; const savedUserProfile = process.env.USERPROFILE; delete process.env.GITNEXUS_HOME; process.env.HOME = fakeHome; @@ -96,8 +96,8 @@ describe('GITNEXUS_HOME path roots', () => { const { readServerMapping } = await import('../../src/core/embeddings/server-mapping.js'); expect(await readServerMapping('my-repo')).toBe('fallback-service'); } finally { - if (savedHome === undefined) delete process.env.HOME; - else process.env.HOME = savedHome; + if (savedProcessHome === undefined) delete process.env.HOME; + else process.env.HOME = savedProcessHome; if (savedUserProfile === undefined) delete process.env.USERPROFILE; else process.env.USERPROFILE = savedUserProfile; await fs.rm(fakeHome, { recursive: true, force: true });