fix: sonar complaints fix

SAP · Jan 9, 2024 · d427fe2 · d427fe2
1 parent dda44b7
commit d427fe2
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 7 deletions.
diff --git a/packages/binding-parser/src/lexer/token.ts b/packages/binding-parser/src/lexer/token.ts
@@ -25,7 +25,7 @@ const whiteSpace = createToken({
 const specialChars = createToken({
   name: SPECIAL_CHARS,
   pattern:
-    /(?:#|&gt;|&#47;|&#x2F;|!|"|\$|%|&|'|\(|\)|\*|\+|-|\.|\/|;|<|=|>|\?|@|\\|\^|_|`|~|\||)+/,
+    /(?:#|&gt;|&#47;|&#x2F;|!|"|\$|%|&|'|\(|\)|\*|\+|-|\.|\/|;|<|=|>|\?|@|\\|\^|_|`|~|\|)+/,
 });
 
 const leftCurly = createToken({

diff --git a/packages/binding-parser/test/unit/parser/binding-parser.test.ts b/packages/binding-parser/test/unit/parser/binding-parser.test.ts
@@ -67,9 +67,10 @@ describe("binding parser", () => {
       continue;
     }
     if (only.includes(t)) {
-      it.only(`${t}`, async () => {
+      const cb = async () => {
         await testParser(t);
-      });
+      };
+      it.only(`${t}`, cb); // NOSONAR
       continue;
     }
     it(`${t}`, async () => {

diff --git a/packages/language-server/src/server.ts b/packages/language-server/src/server.ts
@@ -104,7 +104,7 @@ connection.onInitialize(
   }
 );
 
-connection.onInitialized(async () => {
+connection.onInitialized(async (): Promise<void> => {
   getLogger().info("`onInitialized` event");
   if (hasConfigurationCapability) {
     // Register for all configuration changes
@@ -266,7 +266,7 @@ const validateOpenDocuments = async (changes: FileEvent[]): Promise<void> => {
   }
 };
 
-connection.onDidChangeWatchedFiles(async (changeEvent) => {
+connection.onDidChangeWatchedFiles(async (changeEvent): Promise<void> => {
   getLogger().debug("`onDidChangeWatchedFiles` event", {
     changeEvent,
   });

diff --git a/packages/vscode-ui5-language-assistant/src/extension.ts b/packages/vscode-ui5-language-assistant/src/extension.ts
@@ -74,11 +74,11 @@ export async function activate(context: ExtensionContext): Promise<void> {
   client.start().then(() => {
     client.onNotification(
       "UI5LanguageAssistant/ui5Model",
-      async (model: UI5Model) => await updateCurrentModel(model)
+      async (model: UI5Model): Promise<void> => await updateCurrentModel(model)
     );
     client.onNotification(
       "UI5LanguageAssistant/context-error",
-      async (error: Error) => await handleContextError(error)
+      (error: Error) => handleContextError(error)
     );
   });
   window.onDidChangeActiveTextEditor(async () => {