Merge pull request #1869 from ahoppen/batch-filedidchange-syntacticindex

Batch updates to the syntactic test index on fileDidChange events

Author: ahoppen

Sources/SourceKitLSP/Swift/SyntacticTestIndex.swift

@@ -129,20 +129,24 @@ actor SyntacticTestIndex {
   }
 
   func filesDidChange(_ events: [FileEvent]) {
+    var removedFiles: Set<DocumentURI> = []
+    var filesToRescan: [DocumentURI] = []
     for fileEvent in events {
       switch fileEvent.type {
       case .created:
         // We don't know if this is a potential test file. It would need to be added to the index via
         // `listOfTestFilesDidChange`
         break
       case .changed:
-        rescanFiles([fileEvent.uri])
+        filesToRescan.append(fileEvent.uri)
       case .deleted:
-        removeFilesFromIndex([fileEvent.uri])
+        removedFiles.insert(fileEvent.uri)
       default:
         logger.error("Ignoring unknown FileEvent type \(fileEvent.type.rawValue) in SyntacticTestIndex")
       }
     }
+    removeFilesFromIndex(removedFiles)
+    rescanFiles(filesToRescan)
   }
 
   /// Called when a list of files was updated. Re-scans those files