Fix path tests

Author: kevinwang5658

.run/test_integration_dev -- $FilePathRelativeToProjectRoot$.run.xml

@@ -1,5 +1,5 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="test:integration:dev -- $FilePathRelativeToProjectRoot$" type="js.build_tools.npm" nameIsGenerated="true">
+  <configuration default="false" name="test:integration:dev -- $FilePathRelativeToProjectRoot$" type="js.build_tools.npm">
     <package-json value="$PROJECT_DIR$/package.json" />
     <command value="run" />
     <scripts>

src/resources/shell/path/path-resource.ts

@@ -94,9 +94,6 @@ export class PathResource extends Resource<PathConfig> {
   }
 
   override async refresh(parameters: Partial<PathConfig>, context: RefreshContext<PathConfig>): Promise<Partial<PathConfig> | null> {
-    console.log('My shell', process.env.SHELL);
-    console.log('Other shell method', os.userInfo().shell)
-
     // If declarations only, we only look into files to find potential paths
     if (parameters.declarationsOnly || context.isStateful) {
       const pathsResult = new Set<string>();

test/shell/path.test.ts

@@ -18,10 +18,10 @@ describe('Path resource integration tests', async () => {
       }
     ], {
       validateApply: async () => {
-        expect((await testSpawn(TestUtils.getShellCommand('echo $PATH'))).data).to.include(tempDir1);
+        expect((await testSpawn('echo $PATH', { interactive: true })).data).to.include(tempDir1);
       },
       validateDestroy: async () => {
-        expect((await testSpawn(TestUtils.getShellCommand('echo $PATH'))).data).to.not.include(tempDir1);
+        expect((await testSpawn('echo $PATH', { interactive: true })).data).to.not.include(tempDir1);
       }
     });
   })
@@ -40,12 +40,12 @@ describe('Path resource integration tests', async () => {
         console.log(JSON.stringify(plan, null, 2));
       },
       validateApply: async () => {
-        const { data: path } = await testSpawn('echo $PATH');
+        const { data: path } = await testSpawn('echo $PATH', { interactive: true })
         expect(path).to.include(tempDir1);
         expect(path).to.include(tempDir2);
       },
       validateDestroy: async () => {
-        const { data: path } = await testSpawn('echo $PATH')
+        const { data: path } = await testSpawn('echo $PATH', { interactive: true })
         expect(path).to.not.include(tempDir1);
         expect(path).to.not.include(tempDir2);
       }
@@ -64,12 +64,12 @@ describe('Path resource integration tests', async () => {
       }
     ], {
       validateApply: async () => {
-        const { data: path } = await testSpawn('echo $PATH')
+        const { data: path } = await testSpawn('echo $PATH', { interactive: true })
         expect(path).to.include(tempDir1);
         expect(path).to.include(tempDir2);
       },
       validateDestroy: async () => {
-        const { data: path } = await testSpawn('echo $PATH')
+        const { data: path } = await testSpawn('echo $PATH', { interactive: true })
         expect(path).to.not.include(tempDir1);
         expect(path).to.not.include(tempDir2);
       }
@@ -90,7 +90,7 @@ describe('Path resource integration tests', async () => {
       }
     ], {
       validateApply: async () => {
-        const { data: path } = await testSpawn('echo $PATH');
+        const { data: path } = await testSpawn('echo $PATH', { interactive: true })
         expect(path).to.include(tempDir1);
         expect(path).to.include(tempDir2);
       },
@@ -111,15 +111,15 @@ describe('Path resource integration tests', async () => {
             })])
           })
 
-          const { data: path } = await testSpawn('echo $PATH');
+          const { data: path } = await testSpawn('echo $PATH', { interactive: true })
           expect(path).to.include(tempDir1);
           expect(path).to.include(tempDir2);
           expect(path).to.include(tempDir3);
           expect(path).to.include(tempDir4);
         }
       },
       validateDestroy: async () => {
-        const { data: path } = await testSpawn('echo $PATH');
+        const { data: path } = await testSpawn('echo $PATH', { interactive: true })
         expect(path).to.not.include(tempDir1);
         expect(path).to.not.include(tempDir2);
         expect(path).to.not.include(tempDir3);