feat: Renamed reporter method to promptUserForParameterValues

Author: kevinwang5658

src/orchestrators/import.ts

@@ -49,7 +49,7 @@ export class ImportOrchestrator {
     await ImportOrchestrator.validate(typeIds, project, pluginManager, dependencyMap)
 
     const requiredParameters = await pluginManager.getRequiredParameters(typeIds);
-    const userSuppliedParameters = await reporter.askRequiredParametersForImport(requiredParameters);
+    const userSuppliedParameters = await reporter.promptUserForParameterValues(requiredParameters);
 
     const importResult = await ImportOrchestrator.getImportedConfigs(pluginManager, typeIds, userSuppliedParameters)
 

src/ui/reporters/debug-reporter.ts

@@ -27,7 +27,7 @@ export class DebugReporter implements Reporter {
     ctx.on(Event.SUB_PROCESS_FINISH, (name) => debug(name))
   }
 
-  async askRequiredParametersForImport(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters> {
+  async promptUserForParameterValues(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters> {
     if (requiredParameters.size > 0 || [...requiredParameters.values()].reduce(
       (total, arr) => arr.length + total, 0
     )) {

src/ui/reporters/default-reporter.tsx

@@ -44,7 +44,7 @@ export class DefaultReporter implements Reporter {
     ctx.on(Event.SUB_PROCESS_FINISH, (name, additionalName) => this.onSubprocessFinishEvent(name, additionalName))
   }
 
-  async askRequiredParametersForImport(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters> {
+  async promptUserForParameterValues(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters> {
     if (requiredParameters.size === 0) {
       return new Map();
     }

src/ui/reporters/plain-reporter.ts

@@ -20,7 +20,7 @@ export class PlainReporter implements Reporter {
     ctx.on(Event.SUB_PROCESS_FINISH, (name) => console.log(name))
   }
 
-  async askRequiredParametersForImport(
+  async promptUserForParameterValues(
     requiredParameters: RequiredParameters
   ): Promise<UserSuppliedParameters> {
     if (requiredParameters.size > 0 || [...requiredParameters.values()].reduce(

src/ui/reporters/reporter.ts

@@ -48,8 +48,8 @@ export interface Reporter {
   promptConfirmation(message: string): Promise<boolean>
 
   promptSudo(pluginName: string, data: SudoRequestData, secureMode: boolean): Promise<SudoRequestResponseData>;
-  
-  askRequiredParametersForImport(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters>;
+
+  promptUserForParameterValues(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters>;
 
   displayImportResult(importResult: ImportResult): void;
 }

test/orchestrator/import/import.test.ts

@@ -73,7 +73,7 @@ describe('Import orchestrator tests', () => {
       }
     });
 
-    const askRequiredParametersSpy = vi.spyOn(reporter, 'askRequiredParametersForImport');
+    const askRequiredParametersSpy = vi.spyOn(reporter, 'promptUserForParameterValues');
     const displayImportResultSpy = vi.spyOn(reporter, 'displayImportResult');
 
     MockOs.create('mock', {

test/orchestrator/mocks/reporter.ts

@@ -42,7 +42,7 @@ export class MockReporter implements Reporter {
     }
   }
 
-  async askRequiredParametersForImport(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters> {
+  async promptUserForParameterValues(requiredParameters: RequiredParameters): Promise<UserSuppliedParameters> {
     if (this.config?.askRequiredParametersForImport) {
       return this.config.askRequiredParametersForImport(requiredParameters);
     }