Skip to content

Commit 51c269f

Browse files
authored
Merge pull request #477 from go-jet/sonic-bug-fix
Uncomment TestNorthwindJoinEverythingJsonSonic since sonic library bug is fixed now.
2 parents a8c468b + acd1aa2 commit 51c269f

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

tests/postgres/northwind_test.go

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -119,12 +119,11 @@ func BenchmarkTestNorthwindJoinEverythingSonicJson(b *testing.B) {
119119
})
120120
}
121121

122-
// uncomment when bug is fixed: https://github.com/bytedance/sonic/issues/774
123-
//func TestNorthwindJoinEverythingJsonSonic(t *testing.T) {
124-
// useJsonUnmarshalFunc(sonic.Unmarshal, func() {
125-
// testNorthwindJoinEverythingJson(t)
126-
// })
127-
//}
122+
func TestNorthwindJoinEverythingJsonSonic(t *testing.T) {
123+
useJsonUnmarshalFunc(sonic.Unmarshal, func() {
124+
testNorthwindJoinEverythingJson(t)
125+
})
126+
}
128127

129128
func testNorthwindJoinEverythingJson(t require.TestingT) {
130129

0 commit comments

Comments
 (0)