Merge pull request #3728 from 1c-syntax/copilot/fix-nullpointerexception-issue

Fix NullPointerException in UselessTernaryOperatorDiagnostic for malformed ternary operators

Author: nixel2007

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

@@ -117,7 +117,7 @@ private String getAdaptedText(String text) {
   private static int getBooleanToken(BSLParser.ExpressionContext expCtx) {
 
     var tmpCtx = Optional.of(expCtx)
-      .filter(ctx -> ctx.children.size() == 1)
+      .filter(ctx -> ctx.getChildCount() == 1)
       .map(ctx -> ctx.member(0))
       .map(ctx -> ctx.getChild(0))
       .filter(BSLParser.ConstValueContext.class::isInstance)

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

@@ -29,6 +29,7 @@
 import java.util.List;
 
 import static com.github._1c_syntax.bsl.languageserver.util.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatCode;
 
 class UselessTernaryOperatorDiagnosticTest extends AbstractDiagnosticTest<UselessTernaryOperatorDiagnostic> {
 
@@ -83,4 +84,14 @@ void testQuickFix() {
       .hasNewText("НЕ (Б=0)");
   }
 
+  @Test
+  void testMalformedTernaryOperatorDoesNotThrowNPE() {
+    // Проверяем, что на некорректном синтаксисе не падает NullPointerException
+    // Пример из issue: Return ?(table.Count() = 1, undefined, );
+    var documentContext = getDocumentContext("UselessTernaryOperatorDiagnosticMalformed");
+    
+    assertThatCode(() -> getDiagnostics(documentContext))
+      .doesNotThrowAnyException();
+  }
+
 }

src/test/resources/diagnostics/UselessTernaryOperatorDiagnosticMalformed.bsl

@@ -0,0 +1,2 @@
+// Тест на некорректный синтаксис - не должен падать с NullPointerException
+Возврат ?(таблица.Количество() = 1, Неопределено, );