fix(ts): fix end comma in constructor

phodal · Nov 15, 2022 · ddd139e · ddd139e
1 parent e87a90c
commit ddd139e
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 3 deletions.
diff --git a/chapi-ast-typescript/src/main/antlr/TypeScriptParser.g4 b/chapi-ast-typescript/src/main/antlr/TypeScriptParser.g4
@@ -680,7 +680,7 @@ iteratorDefinition
     ;
 
 formalParameterList
-    : formalParameterArg (',' formalParameterArg)* (',' lastFormalParameterArg)?
+    : formalParameterArg (',' formalParameterArg)* (',' lastFormalParameterArg)? ','?
     ;
 
 formalParameterArg

diff --git a/chapi-ast-typescript/src/test/kotlin/chapi/ast/typescriptast/TypeScriptAnalyserTest.kt b/chapi-ast-typescript/src/test/kotlin/chapi/ast/typescriptast/TypeScriptAnalyserTest.kt
@@ -2,6 +2,7 @@ package chapi.ast.typescriptast
 
 import chapi.domain.core.DataStructType
 import org.junit.jupiter.api.Assertions.assertEquals
+import org.junit.jupiter.api.Disabled
 import org.junit.jupiter.api.Test
 import java.io.File
 
@@ -69,7 +70,7 @@ class TypeScriptAnalyserTest {
     }
 
     @Test
-    @Ignore
+//    @Disabled
     fun someBug() {
         val dir = File("/Users/phodal/bug-ui-system")
         dir.walkTopDown().forEach {

diff --git a/...pi/ast/typescriptast/TypeScriptBugTest.kt → ...typescriptast/TypeScriptRegressionTest.kt b/...pi/ast/typescriptast/TypeScriptBugTest.kt → ...typescriptast/TypeScriptRegressionTest.kt
@@ -2,7 +2,7 @@ package chapi.ast.typescriptast
 
 import org.junit.jupiter.api.Test
 
-class TypeScriptBugTest {
+class TypeScriptRegressionTest {
 
     @Test
     internal fun backend_arrow_function() {
@@ -37,4 +37,14 @@ import {EMPTY, Observable, of} from 'rxjs';
 }"""
         TypeScriptAnalyser().analysis(code, "index.tsx")
     }
+
+    @Test
+    fun end_with_dot() {
+        val code = """export class SomeComponent implements OnInit, ControlValueAccessor {
+  constructor(
+    private fb: FormBuilder,
+  ) { }
+} """
+        TypeScriptAnalyser().analysis(code, "index.tsx")
+    }
 }