parser: fix create table without name (#52)

* parser: fix create table without name #29

* test: negative test cases for parser #29 

Co-authored-by: Tim Satke <[email protected]>

internal/parser/issue29_test.go

@@ -0,0 +1,31 @@
+package parser
+
+import "testing"
+
+func TestIssue29(t *testing.T)  {
+	NegativeTest{
+		Name:  "issue29",
+		Query: "CREATE TABLE(n,FOREIGN KEY()REFERENCES n ON DELETE CASCADE)",
+	}.Run(t)
+}
+
+func TestIssue29WithoutTableName(t *testing.T) {
+	NegativeTest{
+		Name:  "issue29 without table name",
+		Query: "CREATE TABLE (foo)",
+	}.Run(t)
+}
+
+func TestIssue29WithNumericColumn(t *testing.T)  {
+	NegativeTest{
+		Name:  "issue29 with numeric column",
+		Query: "CREATE TABLE foo(1)",
+	}.Run(t)
+}
+
+func TestIssue29WithNumericTable(t *testing.T)  {
+	NegativeTest{
+		Name:  "issue29 with numeric column",
+		Query: "CREATE TABLE 1(foo)",
+	}.Run(t)
+}

internal/parser/negative_test.go

@@ -0,0 +1,27 @@
+package parser
+
+import (
+	"github.com/stretchr/testify/assert"
+	"testing"
+)
+
+type NegativeTest struct {
+	Name  string
+	Query string
+}
+
+func (nt NegativeTest) Run(t *testing.T) {
+	t.Helper()
+	t.Run(nt.Name, func(t *testing.T) {
+		assert := assert.New(t)
+
+		p, err := New(nt.Query)
+		assert.NoError(err)
+		_, errs, ok := p.Next()
+		assert.True(ok, "expected exactly one statement")
+		assert.NotEmpty(errs, "expected errors")
+
+		_, _, ok = p.Next()
+		assert.False(ok, "expected only one statement")
+	})
+}

internal/parser/simple_parser_rules.go

@@ -2532,6 +2532,8 @@ func (p *simpleParser) parseCreateTableStmt(createToken, tempToken, temporaryTok
 		stmt.SchemaName = next
 		stmt.TableName = next
 		p.consumeToken()
+	} else {
+		r.unexpectedToken(token.Literal)
 	}
 
 	next, ok = p.lookahead(r)