Fix panic with the invalid SQL (#130)

Author: git-hulk

.github/workflows/ci.yaml

@@ -29,7 +29,7 @@ jobs:
           fetch-depth: '0'
 
       - name: Restore Go Module Cache
-        uses: actions/cache@v2
+        uses: actions/cache@v4
         with:
           path: ~/go/internal/mod
           key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}

parser/parser_common.go

@@ -274,7 +274,7 @@ func (p *Parser) parseLiteral(pos Pos) (Literal, error) {
 		// accept the NULL keyword
 		return &NullLiteral{NullPos: pos}, nil
 	default:
-		return nil, fmt.Errorf("expected <int>, <string> or keyword <NULL>, but got %q", p.last().Kind)
+		return nil, fmt.Errorf("expected <int>, <string> or keyword <NULL>, but got %q", p.lastTokenKind())
 	}
 }
 

parser/parser_query.go

@@ -214,7 +214,7 @@ func (p *Parser) parseJoinTableExpr(_ Pos) (Expr, error) {
 			StatementEnd: statementEnd,
 		}, nil
 	default:
-		return nil, fmt.Errorf("expected table name or subquery, got %s", p.last().Kind)
+		return nil, fmt.Errorf("expected table name or subquery, got %s", fmt.Sprintf("%v", p.lastTokenKind()))
 	}
 }
 

parser/parser_test.go

@@ -127,3 +127,14 @@ func validFormatSQL(t *testing.T, sql string) {
 	}
 	require.Equal(t, sql, builder.String())
 }
+
+func TestParser_InvalidSyntax(t *testing.T) {
+	invalidSQLs := []string{
+		"SELECT * FROM",
+	}
+	for _, sql := range invalidSQLs {
+		parser := NewParser(sql)
+		_, err := parser.ParseStmts()
+		require.Error(t, err)
+	}
+}