Browse Source

FIX: waiting for worker fireup was making the test slow

Dodging the race condition with a sleep was fine, but we slept
for a whole second which was longer than this test takes to run!

I've scaled the sleep down to 250 miliseconds and increased the tries.

Even this is probably excessively long but now the test is fast agin.
tags/0.2
Joe Higton 10 years ago
parent
commit
912db95c0a
1 changed files with 2 additions and 2 deletions
  1. +2
    -2
      worker/worker_test.go

+ 2
- 2
worker/worker_test.go View File

@@ -115,7 +115,7 @@ func TestWorkWithoutReady(t * testing.T){
// With the all-in-one Work() we don't know if the
// worker is ready at this stage so we may have to wait a sec:
go func(){
tries := 3
tries := 5
for( tries > 0 ){
if other_worker.ready {
other_worker.Echo([]byte("Hello"))
@@ -123,7 +123,7 @@ func TestWorkWithoutReady(t * testing.T){
}

// still waiting for it to be ready..
time.Sleep(1 * time.Second)
time.Sleep(250 * time.Millisecond)
tries--
}
}()


Loading…
Cancel
Save