chore: remove onDidChangeTextDocument CA trigger

Signed-off-by: Ilona Shishov <[email protected]>
fabric8-analytics · Feb 12, 2024 · d7be642 · d7be642
1 parent 966eb6c
commit d7be642
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 20 deletions.
diff --git a/src/server.ts b/src/server.ts
@@ -12,11 +12,6 @@ import { globalConfig } from './config';
 import { AnalysisLSPServer } from './fileHandler';
 import { getDiagnosticsCodeActions, clearCodeActionsMap } from './codeActionHandler';
 
-/**
- * Declares timeout identifier to track delays for server.handleFileEvent execution
- */
-let checkDelay: NodeJS.Timeout;
-
 /**
  * Create a connection for the server, using Node's IPC as a transport.
  */
@@ -65,36 +60,30 @@ connection.onInitialized(() => {
 const server = new AnalysisLSPServer(connection);
 
 /**
- * On open document event handler
+ * On open document trigger event handler
  */
 connection.onDidOpenTextDocument((params) => {
     server.handleFileEvent(params.textDocument.uri, params.textDocument.text);
 });
 
 /**
- * On save document event handler
+ * On changes applied, apply to server file data
  */
-connection.onDidSaveTextDocument((params) => {
-    clearTimeout(checkDelay);
-    server.handleFileEvent(params.textDocument.uri, server.files.fileData[params.textDocument.uri]);
+connection.onDidChangeTextDocument((params) => {
+    server.files.fileData[params.textDocument.uri] = params.contentChanges[0].text;
 });
 
 /**
- * On changes applied to document event handler
+ * On save document trigger event handler
  */
-connection.onDidChangeTextDocument((params) => {
-    server.files.fileData[params.textDocument.uri] = params.contentChanges[0].text;
-    clearTimeout(checkDelay);
-    checkDelay = setTimeout(() => {
-        server.handleFileEvent(params.textDocument.uri, server.files.fileData[params.textDocument.uri]);
-    }, 3000);
+connection.onDidSaveTextDocument((params) => {
+    server.handleFileEvent(params.textDocument.uri, server.files.fileData[params.textDocument.uri]);
 });
 
 /**
- * On close document event handler
+ * On close document clear URI from codeActions Map
  */
 connection.onDidCloseTextDocument((params) => {
-    clearTimeout(checkDelay);
     clearCodeActionsMap(params.textDocument.uri);
 });
 

diff --git a/test/providers/pom.xml.test.ts b/test/providers/pom.xml.test.ts
@@ -153,7 +153,7 @@ describe('Java Maven pom.xml parser tests', () => {
         ]);
     });
 
-    it('test duplicate dependencies where none specify a version', async () => {
+    it('tests duplicate dependencies where none specify a version', async () => {
         const pom = `
             <project>
                 <dependencies>