Merge pull request #10 from muonsoft/fix-panic

fix panic and add debug helper

.golangci.yml

@@ -23,7 +23,6 @@ linters:
     - gocyclo
     - godot
     - godox
-    - goerr113
     - gofmt
     - gofumpt
     - goimports

README.md

@@ -243,6 +243,9 @@ func TestYourAPI(t *testing.T) {
         assert.Equal(t, "2022-10-16T15:14:32+03:00", json.Node("time").Time().Format(time.RFC3339))
         assert.Equal(t, "23e98a0c-26c8-410f-978f-d1d67228af87", json.Node("uuid").IsUUID().Value().String())
         assert.Equal(t, "23e98a0c-26c8-410f-978f-d1d67228af87", json.Node("uuid").UUID().String())
+
+        // debug helpers
+        json.Node("bookstore", "books", 1).Print()
     })
 }
 ```

assertjson/assertjson.go

@@ -18,6 +18,7 @@ import (
 type TestingT interface {
 	Helper()
 	Errorf(format string, args ...interface{})
+	Log(args ...interface{})
 }
 
 // AssertJSON - main structure that holds parsed JSON.
@@ -163,6 +164,12 @@ func pathFromJSONPointer(p string) []interface{} {
 func getValueByPath(data interface{}, path ...interface{}) (interface{}, error) {
 	v := jsoniter.Wrap(data)
 	for _, e := range path {
+		if _, ok := e.(int); ok {
+			if _, ok := v.GetInterface().(map[string]interface{}); ok {
+				return nil, fmt.Errorf("value of type int is not assignable to type string")
+			}
+		}
+
 		v = v.Get(e)
 		if v.LastError() != nil {
 			return nil, v.LastError()

assertjson/assertjson_test.go

@@ -226,6 +226,9 @@ func TestFileHas(t *testing.T) {
 				}).
 				Raw,
 		)
+
+		// debug helpers
+		json.Node("bookstore", "books", 1).Print()
 	})
 }
 
@@ -2591,6 +2594,37 @@ func TestHas(t *testing.T) {
 				`failed asserting that JSON node "a.b.c[0]": equal to "expected", actual is "value"`,
 			},
 		},
+		{
+			name: "seek by json iterator",
+			json: `{"a": {"b": {"c": ["value"]}}}`,
+			assert: func(json *assertjson.AssertJSON) {
+				json.Node("a", 0).IsString()
+			},
+			wantMessages: []string{
+				`failed to find JSON node "a[0]": value of type int is not assignable to type string`,
+			},
+		},
+		{
+			name: "seek by json iterator",
+			json: `{"a": [{"b": 1}]}`,
+			assert: func(json *assertjson.AssertJSON) {
+				json.Node("a", "b").IsString()
+			},
+			wantMessages: []string{
+				`failed to find JSON node "a.b": [b] not found`,
+			},
+		},
+		// debug helpers
+		{
+			name: "print json node",
+			json: `{"a": {"b": {"c": ["value"]}}}`,
+			assert: func(json *assertjson.AssertJSON) {
+				json.Node("a", "b").Print()
+			},
+			wantMessages: []string{
+				`JSON node at "a.b"`,
+			},
+		},
 		// deprecated behaviour: seek by json pointer path
 		{
 			name: "deprecated: json pointer path",

assertjson/node.go

@@ -141,6 +141,12 @@ func (node *AssertNode) Assert(jsonAssert JSONAssertFunc) {
 	})
 }
 
+func (node *AssertNode) Print() {
+	node.t.Helper()
+	printableJSON, _ := json.MarshalIndent(node.value, "", "\t")
+	node.t.Log(fmt.Sprintf("JSON node at \"%s\":\n", node.path.String()), string(printableJSON))
+}
+
 func (node *AssertNode) fail(message string, msgAndArgs ...interface{}) {
 	node.t.Helper()
 	assert.Fail(node.t, node.message+message, msgAndArgs...)

internal/mock/tester.go

@@ -24,6 +24,10 @@ func (tester *Tester) Fatal(args ...interface{}) {
 	tester.messages = append(tester.messages, fmt.Sprint(args...))
 }
 
+func (tester *Tester) Log(args ...interface{}) {
+	tester.messages = append(tester.messages, fmt.Sprint(args...))
+}
+
 func (tester *Tester) AssertContains(t *testing.T, messages []string) {
 	t.Helper()
 	if len(tester.messages) != len(messages) {