ayush00git commented on issue #3614:
URL: https://github.com/apache/fory/issues/3614#issuecomment-4312406277
@chaokunyang that's the failing error -
```bash
❯ go test -v -count=1 -run
"TestRefResolver_OOBPanic|TestMetaStringResolver_NegativeIndexPanic" 2>&1 |
tail -40
/usr/local/go/src/testing/testing.go:2101 +0x4c5
Test: TestRefResolver_OOBPanic
Messages: RefResolver.GetReadObject should not panic
on OOB index
--- FAIL: TestRefResolver_OOBPanic (0.00s)
=== RUN TestMetaStringResolver_NegativeIndexPanic
security_test.go:57:
Error Trace: security_test.go:57
Error: func (assert.PanicTestFunc)(0x65e920) should
not panic
Panic value: runtime error: index
out of range [-1]
Panic stack: goroutine 8
[running]:
runtime/debug.Stack()
/usr/local/go/src/runtime/debug/stack.go:26 +0x5e
github.com/stretchr/testify/assert.didPanic.func1.1()
/home/ayush/go/pkg/mod/github.com/stretchr/[email protected]/assert/assertions.go:1013
+0x69
panic({0x775f20?, 0x2ce332042978?})
/usr/local/go/src/runtime/panic.go:860 +0x13a
github.com/apache/fory/go/fory.(*MetaStringResolver).ReadMetaStringBytes(0x2ce33203c3c0,
0x2ce3320bec80, 0x2ce33203c420)
/home/ayush/contributions/apache/fory/go/fory/meta_string_resolver.go:130 +0x827
github.com/apache/fory/go/fory.TestMetaStringResolver_NegativeIndexPanic.func1()
/home/ayush/contributions/apache/fory/go/fory/security_test.go:58 +0x28
github.com/stretchr/testify/assert.didPanic.func1(0x2ce3321ddda0?, 0x5cb44e?,
0x41e07f?, 0x73ce60?)
/home/ayush/go/pkg/mod/github.com/stretchr/[email protected]/assert/assertions.go:1018
+0x4c
github.com/stretchr/testify/assert.didPanic(0xa5a5c0?)
/home/ayush/go/pkg/mod/github.com/stretchr/[email protected]/assert/assertions.go:1020
+0x36
github.com/stretchr/testify/assert.NotPanics({0x7b7e40, 0x2ce3321a0908},
0x2ce3321ddd70, {0x2ce3320a5f48, 0x1, 0x1})
/home/ayush/go/pkg/mod/github.com/stretchr/[email protected]/assert/assertions.go:1091
+0x7e
github.com/stretchr/testify/require.NotPanics({0x7b8b68, 0x2ce3321a0908},
0x2ce3321ddd70, {0x2ce332094f48, 0x1, 0x1})
/home/ayush/go/pkg/mod/github.com/stretchr/[email protected]/require/require.go:1456
+0xa6
github.com/apache/fory/go/fory.TestMetaStringResolver_NegativeIndexPanic(0x2ce3321a0908)
/home/ayush/contributions/apache/fory/go/fory/security_test.go:57 +0x194
testing.tRunner(0x2ce3321a0908, 0x7b3580)
/usr/local/go/src/testing/testing.go:2036 +0xea
created by testing.(*T).Run in goroutine 1
/usr/local/go/src/testing/testing.go:2101 +0x4c5
Test: TestMetaStringResolver_NegativeIndexPanic
Messages: MetaStringResolver should not panic on
negative index
--- FAIL: TestMetaStringResolver_NegativeIndexPanic (0.00s)
FAIL
exit status 1
FAIL github.com/apache/fory/go/fory 0.004s
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
