Merge branch 'develop'

Author: nixel2007

src/main/java/org/github/_1c_syntax/bsl/languageserver/diagnostics/MagicNumberDiagnostic.java

@@ -21,6 +21,7 @@
  */
 package org.github._1c_syntax.bsl.languageserver.diagnostics;
 
+import org.antlr.v4.runtime.ParserRuleContext;
 import org.antlr.v4.runtime.tree.ParseTree;
 import org.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticMetadata;
 import org.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticParameter;
@@ -59,7 +60,7 @@ private boolean isExcluded(String s) {
     return false;
   }
 
-  private boolean isNumericExpression(BSLParser.ExpressionContext expression) {
+  private static boolean isNumericExpression(BSLParser.ExpressionContext expression) {
     return (expression.getChildCount() <= 1);
   }
 
@@ -75,16 +76,21 @@ public void configure(Map<String, Object> configuration) {
     }
   }
 
+  @Override
   public ParseTree visitNumeric(BSLParser.NumericContext ctx) {
     String checked = ctx.getText();
 
     if(checked == null || isExcluded(checked)) {
       return super.visitNumeric(ctx);
     }
 
-    BSLParser.ExpressionContext expression = (BSLParser.ExpressionContext) ctx.getParent().getParent().getParent();
+    ParserRuleContext expression = ctx.getParent().getParent().getParent();
+
+    if (!(expression instanceof BSLParser.ExpressionContext)) {
+      return ctx;
+    }
 
-    if (isNumericExpression(expression)) {
+    if (isNumericExpression((BSLParser.ExpressionContext) expression)) {
       return super.visitNumeric(ctx);
     }
 

src/test/java/org/github/_1c_syntax/bsl/languageserver/diagnostics/MagicNumberDiagnosticTest.java

@@ -46,9 +46,9 @@ void runTest() {
     assertThat(diagnostics.get(1).getRange()).isEqualTo(RangeHelper.newRange(3, 23, 3, 25));
     assertThat(diagnostics.get(2).getRange()).isEqualTo(RangeHelper.newRange(7, 31, 7, 33));
     assertThat(diagnostics.get(3).getRange()).isEqualTo(RangeHelper.newRange(11, 20, 11, 21));
-    assertThat(diagnostics.get(4).getRange()).isEqualTo(RangeHelper.newRange(19, 17, 19, 19));
-    assertThat(diagnostics.get(5).getRange()).isEqualTo(RangeHelper.newRange(22, 20, 22, 22));
-    assertThat(diagnostics.get(6).getRange()).isEqualTo(RangeHelper.newRange(26, 30, 26, 31));
+    assertThat(diagnostics.get(4).getRange()).isEqualTo(RangeHelper.newRange(20, 21, 20, 23));
+    assertThat(diagnostics.get(5).getRange()).isEqualTo(RangeHelper.newRange(23, 24, 23, 26));
+    assertThat(diagnostics.get(6).getRange()).isEqualTo(RangeHelper.newRange(27, 34, 27, 35));
   }
 
   @Test
@@ -65,7 +65,7 @@ void testConfigure() {
     assertThat(diagnostics).hasSize(4);
     assertThat(diagnostics.get(0).getRange()).isEqualTo(RangeHelper.newRange(7, 31, 7, 33));
     assertThat(diagnostics.get(1).getRange()).isEqualTo(RangeHelper.newRange(11, 20, 11, 21));
-    assertThat(diagnostics.get(2).getRange()).isEqualTo(RangeHelper.newRange(19, 17, 19, 19));
-    assertThat(diagnostics.get(3).getRange()).isEqualTo(RangeHelper.newRange(22, 20, 22, 22));
+    assertThat(diagnostics.get(2).getRange()).isEqualTo(RangeHelper.newRange(20, 21, 20, 23));
+    assertThat(diagnostics.get(3).getRange()).isEqualTo(RangeHelper.newRange(23, 24, 23, 26));
   }
 }

src/test/resources/diagnostics/MagicNumberDiagnostic.bsl

@@ -17,28 +17,35 @@
 
 КонецПроцедуры
 
-Если Описание2 > 11 Тогда // Ошибка на 11
+Процедура Б()
+    Если Описание2 > 11 Тогда // Ошибка на 11
 
-    Чтото = Чтото + 1; // Нет ошибки из-за исключения
-    Чтото = Чтото + 14; // Ошибка на 14
+        Чтото = Чтото + 1; // Нет ошибки из-за исключения
+        Чтото = Чтото + 14; // Ошибка на 14
 
-КонецЕсли;
+    КонецЕсли;
+
+    ЭтоВоскресенье = ДеньНедели = 7; // Тут ошибка, хоть и вглядит нормально.
+    ДеньНеделиВоскресенье = 7;
+    ЭтоВоскресенье = ДеньНедели = ДеньНеделиВоскресенье; // А вот тут уже ошибки нет
 
-ЭтоВоскресенье = ДеньНедели = 7; // Тут ошибка, хоть и вглядит нормально.
-ДеньНеделиВоскресенье = 7;
-ЭтоВоскресенье = ДеньНедели = ДеньНеделиВоскресенье; // А вот тут уже ошибки нет
+    // Далеше без ошибок
+    ПроверочноеПеречисление = Новый Массив;
+    ПроверочноеПеречисление.Добавить(1);
+    ПроверочноеПеречисление.Добавить(2);
+    ПроверочноеПеречисление.Добавить(3);
 
-// Далеше без ошибок
-ПроверочноеПеречисление = Новый Массив;
-ПроверочноеПеречисление.Добавить(1);
-ПроверочноеПеречисление.Добавить(2);
-ПроверочноеПеречисление.Добавить(3);
+    ПроверочнаяСтруктура = Новый Структура("Авто,ПростойВариант,СложныйВариант", 0, 1, 2);
+    ПроверочнаяСтруктура.Добавить("ЭкспертныйВариант", 3);
+
+КонецПроцедуры
 
-ПроверочнаяСтруктура = Новый Структура("Авто,ПростойВариант,СложныйВариант", 0, 1, 2);
-ПроверочнаяСтруктура.Добавить("ЭкспертныйВариант", 3);
+Процедура А(А = 566)
+
+КонецПроцедуры
 
 Функция КодОшибки()
-    
+
     Возврат 12;
 
 КонецФункции