параметр для перехода по нави-ссылкам

Author: artbear

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/ExternalAppStartingDiagnostic.java

@@ -22,12 +22,14 @@
 package com.github._1c_syntax.bsl.languageserver.diagnostics;
 
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticMetadata;
+import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticParameter;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticScope;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticSeverity;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticTag;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticType;
 import com.github._1c_syntax.utils.CaseInsensitivePattern;
 
+import java.util.Map;
 import java.util.regex.Pattern;
 
 @DiagnosticMetadata(
@@ -41,15 +43,34 @@
 
 )
 public class ExternalAppStartingDiagnostic extends AbstractFindMethodDiagnostic  {
-  private static final Pattern messagePattern = CaseInsensitivePattern.compile(
+  private static final String MAIN_PATTERN_STRING =
     "КомандаСистемы|System|ЗапуститьПриложение|RunApp|НачатьЗапускПриложения|BeginRunningApplication" +
-      "|ЗапуститьПриложениеАсинх|RunAppAsync|ПерейтиПоНавигационнойСсылке|GotoURL" +
-      "|ОткрытьНавигационнуюСсылку|ЗапуститьПрограмму|ОткрытьПроводник|ОткрытьФайл"
-  );
+      "|ЗапуститьПриложениеАсинх|RunAppAsync|ЗапуститьПрограмму|ОткрытьПроводник|ОткрытьФайл";
+  private static final String PATTERN_STRING_FOR_NAVI =
+    "|ПерейтиПоНавигационнойСсылке|GotoURL|ОткрытьНавигационнуюСсылку";
   // TODO Использование COM объектов "Wscript.Shell" и "Shell.Application" + документация в описании правила
+  private static final Pattern MAIN_PATTERN = CaseInsensitivePattern.compile(MAIN_PATTERN_STRING);
+  private static final Pattern FULL_PATTERN = CaseInsensitivePattern.compile(
+    MAIN_PATTERN_STRING + PATTERN_STRING_FOR_NAVI);
+  private static final boolean CHECK_GOTO_URL = true;
+
+  @DiagnosticParameter(
+    type = Boolean.class,
+    defaultValue = "" + CHECK_GOTO_URL
+  )
+  private boolean checkGotoUrl = CHECK_GOTO_URL;
 
   public ExternalAppStartingDiagnostic() {
-    super(messagePattern);
+    super(FULL_PATTERN);
   }
 
+  @Override
+  public void configure(Map<String, Object> configuration) {
+    super.configure(configuration);
+    if (checkGotoUrl){
+      setMethodPattern(FULL_PATTERN);
+    } else {
+      setMethodPattern(MAIN_PATTERN);
+    }
+  }
 }

src/main/resources/com/github/_1c_syntax/bsl/languageserver/diagnostics/ExternalAppStartingDiagnostic_en.properties

@@ -1,2 +1,3 @@
 diagnosticMessage=Check the launch of an external application
 diagnosticName=External applications starting
+checkGotoUrl=Check navigation links

src/main/resources/com/github/_1c_syntax/bsl/languageserver/diagnostics/ExternalAppStartingDiagnostic_ru.properties

@@ -1,2 +1,3 @@
 diagnosticMessage=Проверьте запуск внешнего приложения
 diagnosticName=Запуск внешних приложений
+checkGotoUrl=Проверять переход по навигационным ссылкам

src/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/ExternalAppStartingDiagnosticTest.java

@@ -25,6 +25,7 @@
 import org.junit.jupiter.api.Test;
 
 import java.util.List;
+import java.util.Map;
 
 import static com.github._1c_syntax.bsl.languageserver.util.Assertions.assertThat;
 
@@ -54,7 +55,34 @@ void test() {
       .hasRange(24, 26, 37)
       .hasRange(25, 26, 37)
       .hasRange(35, 10, 34)
-      .hasSize(15)
-    ;
+      .hasSize(15);
+  }
+
+  @Test
+  void testConfigure() {
+
+    Map<String, Object> configuration = diagnosticInstance.info.getDefaultConfiguration();
+    configuration.put("checkGotoUrl", false);
+    diagnosticInstance.configure(configuration);
+
+    List<Diagnostic> diagnostics = getDiagnostics();
+
+    assertThat(diagnostics, true)
+      .hasRange(8, 4, 18)
+      .hasRange(9, 4, 23)
+      .hasRange(10, 4, 23)
+      .hasRange(12, 4, 26)
+//      .hasRange(14, 4, 32)
+//      .hasRange(15, 26, 52)
+//      .hasRange(16, 26, 52)
+      .hasRange(18, 26, 44)
+      .hasRange(19, 26, 44)
+      .hasRange(20, 20, 38)
+      .hasRange(21, 20, 38)
+      .hasRange(23, 26, 42)
+      .hasRange(24, 26, 37)
+      .hasRange(25, 26, 37)
+      .hasRange(35, 10, 34)
+      .hasSize(12);
   }
 }

src/test/resources/diagnostics/ExternalAppStartingDiagnostic.bsl

@@ -35,3 +35,12 @@
 
     Ждать ЗапуститьПриложениеАсинх(СтрокаКоманды, ТекущийКаталог, ДождатьсяЗавершения); // есть замечание
 КонецПроцедуры
+
+&НаСервере
+Процедура ПараметрНаИмяВнешнегоПриложения()
+    СтрокаКоманды = "";
+    ТекущийКаталог = "";
+    ДождатьсяЗавершения = Истина;
+
+    МойОбщийМодуль.ЗапуститьВнешнееПриложение(СтрокаКоманды, ТекущийКаталог, ДождатьсяЗавершения); // есть замечание
+КонецПроцедуры