параметр произвольного шаблона поиска методов

Author: artbear

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

@@ -48,8 +48,6 @@ public class ExternalAppStartingDiagnostic extends AbstractFindMethodDiagnostic
       "|ЗапуститьПриложениеАсинх|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;
@@ -60,17 +58,23 @@ public class ExternalAppStartingDiagnostic extends AbstractFindMethodDiagnostic
   )
   private boolean checkGotoUrl = CHECK_GOTO_URL;
 
+  @DiagnosticParameter(
+    type = String.class,
+    defaultValue = MAIN_PATTERN_STRING
+  )
+  private String userPatternString = MAIN_PATTERN_STRING;
+
   public ExternalAppStartingDiagnostic() {
     super(FULL_PATTERN);
   }
 
   @Override
   public void configure(Map<String, Object> configuration) {
     super.configure(configuration);
+    var pattern = userPatternString;
     if (checkGotoUrl){
-      setMethodPattern(FULL_PATTERN);
-    } else {
-      setMethodPattern(MAIN_PATTERN);
+      pattern += PATTERN_STRING_FOR_NAVI;
     }
+    setMethodPattern(CaseInsensitivePattern.compile(pattern));
   }
 }

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

@@ -1,3 +1,4 @@
 diagnosticMessage=Check the launch of an external application
 diagnosticName=External applications starting
 checkGotoUrl=Check navigation links
+userPatternString=User regex pattern

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

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

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

@@ -59,7 +59,7 @@ void test() {
   }
 
   @Test
-  void testConfigure() {
+  void testConfigure_checkGotoUrl() {
 
     Map<String, Object> configuration = diagnosticInstance.info.getDefaultConfiguration();
     configuration.put("checkGotoUrl", false);
@@ -72,9 +72,7 @@ void testConfigure() {
       .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)
@@ -85,4 +83,38 @@ void testConfigure() {
       .hasRange(35, 10, 34)
       .hasSize(12);
   }
+
+  @Test
+  void testConfigure_userPatternString() {
+
+    Map<String, Object> configuration = diagnosticInstance.info.getDefaultConfiguration();
+    configuration.put("userPatternString", "КомандаСистемы");
+    diagnosticInstance.configure(configuration);
+
+    List<Diagnostic> diagnostics = getDiagnostics();
+
+    assertThat(diagnostics, true)
+      .hasRange(8, 4, 18)
+
+      .hasRange(14, 4, 32)
+      .hasRange(15, 26, 52)
+      .hasRange(16, 26, 52)
+
+      .hasSize(4);
+  }
+
+  @Test
+  void testConfigure_userPatternString_checkGotoUrl() {
+
+    Map<String, Object> configuration = diagnosticInstance.info.getDefaultConfiguration();
+    configuration.put("checkGotoUrl", false);
+    configuration.put("userPatternString", "КомандаСистемы");
+    diagnosticInstance.configure(configuration);
+
+    List<Diagnostic> diagnostics = getDiagnostics();
+
+    assertThat(diagnostics, true)
+      .hasRange(8, 4, 18)
+      .hasSize(1);
+  }
 }