Skip to content

Commit

Permalink
eth/downloader: throttling tests are time-sensitive, don't run parallel
Browse files Browse the repository at this point in the history
  • Loading branch information
karalabe committed Dec 30, 2015
1 parent dbbcf55 commit 1f39746
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions eth/downloader/downloader_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -720,8 +720,6 @@ func TestThrottling64Full(t *testing.T) { testThrottling(t, 64, FullSync) }
func TestThrottling64Fast(t *testing.T) { testThrottling(t, 64, FastSync) }

func testThrottling(t *testing.T, protocol int, mode SyncMode) {
t.Parallel()

// Create a long block chain to download and the tester
targetBlocks := 8 * blockCacheLimit
hashes, headers, blocks, receipts := makeChain(targetBlocks, 0, genesis, nil)
Expand Down Expand Up @@ -751,7 +749,7 @@ func testThrottling(t *testing.T, protocol int, mode SyncMode) {
}
// Wait a bit for sync to throttle itself
var cached, frozen int
for start := time.Now(); time.Since(start) < time.Second; {
for start := time.Now(); time.Since(start) < 3*time.Second; {
time.Sleep(25 * time.Millisecond)

tester.lock.Lock()
Expand Down

0 comments on commit 1f39746

Please sign in to comment.