Skip to content

Commit 2834c7e

Browse files
committed
Remove large test values that fail on 32-bit architectures
1 parent d296277 commit 2834c7e

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

api_tests/config_test.go

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,19 +43,12 @@ func Test_max_depth(t *testing.T) {
4343
{jsonDepth: 5, cfgMaxDepth: 6},
4444
{jsonDepth: 5, cfgMaxDepth: 5},
4545
{jsonDepth: 5, cfgMaxDepth: 4, expectedErr: "max depth"},
46-
// Now try some larger values to figure out the limit
46+
// Try a large depth without a limit
4747
{jsonDepth: 128000, cfgMaxDepth: -1},
48-
{jsonDepth: 512000, cfgMaxDepth: -1},
49-
{jsonDepth: 768000, cfgMaxDepth: -1},
50-
{jsonDepth: 860367, cfgMaxDepth: -1}, // largest value for jsoniter without stack overflow
5148
}
5249

5350
for _, test := range tests {
5451
t.Run(fmt.Sprintf("jsonDepth:%v_cfgMaxDepth:%v", test.jsonDepth, test.cfgMaxDepth), func(t *testing.T) {
55-
if testing.Short() && test.jsonDepth >= 512000 {
56-
t.Skip("skipping in -short due to large input data")
57-
}
58-
5952
should := require.New(t)
6053
cfg := jsoniter.Config{MaxDepth: test.cfgMaxDepth}.Froze()
6154

0 commit comments

Comments
 (0)