feat(grammar): Improve support for TO_BINARY_FLOAT syntax

felipebz · Sep 27, 2024 · eab1962 · eab1962
1 parent de53cbd
commit eab1962
Show file tree

Hide file tree

Showing 5 changed files with 61 additions and 6 deletions.
diff --git a/zpa-checks/src/integrationTest/resources/expected/antlr-grammars-v4/ParsingErrorCheck.json b/zpa-checks/src/integrationTest/resources/expected/antlr-grammars-v4/ParsingErrorCheck.json
@@ -413,9 +413,6 @@
   "examples/single_statement.sql" : [
     1
   ],
-  "examples/to_binary_float.sql" : [
-    6
-  ],
   "examples/to_date.sql" : [
     10
   ],

diff --git a/zpa-checks/src/integrationTest/resources/expected/oracle-database_23/ParsingErrorCheck.json b/zpa-checks/src/integrationTest/resources/expected/oracle-database_23/ParsingErrorCheck.json
@@ -2858,9 +2858,6 @@
   "sqlrf/Syntax-for-Schema-Objects-and-Parts-in-SQL-Statements-4.sql" : [
     3
   ],
-  "sqlrf/TO_BINARY_FLOAT-1.sql" : [
-    2
-  ],
   "sqlrf/TO_DATE-3.sql" : [
     3
   ],

diff --git a/zpa-core/src/main/kotlin/org/sonar/plugins/plsqlopen/api/PlSqlKeyword.kt b/zpa-core/src/main/kotlin/org/sonar/plugins/plsqlopen/api/PlSqlKeyword.kt
@@ -516,6 +516,7 @@ enum class PlSqlKeyword(override val value: String, val isReserved: Boolean = fa
     TIME("time"),
     TIMESTAMP("timestamp"),
     TO_BINARY_DOUBLE("to_binary_double"),
+    TO_BINARY_FLOAT("to_binary_float"),
     TRAILING("trailing"),
     TRANSACTION("transaction"),
     TREAT("treat"),

diff --git a/zpa-core/src/main/kotlin/org/sonar/plugins/plsqlopen/api/SingleRowSqlFunctionsGrammar.kt b/zpa-core/src/main/kotlin/org/sonar/plugins/plsqlopen/api/SingleRowSqlFunctionsGrammar.kt
@@ -129,6 +129,7 @@ enum class SingleRowSqlFunctionsGrammar : GrammarRuleKey {
     SET_EXPRESSION,
     CAST_EXPRESSION,
     TO_BINARY_DOUBLE_EXPRESSION,
+    TO_BINARY_FLOAT_EXPRESSION,
     TRIM_EXPRESSION,
     TABLE_EXPRESSION,
     THE_EXPRESSION,
@@ -181,6 +182,7 @@ enum class SingleRowSqlFunctionsGrammar : GrammarRuleKey {
                     SET_EXPRESSION,
                     CAST_EXPRESSION,
                     TO_BINARY_DOUBLE_EXPRESSION,
+                    TO_BINARY_FLOAT_EXPRESSION,
                     TRIM_EXPRESSION,
                     TABLE_EXPRESSION,
                     THE_EXPRESSION,
@@ -228,6 +230,14 @@ enum class SingleRowSqlFunctionsGrammar : GrammarRuleKey {
                 RPARENTHESIS
             )
 
+            b.rule(TO_BINARY_FLOAT_EXPRESSION).define(
+                TO_BINARY_FLOAT, LPARENTHESIS,
+                EXPRESSION,
+                b.optional(DEFAULT_ON_ERROR_CLAUSE),
+                b.optional(COMMA, EXPRESSION, b.optional(COMMA, EXPRESSION)),
+                RPARENTHESIS
+            )
+
             b.rule(TABLE_EXPRESSION).define(
                 TABLE, LPARENTHESIS,
                 b.firstOf(DmlGrammar.SELECT_EXPRESSION, EXPRESSION),

diff --git a/...rc/test/kotlin/org/sonar/plugins/plsqlopen/api/expressions/ToBinaryFloatExpressionTest.kt b/...rc/test/kotlin/org/sonar/plugins/plsqlopen/api/expressions/ToBinaryFloatExpressionTest.kt
@@ -0,0 +1,50 @@
+/**
+ * Z PL/SQL Analyzer
+ * Copyright (C) 2015-2024 Felipe Zorzo
+ * mailto:felipe AT felipezorzo DOT com DOT br
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 3 of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with this program; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+ */
+package org.sonar.plugins.plsqlopen.api.expressions
+
+import com.felipebz.flr.tests.Assertions.assertThat
+import org.junit.jupiter.api.BeforeEach
+import org.junit.jupiter.api.Test
+import org.sonar.plugins.plsqlopen.api.PlSqlGrammar
+import org.sonar.plugins.plsqlopen.api.RuleTest
+
+class ToBinaryFloatExpressionTest : RuleTest() {
+
+    @BeforeEach
+    fun init() {
+        setRootRule(PlSqlGrammar.EXPRESSION)
+    }
+
+    @Test
+    fun matchesSimple() {
+        assertThat(p).matches("to_binary_float(foo)")
+    }
+
+    @Test
+    fun matchesDefaultClause() {
+        assertThat(p).matches("to_binary_float(foo default 0 on conversion error)")
+    }
+
+    @Test
+    fun matchesMultipleArguments() {
+        assertThat(p).matches("to_binary_float(foo, '9999', 'NLS_DATE_LANGUAGE = American')")
+    }
+
+}