Add tests for prompt display based on flag input during init

packages/cli/src/commands/hydrogen/init.test.ts

@@ -25,69 +25,80 @@ describe('init', () => {
     ...stubs,
   });
 
-  describe('installDeps', () => {
-    it('prompts the user to install dependencies when installDeps is not passed', async () => {
+  describe.each([
+    {flag: 'template', value: 'hello-world'},
+    {flag: 'installDeps', value: true},
+    {flag: 'language', value: 'ts'},
+    {flag: 'path', value: './my-app'},
+  ])('flag $flag', ({flag, value}) => {
+    it(`does not prompt the user for ${flag} when a value is passed in options`, async () => {
       await temporaryDirectoryTask(async (tmpDir) => {
         // Given
-        const options = defaultOptions({path: tmpDir});
-
-        vi.mocked(ui.prompt).mockImplementation(() =>
-          Promise.resolve({installDeps: 'false'}),
-        );
+        const options = defaultOptions({
+          path: tmpDir,
+          [flag as string]: value,
+        });
 
         // When
         await runInit(options);
 
         // Then
-        expect(ui.prompt).toHaveBeenCalledWith(
+        expect(ui.prompt).not.toHaveBeenCalledWith(
           expect.arrayContaining([
             expect.objectContaining({
-              name: 'installDeps',
+              name: flag,
             }),
           ]),
         );
-        expect(installNodeModules).not.toHaveBeenCalled();
       });
     });
 
-    it('does not prompt the user to install dependencies when installDeps is true', async () => {
+    it(`prompts the user for ${flag} when no value is passed in options`, async () => {
       await temporaryDirectoryTask(async (tmpDir) => {
         // Given
-        const options = defaultOptions({installDeps: true, path: tmpDir});
+        const options = defaultOptions({
+          path: tmpDir,
+          [flag as string]: undefined,
+        });
 
         // When
         await runInit(options);
 
         // Then
-        expect(ui.prompt).not.toHaveBeenCalledWith(
+        expect(ui.prompt).toHaveBeenCalledWith(
           expect.arrayContaining([
             expect.objectContaining({
-              name: 'installDeps',
+              name: flag,
             }),
           ]),
         );
-        expect(installNodeModules).toHaveBeenCalled();
       });
     });
+  });
 
-    it('does not show a prompt to install dependencies when installDeps is false', async () => {
-      await temporaryDirectoryTask(async (tmpDir) => {
-        // Given
-        const options = defaultOptions({installDeps: false, path: tmpDir});
+  it('installs dependencies when installDeps is true', async () => {
+    await temporaryDirectoryTask(async (tmpDir) => {
+      // Given
+      const options = defaultOptions({installDeps: true, path: tmpDir});
 
-        // When
-        await runInit(options);
+      // When
+      await runInit(options);
 
-        // Then
-        expect(ui.prompt).not.toHaveBeenCalledWith(
-          expect.arrayContaining([
-            expect.objectContaining({
-              name: 'installDeps',
-            }),
-          ]),
-        );
-        expect(installNodeModules).not.toHaveBeenCalled();
-      });
+      // Then
+      expect(installNodeModules).toHaveBeenCalled();
+    });
+  });
+
+  it('does not install dependencies when installDeps is false', async () => {
+    await temporaryDirectoryTask(async (tmpDir) => {
+      // Given
+      const options = defaultOptions({installDeps: false, path: tmpDir});
+
+      // When
+      await runInit(options);
+
+      // Then
+      expect(installNodeModules).not.toHaveBeenCalled();
     });
   });
 });

packages/cli/src/commands/hydrogen/init.ts

@@ -41,7 +41,7 @@ export default class Init extends Command {
     }),
     'install-deps': Flags.boolean({
       description: 'Auto install dependencies using the active package manager',
-      env: 'SHOPIFY_HYDROGEN_INSTALL_DEPS',
+      env: 'SHOPIFY_HYDROGEN_FLAG_INSTALL_DEPS',
       allowNo: true,
     }),
   };