comment

Author: roger-zhangg

packages/core/src/test/shared/env/resolveEnv.test.ts

@@ -19,10 +19,27 @@ describe('resolveEnv', async function () {
     })
 
     describe('resolveWindows', async function () {
+        beforeEach(function () {
+            sandbox.stub(process, 'platform').value('win32')
+        })
+
         it('mergeResolvedShellPath should not change path on windows', async function () {
             const env = await mergeResolvedShellPath(process.env)
             assert(env.PATH)
             assert.strictEqual(env, process.env)
         })
     })
+
+    describe('resolveMac', async function () {
+        beforeEach(function () {
+            sandbox.stub(process, 'platform').value('darwin')
+        })
+
+        it('mergeResolvedShellPath should get path on mac', async function () {
+            sandbox.stub(process.env, 'PATH').value('')
+            const env = await mergeResolvedShellPath(process.env)
+            assert(env.PATH)
+            assert.notEqual(env, process.env)
+        })
+    })
 })

packages/toolkit/.changes/next-release/Bug Fix-268ceb41-7bec-4b57-b195-d72f19d00faa.json

@@ -0,0 +1,4 @@
+{
+	"type": "Bug Fix",
+	"description": "getSpawnEnv always return correct env on windows"
+}