Skip to content

Commit 6c8d77d

Browse files
committed
Fail test when encounter error
1 parent 98b7067 commit 6c8d77d

File tree

1 file changed

+19
-9
lines changed

1 file changed

+19
-9
lines changed

bus_test.go

+19-9
Original file line numberDiff line numberDiff line change
@@ -33,14 +33,16 @@ func TestUnsubscribe(t *testing.T) {
3333

3434
handler := func() {}
3535

36-
bus.Subscribe("test", handler)
36+
if err := bus.Subscribe("test", handler); err != nil {
37+
t.Fatal(err)
38+
}
3739

3840
if err := bus.Unsubscribe("test", handler); err != nil {
3941
fmt.Println(err)
4042
t.Fail()
4143
}
4244

43-
if err := bus.Unsubscribe("unexisted", func() {}); err == nil {
45+
if err := bus.Unsubscribe("non-existed", func() {}); err == nil {
4446
fmt.Println(err)
4547
t.Fail()
4648
}
@@ -51,7 +53,9 @@ func TestClose(t *testing.T) {
5153

5254
handler := func() {}
5355

54-
bus.Subscribe("test", handler)
56+
if err := bus.Subscribe("test", handler); err != nil {
57+
t.Fatal(err)
58+
}
5559

5660
original, ok := bus.(*messageBus)
5761
if !ok {
@@ -81,15 +85,19 @@ func TestPublish(t *testing.T) {
8185
first := false
8286
second := false
8387

84-
bus.Subscribe("topic", func(v bool) {
88+
if err := bus.Subscribe("topic", func(v bool) {
8589
defer wg.Done()
8690
first = v
87-
})
91+
}); err != nil {
92+
t.Fatal(err)
93+
}
8894

89-
bus.Subscribe("topic", func(v bool) {
95+
if err := bus.Subscribe("topic", func(v bool) {
9096
defer wg.Done()
9197
second = v
92-
})
98+
}); err != nil {
99+
t.Fatal(err)
100+
}
93101

94102
bus.Publish("topic", true)
95103

@@ -102,9 +110,11 @@ func TestPublish(t *testing.T) {
102110

103111
func TestHandleError(t *testing.T) {
104112
bus := New(runtime.NumCPU())
105-
bus.Subscribe("topic", func(out chan<- error) {
113+
if err := bus.Subscribe("topic", func(out chan<- error) {
106114
out <- errors.New("I do throw error")
107-
})
115+
}); err != nil {
116+
t.Fatal(err)
117+
}
108118

109119
out := make(chan error)
110120
defer close(out)

0 commit comments

Comments
 (0)