From 706cc13b211e7c8745c47ee1e49b8e765634bb6b Mon Sep 17 00:00:00 2001 From: Rob Pike Date: Wed, 4 Feb 2015 13:50:08 -0800 Subject: [PATCH] cmd/pack: simplify the testing logic slightly Followup to https://go-review.googlesource.com/3910 We only need 1000 iteratinons. Change-Id: Ib63ae53105176abec77bad9609d638aeda7bcd61 Reviewed-on: https://go-review.googlesource.com/3901 Reviewed-by: Minux Ma Reviewed-by: Ian Lance Taylor --- src/cmd/pack/pack_test.go | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/cmd/pack/pack_test.go b/src/cmd/pack/pack_test.go index fbc6a3c698..0c58d628b4 100644 --- a/src/cmd/pack/pack_test.go +++ b/src/cmd/pack/pack_test.go @@ -257,15 +257,8 @@ func TestLargeDefs(t *testing.T) { } } - n := 10000 - if testing.Short() { - // Issue 9656: 10,000 is too aggressive for several - // builders, with ~120 MB of disk consumed. 1,000 is - // still enough to exercise the old bug. - n = 1000 - } printf("package large\n\ntype T struct {\n") - for i := 0; i < n; i++ { + for i := 0; i < 1000; i++ { printf("f%d int `tag:\"", i) for j := 0; j < 100; j++ { printf("t%d=%d,", j, j) -- 2.50.0