test(diagnostics): улучшены тесты MagicDateDiagnosticTest с добавлением комментариев и улучшением читаемости

- Добавлены комментарии к тестам для лучшего понимания их целей.
- Упрощены утверждения в тестах, убраны избыточные комментарии.
- Обновлены проверки диапазонов для различных форматов дат и присваиваний, обеспечивая полное покрытие тестовых случаев.

Author: johnnyshut

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

@@ -82,56 +82,61 @@ void testConfigure() {
       .hasRange(65, 22, 32);
   }
 
+  /**
+   * Тест для проверки различных форматов дат.
+   * Проверяет, что невалидные даты генерируют диагностики.
+   */
   @Test
   void testValidDateFormats() {
-    // Тест для проверки различных форматов дат
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    // Проверяем, что невалидные даты генерируют диагностики
     assertThat(diagnostics, true)
-      .hasRange(11, 12, 22)  // "00020101" - должна быть диагностика
-      .hasRange(12, 12, 28)  // "00020101121314" - должна быть диагностика
-      .hasRange(13, 7, 17)   // '00010102' - должна быть диагностика
-      .hasRange(14, 14, 24); // '12340101' - должна быть диагностика
+      .hasRange(11, 12, 22)
+      .hasRange(12, 12, 28)
+      .hasRange(13, 7, 17)
+      .hasRange(14, 14, 24);
   }
 
+  /**
+   * Тест для проверки исключенных дат (авторизованных).
+   * Проверяет, что авторизованные даты не генерируют диагностики.
+   * Авторизованная дата не должна генерировать диагностику.
+   * Невалидный формат не должен генерировать диагностику.
+   */
   @Test
   void testExcludedDates() {
-    // Тест для проверки исключенных дат (авторизованных)
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    // Проверяем, что авторизованные даты не генерируют диагностики
-    // Строка 22: Дата("00010101") - авторизованная дата, не должна генерировать диагностику
-    // Строка 23: '0001-01why not?01' - невалидный формат, не должна генерировать диагностику
     assertThat(diagnostics, true)
-      .hasRange(23, 7, 26); // '0001-01why not?02' - должна быть диагностика
+      .hasRange(23, 7, 26);
   }
 
+  /**
+   * Тест для проверки подавления диагностики при простом присваивании даты.
+   * Проверяет, что присваивания дат генерируют диагностики.
+   */
   @Test
   void testSimpleDateAssignment() {
-    // Тест для проверки подавления диагностики при простом присваивании даты
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    // Проверяем, что присваивания дат генерируют диагностики
     assertThat(diagnostics, true)
-      .hasRange(11, 12, 22)  // День = Дата("00020101");
-      .hasRange(12, 12, 28)  // День = Дата("00020101121314") + Шаг;
-      .hasRange(13, 7, 17);  // День = '00010102' + Шаг;
+      .hasRange(11, 12, 22)
+      .hasRange(12, 12, 28)
+      .hasRange(13, 7, 17);
   }
 
+  /**
+   * Тест для проверки подавления диагностики при возврате даты из функции.
+   * Проверяет, что возврат дат из функций НЕ генерирует диагностики.
+   * Возврат дат не должен генерировать диагностики.
+   * Это проверяется отсутствием диагностик в этих строках.
+   */
   @Test
   void testReturnSimpleDate() {
-    // Тест для проверки подавления диагностики при возврате даты из функции
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    // Проверяем, что возврат дат из функций НЕ генерирует диагностики
-    // Строки 2 и 6: Возврат '39991231235959' и '99990101000000' - не должны генерировать диагностики
-    // Это проверяется отсутствием диагностик в этих строках
-    
-    // Проверяем, что общее количество диагностик соответствует ожидаемому
     assertThat(diagnostics).hasSize(17);
 
-    // Проверяем, что диагностики есть в других местах, но НЕ в строках с возвратом дат
     assertThat(diagnostics, true)
       .hasRange(11, 12, 22)
       .hasRange(12, 12, 28)
@@ -152,48 +157,50 @@ void testReturnSimpleDate() {
       .hasRange(65, 22, 32);
   }
 
+  /**
+   * Тест для проверки подавления диагностики при присваивании с методом Дата().
+   * Проверяет, что присваивания с методом Дата() генерируют диагностики.
+   */
   @Test
   void testAssignmentWithDateMethod() {
-    // Тест для проверки подавления диагностики при присваивании с методом Дата()
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    // Проверяем, что присваивания с методом Дата() генерируют диагностики
     assertThat(diagnostics, true)
-      .hasRange(11, 12, 22)  // День = Дата("00020101");
-      .hasRange(12, 12, 28) // День = Дата("00020101121314") + Шаг;
-      .hasRange(25, 87, 97) // ИменаПараметров = ... Дата("00050101")
-      .hasRange(26, 80, 90) // ИменаПараметров = ... "00050101", "00050101"
+      .hasRange(11, 12, 22)
+      .hasRange(12, 12, 28)
+      .hasRange(25, 87, 97)
+      .hasRange(26, 80, 90)
       .hasRange(26, 92, 102);
   }
 
+  /**
+   * Тест для проверки подавления диагностики при присваивании свойства структуры.
+   * Проверяет, что диагностика подавлена при присваивании свойства структуры.
+   * Присваивание свойства структуры не должно генерировать диагностику.
+   */
   @Test
   void testStructurePropertyAssignment() {
-    // Тест для проверки подавления диагностики при присваивании свойства структуры
-    // Используем существующий файл с тестовыми данными
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    // Проверяем, что диагностика на строке 30 подавлена (не входит в список)
-    // Строка 30: Выполнить("00020501121314" + '12350101') - присваивание свойства структуры
     assertThat(diagnostics, true)
-      .hasRange(11, 12, 22)  // День = Дата("00020101");
-      .hasRange(12, 12, 28)  // День = Дата("00020101121314") + Шаг;
-      .hasRange(13, 7, 17)   // День = '00010102' + Шаг;
-      .hasRange(14, 14, 24)  // Пока Сейчас < '12340101' Цикл
-      .hasRange(23, 7, 26)   // День = '0001-01why not?02' + Шаг;
-      .hasRange(25, 87, 97)  // ИменаПараметров = ... Дата("00050101")
-      .hasRange(26, 80, 90)  // ИменаПараметров = ... "00050101", "00050101"
-      .hasRange(26, 92, 102) // ИменаПараметров = ... "00050101", "00050101"
-      .hasRange(27, 22, 32)  // Настройки = Настройки('12350101');
-      .hasRange(28, 19, 35)  // Настройки.Свойство("00020501121314", ...)
-      .hasRange(29, 10, 26)  // Выполнить("00020501121314" + '12350101')
-      .hasRange(29, 29, 39)  // Выполнить("00020501121314" + '12350101')
-      .hasRange(58, 17, 27)  // Значение = ?(А = '39990202', ...)
-      .hasRange(58, 29, 45)  // Значение = ?(А = '39990202', ...)
-      .hasRange(58, 47, 63)  // Значение = ?(А = '39990202', ...)
-      .hasRange(60, 19, 29)  // Если Сейчас < Дата("12340101") Тогда
-      .hasRange(65, 22, 32); // Функция Метод(Дата1 = '39990202')
+      .hasRange(11, 12, 22)
+      .hasRange(12, 12, 28)
+      .hasRange(13, 7, 17)
+      .hasRange(14, 14, 24)
+      .hasRange(23, 7, 26)
+      .hasRange(25, 87, 97)
+      .hasRange(26, 80, 90)
+      .hasRange(26, 92, 102)
+      .hasRange(27, 22, 32)
+      .hasRange(28, 19, 35)
+      .hasRange(29, 10, 26)
+      .hasRange(29, 29, 39)
+      .hasRange(58, 17, 27)
+      .hasRange(58, 29, 45)
+      .hasRange(58, 47, 63)
+      .hasRange(60, 19, 29)
+      .hasRange(65, 22, 32);
 
-    // Проверяем, что общее количество диагностик равно 17 (без изменений)
     assertThat(diagnostics).hasSize(17);
   }