From: Rob Pike <r@golang.org>
Date: Wed, 4 Feb 2015 21:50:08 +0000 (-0800)
Subject: cmd/pack: simplify the testing logic slightly
X-Git-Tag: go1.5beta1~2132
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=706cc13b211e7c8745c47ee1e49b8e765634bb6b;p=gostls13.git

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 <minux@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
---

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)