[FP] RefOveruse

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

@@ -130,7 +130,7 @@ private Map<String, Boolean> dataSourcesWithTabularSection(SDBLParser.QueryConte
     var newResult = findAllDataSourceWithoutInnerQueries(ctx)
       .collect(Collectors.toMap(
         RefOveruseDiagnostic::getTableNameOrAlias,
-        RefOveruseDiagnostic::isTableWithTabularSection,
+        this::isTableWithTabularSection,
         (existing, replacement) -> existing,
         HashMap::new));
 
@@ -196,7 +196,7 @@ private static String getTableNameOrAlias(DataSourceContext dataSource) {
       .orElse("");
   }
 
-  private static boolean isTableWithTabularSection(DataSourceContext dataSourceContext) {
+  private boolean isTableWithTabularSection(DataSourceContext dataSourceContext) {
     final var table = dataSourceContext.table();
     if (table == null) {
       return dataSourceContext.virtualTable() != null;

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

@@ -21,6 +21,7 @@
  */
 package com.github._1c_syntax.bsl.languageserver.diagnostics;
 
+import com.github._1c_syntax.utils.Absolute;
 import org.eclipse.lsp4j.Diagnostic;
 import org.junit.jupiter.api.Test;
 
@@ -29,12 +30,46 @@
 import static com.github._1c_syntax.bsl.languageserver.util.Assertions.assertThat;
 
 class RefOveruseDiagnosticTest extends AbstractDiagnosticTest<RefOveruseDiagnostic> {
+  private static final String PATH_TO_METADATA = "src/test/resources/metadata/designer";
+
   RefOveruseDiagnosticTest() {
     super(RefOveruseDiagnostic.class);
   }
 
   @Test
   void test() {
+    initServerContext(Absolute.path(PATH_TO_METADATA));
+
+    List<Diagnostic> diagnostics = getDiagnostics();
+
+    assertThat(diagnostics, true)
+      .hasRange(3, 28, 3, 45)
+      .hasRange(13, 8, 13, 34)
+      .hasRange(14, 8, 14, 38)
+      .hasRange(25, 8, 25, 21)
+      .hasRange(37, 8, 37, 29)
+      .hasRange(38, 8, 38, 35)
+      .hasRange(56, 37, 56, 43)
+      .hasRange(57, 42, 57, 48)
+      .hasRange(92, 8, 29)
+      .hasRange(153, 13, 153, 41)
+      .hasRange(164, 13, 164, 53)
+      .hasRange(178, 13, 178, 35)
+      .hasRange(216, 13, 37)
+      .hasRange(226, 13, 37)
+      .hasRange(238, 13, 38)
+      .hasRange(296, 33, 80)
+      .hasRange(300, 33, 70)
+      .hasRange(309, 12, 28)
+      .hasRange(309, 12, 28)
+      .hasRange(342, 12, 56)
+      .hasRange(343, 12, 56)
+      .hasRange(354, 26, 96)
+//      .hasRange(375, 20, 92)
+      .hasSize(21);
+  }
+  @Test
+  void testSingleFile() {
 
     List<Diagnostic> diagnostics = getDiagnostics();
 
@@ -61,6 +96,7 @@ void test() {
       .hasRange(342, 12, 56)
       .hasRange(343, 12, 56)
       .hasRange(354, 26, 96)
+//      .hasRange(375, 20, 92)
       .hasSize(21);
   }
 }

src/test/resources/diagnostics/RefOveruseDiagnostic.bsl

@@ -1,3 +1,10 @@
+Процедура Тест31()
+    Запрос.Текст =
+    "ВЫБРАТЬ
+    |    Справочник1.ТабличнаяЧасть1.Ссылка
+    |ИЗ
+    |    Справочник.Справочник1 КАК Справочник1";
+КонецПроцедуры
 Процедура Тест()
 
     Запрос = Новый Запрос;
@@ -359,20 +366,22 @@
         |	Справочник.Пользователи.ДополнительныеРеквизиты КАК ПользователиДополнительныеРеквизиты";
 КонецПроцедуры
 
-Процедура Тест30()
-
-Запрос.Текст =
-	"ВЫБРАТЬ
-	|	УдалитьСоглашенияОбИспользованииЭД.Ссылка
-	|ИЗ
-	|	Справочник.УдалитьСоглашенияОбИспользованииЭД КАК УдалитьСоглашенияОбИспользованииЭД
-	|ГДЕ
-	|	НЕ УдалитьСоглашенияОбИспользованииЭД.УдалитьИспользуетсяКриптография
-	|
-	|СГРУППИРОВАТЬ ПО
-	|	УдалитьСоглашенияОбИспользованииЭД.Ссылка
-	|
-	|ИМЕЮЩИЕ
-	|	КОЛИЧЕСТВО(УдалитьСоглашенияОбИспользованииЭД.СертификатыПодписейОрганизации.Ссылка) > 0"
+Процедура Тест31()
+    Запрос.Текст =
+    "ВЫБРАТЬ
+    |    Справочник1.ТабличнаяЧасть1.Ссылка
+    |ИЗ
+    |    Справочник.Справочник1 КАК Справочник1";
+КонецПроцедуры
 
+Процедура Тест32()
+    Запрос.Текст =
+    "ВЫБРАТЬ
+    |    Справочник1.Ссылка
+    |ИЗ
+    |    Справочник.Справочник1 КАК Справочник1
+    |СГРУППИРОВАТЬ ПО
+    |    Справочник1.Ссылка
+    |ИМЕЮЩИЕ
+    |    КОЛИЧЕСТВО(Справочник1.ТабличнаяЧасть1.Ссылка) > 0";
 КонецПроцедуры