Merge pull request #4 from absolutscottie/comma-as-operator

Treat comma as an operation in ParseOp

Author: Dean Karn

CHANGELOG.md

@@ -6,7 +6,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-## [0.1.0] - 2022-06-08
+## [0.1.2] - 2022-06-08
+### Fixed
+- Handling of commas in arrays.
+
+## [0.1.1] - 2022-06-08
 ### Fixed
 - CLI output to be JSON.
 

README.md

@@ -1,6 +1,6 @@
 ksql
 =====
-![Project status](https://img.shields.io/badge/version-0.1.1-green.svg)
+![Project status](https://img.shields.io/badge/version-0.1.2-green.svg)
 [![GoDoc](https://godoc.org/github.com/go-playground/ksql?status.svg)](https://pkg.go.dev/github.com/go-playground/ksql)
 ![License](https://img.shields.io/dub/l/vibe-d.svg)
 

parser.go

@@ -316,7 +316,7 @@ func parseOp(value Expression, tokens []Token, pos *int) (Expression, error) {
 			return nil, errors.New("no value between ()")
 		}
 		return parseOp(op, tokens, pos)
-	case CloseParen, CloseBracket:
+	case CloseParen, CloseBracket, Comma:
 		return value, nil
 	default:
 		return nil, fmt.Errorf("invalid token after ident %d %s", tok.kind, tok.value)

parser_test.go

@@ -198,6 +198,18 @@ func TestParser(t *testing.T) {
 			src:      `{"field1":[]}`,
 			expected: false,
 		},
+		{
+			name: 		"ident IN value true",
+			exp:		`.field1 IN ["test"]`,
+			src: 		`{"field1":"test"}`,
+			expected:	true,
+		},
+		{
+			name: 		"ident IN value true multiple",
+			exp:		`.field1 IN ["test","foo","bar",]`,
+			src: 		`{"field1":"test"}`,
+			expected:	true,
+		},
 		{
 			name:     "array eq false",
 			exp:      `[] == ["test"]`,