fix: Fixed test and plain reporter

Author: kevinwang5658

src/ui/reporters/plain-reporter.ts

@@ -25,6 +25,11 @@ export class PlainReporter implements Reporter {
     }
   }
 
+  async displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void> {
+    console.log(chalk.bold('Additional information is required to continue import'))
+    console.log('Some of the resources specified in the import support multiple instances. Additional information is required to identify the specific instance to import. If importing multiple instances is desired (for ex: multiple git clones) additional imports can be added in the prompt.')
+  }
+
   async promptOptions(message: string, options: string[]): Promise<number> {
     console.log(message);
     console.log('')

test/orchestrator/mocks/reporter.ts

@@ -17,6 +17,7 @@ export interface MockReporterConfig {
   promptUserForValues?: (resourceInfo: ResourceInfo[]) => Promise<ResourceConfig[]> | ResourceConfig[];
   displayImportResult?: (importResult: ImportResult, showConfigs: boolean) => Promise<void> | void;
   displayFileModifications?: (diff: { file: string; modification: FileModificationResult; }[]) => void,
+  displayImportWarning?: (requiresParameters: string[], noParametersRequired: string[]) => void
 }
 
 export class MockReporter implements Reporter {
@@ -26,6 +27,14 @@ export class MockReporter implements Reporter {
     this.config = config ?? null;
   }
 
+  async displayImportWarning(requiresParameters: string[], noParametersRequired: string[]): Promise<void> {
+    console.log('Display import warning');
+    console.log(requiresParameters);
+    console.log(noParametersRequired);
+
+    this.config?.displayImportWarning?.(requiresParameters, noParametersRequired);
+  }
+
   async promptOptions(message: string, options: string[]): Promise<number> {
     return this.config?.promptOptions?.(message, options) ?? 0;
   }