Query with bracket notation and object literals (@ in attribute name) about ojg

[ohler55]

That is interesting. Sounds like a bug that will need to be fixed.

[hspaay]

Simple test example that shows this.
Query1 returns 1 item in the result. Query 2 returns no items in the result. The only difference is change from query of 'type' to '@type', both using the bracket notation.
Hope this helps.

func TestQuery2(t *testing.T) {
	store := make(map[string]interface{})

	jsondoc := `{
		"item1": {
			"id": "item1",
			"type": "type1",
			"@type": "attype1"
		},
		"item2": {
			"id": "item2",
			"type": "type2",
			"@type": "attype2"
		}
	}`
	query1 := `$[?(@['type']=="type1")]`           // bracket notation without object literal
	query2 := `$[?(@['@type']=="type1")]`          // with object literal 

	err := json.Unmarshal([]byte(jsondoc), &store)
	assert.NoError(t, err)

	jpExpr, err := jp.ParseString(query1)
	assert.NoError(t, err)
	result := jpExpr.Get(store)         // this returns one result as expected
	assert.NotEmpty(t, result)

	jpExpr, err = jp.ParseString(query2)
	assert.NoError(t, err)
	result = jpExpr.Get(store)
	assert.NotEmpty(t, result)      // this fails with empty result
}

[ohler55]

Thanks, that helps.

[ohler55]

I found the problem. query2 is checking for @type being equal to "type1". There is no match for that in the json doc. There is a match for "attype1" though. This worked for me:

func TestFilterAt(t *testing.T) {
	jsondoc := `{
			"item1": {
				"id": "item1",
				"type": "type1",
				"@type": "attype1"
			},
			"item2": {
				"id": "item2",
				"type": "type2",
				"@type": "attype2"
			}
		}`
	store := oj.MustParseString(jsondoc)
	x := jp.MustParseString(`$[?(@['@type']=="attype1")]`)
	result := x.Get(store)
	tt.Equal(t, 1, len(result))
}

That test has been added to the jp/get_test.go file in the project.

[hspaay]

Arggh, that was a silly mistake with my test cases.
Sorry for wasting your time and thank you for looking into it.

ps: thank you again for a great library!

[ohler55]

All good. It lets me know there are some users out there. :-)