Browse Source

fix misuse of break, clean up tests

Aneurin Barker Snook 1 year ago
parent
commit
a94c9f93d3
2 changed files with 13 additions and 1 deletions
  1. 1 0
      params_test.go
  2. 12 1
      query_test.go

+ 1 - 0
params_test.go

@@ -31,6 +31,7 @@ func TestReadParams(t *testing.T) {
 			t.Errorf("Expected empty slice, got nil")
 			continue
 		}
+
 		if len(params) != len(tc.Output) {
 			t.Errorf("Expected %d parameters", len(tc.Output))
 		}

+ 12 - 1
query_test.go

@@ -12,6 +12,17 @@ func TestQueryAppend(t *testing.T) {
 	}
 
 	testCases := []TestCase{
+		// Append without parameters
+		{
+			Input: NewQuery().
+				Append("FOR doc IN @@collection").
+				Append("FILTER doc.title == @title").
+				Append("RETURN doc"),
+			ExpectedStr: `FOR doc IN @@collection
+FILTER doc.title == @title
+RETURN doc`,
+			ExpectedParams: map[string]any{},
+		},
 		// Append with parameters
 		{
 			Input: NewQuery().
@@ -40,7 +51,7 @@ RETURN doc`,
 				"title":      "Spaghetti",
 			},
 		},
-		// Append and bind
+		// Append without parameters and bind after
 		{
 			Input: NewQuery().
 				Append("FOR doc IN @@collection").