skip path test on windows, avoid generate assertion on windows

Author: Hweinstock

packages/core/src/awsService/ec2/sshKeyPair.ts

@@ -45,11 +45,11 @@ export class SshKeyPair {
 
     public static async generateSshKeyPair(keyPath: string): Promise<void> {
         const keyGenerated = await SshKeyPair.tryKeyTypes(keyPath, ['ed25519', 'rsa'])
-        await SshKeyPair.assertGenerated(keyPath, keyGenerated)
         // Should already be the case, but just in case we assert permissions.
         // skip on Windows since it only allows write permission to be changed.
         if (!globals.isWeb && os.platform() !== 'win32') {
             await fs.chmod(keyPath, 0o600)
+            await SshKeyPair.assertGenerated(keyPath, keyGenerated)
         }
     }
     /**

packages/core/src/test/shared/utilities/testUtils.test.ts

@@ -10,7 +10,12 @@ import { isWin } from '../../../shared/vscode/env'
 import { tryRun } from '../../../shared/utilities/pathFind'
 
 describe('setEnv', function () {
-    it('modifies the node environment variables', function () {
+    it('modifies the node environment variables (Non-Windows)', function () {
+        // PATH returns undefined on Windows.
+        if (isWin()) {
+            this.skip()
+        }
+
         const originalPath = copyEnv().PATH
         const fakePath = 'fakePath'
         process.env.PATH = fakePath