Skip to content

Commit

Permalink
Merge branch 'crgimenes-fix-issue-60'
Browse files Browse the repository at this point in the history
* crgimenes-fix-issue-60:
  remove unnecessary goroutine and with that fix the possible data race condition in tests
  • Loading branch information
avelino committed Oct 30, 2018
2 parents b239dff + 7f46317 commit efeead9
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions subscription/subscription_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -17,12 +17,8 @@ func TestCreateSubscription(t *testing.T) {
}

func TestSubscription(t *testing.T) {
var first time.Time
go func() {
first = time.Now()
}()

sub := New().Subscribe()
first := sub.SubscribeAt
<-time.After(10 * time.Millisecond)
sub = sub.Unsubscribe()

Expand Down

0 comments on commit efeead9

Please sign in to comment.