Skip to content

Commit

Permalink
fix: fix inconsistent tests
Browse files Browse the repository at this point in the history
  • Loading branch information
moul committed Oct 3, 2018
1 parent ecd6cd8 commit 088624d
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 1 deletion.
8 changes: 7 additions & 1 deletion core/test/app_mock.go
Expand Up @@ -3,6 +3,7 @@ package test
import (
"context"
"fmt"
"io"
"io/ioutil"
"net"
"strings"
Expand Down Expand Up @@ -125,8 +126,13 @@ func (a *AppMock) InitEventStream() error {
go func() {
for {
data, err := stream.Recv()
if err == io.EOF {
logger().Warn("eventstream EOF", zap.Error(err))
return
}
if err != nil {
logger().Debug("failed to receive stream data", zap.Error(err))
logger().Warn("failed to receive stream data", zap.Error(err))
return
}
a.eventStream <- data
}
Expand Down
13 changes: 13 additions & 0 deletions core/test/test_test.go
Expand Up @@ -62,6 +62,19 @@ func nodeChansLens(apps ...*AppMock) []int {
app.node.AsyncWait()
out = append(out, len(app.networkDriver.(*mock.Enqueuer).Queue()))
out = append(out, len(app.eventStream))
if len(app.eventStream) > 99 {
log.Println("!! QUEUE SHOULD NOT BE AS MUCH FILLED !!")
log.Println("--- queue")
for len(app.networkDriver.(*mock.Enqueuer).Queue()) > 0 {
event := <-app.networkDriver.(*mock.Enqueuer).Queue()
jsonPrint(event)
}
log.Println("--- eventStream")
for len(app.eventStream) > 0 {
event := <-app.eventStream
jsonPrint(event)
}
}
}
return out
}
Expand Down

0 comments on commit 088624d

Please sign in to comment.