SLVSCODE-873 crawl directory before listFilesInFolder is called

SonarSource · Nov 25, 2024 · fd29382 · fd29382
1 parent b93fd9c
commit fd29382
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/its/src/test/javaSuite/java.test.ts b/its/src/test/javaSuite/java.test.ts
@@ -66,9 +66,6 @@ suite('Java Test Suite', () => {
     const rangeInMiddleOfThrowsMyException = new vscode.Range(8, 54, 8, 54);
     const codeActionsResult = (await vscode.commands.executeCommand<(vscode.Command | vscode.CodeAction)[]>('vscode.executeCodeActionProvider', document.uri, rangeInMiddleOfThrowsMyException, vscode.CodeActionKind.QuickFix.value));
     // With old versions of VSCode, code actions are not necessarily filtered on kind
-    console.log('+++++++', codeActionsResult[0].title)
-    console.log('+++++++', codeActionsResult[2].title)
-    console.log('+++++++', codeActionsResult[1].title)
     const expectedActionTitles = [
       "SonarQube: Deactivate rule 'java:S1130'",
       "SonarQube: Remove \"MyException\"",

diff --git a/src/extension.ts b/src/extension.ts
@@ -231,6 +231,10 @@ export async function activate(context: VSCode.ExtensionContext) {
   });
   FixSuggestionService.init(languageClient);
 
+  VSCode.workspace.workspaceFolders.map((folder) => {
+    FileSystemServiceImpl.instance.crawlDirectory(folder.uri);
+  });
+
   installCustomRequestHandlers(context);
 
   const referenceBranchStatusItem = VSCode.window.createStatusBarItem(VSCode.StatusBarAlignment.Left, 1);
@@ -568,7 +572,6 @@ function installCustomRequestHandlers(context: VSCode.ExtensionContext) {
   languageClient.onNotification(protocol.ShowRuleDescriptionNotification.type, showRuleDescription(context));
   languageClient.onNotification(protocol.SuggestBindingNotification.type, params => suggestBinding(params));
   languageClient.onRequest(protocol.ListFilesInFolderRequest.type, async (params) => {
-    await FileSystemServiceImpl.instance.crawlDirectory(VSCode.Uri.parse(params.folderUri));
     return AutoBindingService.instance.listAutobindingFilesInFolder(params);
   }
   );

diff --git a/src/fileSystem/fileSystemServiceImpl.ts b/src/fileSystem/fileSystemServiceImpl.ts
@@ -30,7 +30,7 @@ export class FileSystemServiceImpl implements FileSystemService {
   }
 
   public async crawlDirectory(uri: Uri) {
-    await this.listFilesRecursively(uri, uri);
+    this.listFilesRecursively(uri, uri);
   }
 
   private async listFilesRecursively(configScopeUri: Uri, currentDirectory: Uri) {
@@ -45,7 +45,7 @@ export class FileSystemServiceImpl implements FileSystemService {
         }
         // .sonarlint folder is already handled separately, skipping it in recursive crawl
         if (type === vscode.FileType.Directory && name !== '.sonarlint') {
-          await this.listFilesRecursively(configScopeUri, fullFileUri);
+          this.listFilesRecursively(configScopeUri, fullFileUri);
         }
       }
     } catch (error) {