Skip to content

Commit 05a0a2a

Browse files
committed
test: add missing error handling
1 parent 40c82c7 commit 05a0a2a

File tree

1 file changed

+102
-34
lines changed

1 file changed

+102
-34
lines changed

structure/circularqueue/circularqueue_test.go

Lines changed: 102 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,21 @@ func TestCircularQueue(t *testing.T) {
1616
t.Errorf("Expected size: %v, got: %v\n", expectedSize, gotSize)
1717
}
1818

19-
queue.Enqueue(1)
20-
queue.Enqueue(2)
21-
queue.Enqueue(3)
22-
queue.Enqueue(4)
23-
queue.Enqueue(5)
19+
if err := queue.Enqueue(1); err != nil {
20+
t.Error(err)
21+
}
22+
if err := queue.Enqueue(2); err != nil {
23+
t.Error(err)
24+
}
25+
if err := queue.Enqueue(3); err != nil {
26+
t.Error(err)
27+
}
28+
if err := queue.Enqueue(4); err != nil {
29+
t.Error(err)
30+
}
31+
if err := queue.Enqueue(5); err != nil {
32+
t.Error(err)
33+
}
2434

2535
err = queue.Enqueue(6)
2636
if err == nil {
@@ -33,8 +43,12 @@ func TestCircularQueue(t *testing.T) {
3343
t.Errorf("Expected size: %v, got: %v\n", expectedSize, gotSize)
3444
}
3545

36-
queue.Dequeue()
37-
queue.Dequeue()
46+
if _, err := queue.Dequeue(); err != nil {
47+
t.Error(err)
48+
}
49+
if _, err := queue.Dequeue(); err != nil {
50+
t.Error(err)
51+
}
3852

3953
err = queue.Enqueue(6)
4054
if err != nil {
@@ -50,9 +64,15 @@ func TestCircularQueue(t *testing.T) {
5064
t.Run("Enqueue", func(t *testing.T) {
5165
queue, _ := NewCircularQueue[int](10)
5266

53-
queue.Enqueue(1)
54-
queue.Enqueue(2)
55-
queue.Enqueue(3)
67+
if err := queue.Enqueue(1); err != nil {
68+
t.Error(err)
69+
}
70+
if err := queue.Enqueue(2); err != nil {
71+
t.Error(err)
72+
}
73+
if err := queue.Enqueue(3); err != nil {
74+
t.Error(err)
75+
}
5676

5777
expected := 1
5878
got, err := queue.Peek()
@@ -66,9 +86,15 @@ func TestCircularQueue(t *testing.T) {
6686
t.Run("Dequeue", func(t *testing.T) {
6787
queue, _ := NewCircularQueue[string](10)
6888

69-
queue.Enqueue("one")
70-
queue.Enqueue("two")
71-
queue.Enqueue("three")
89+
if err := queue.Enqueue("one"); err != nil {
90+
t.Error(err)
91+
}
92+
if err := queue.Enqueue("two"); err != nil {
93+
t.Error(err)
94+
}
95+
if err := queue.Enqueue("three"); err != nil {
96+
t.Error(err)
97+
}
7298

7399
expected := "one"
74100
got, err := queue.Dequeue()
@@ -91,14 +117,30 @@ func TestCircularQueue(t *testing.T) {
91117
t.Run("Circularity", func(t *testing.T) {
92118
queue, _ := NewCircularQueue[int](10)
93119

94-
queue.Enqueue(1)
95-
queue.Enqueue(2)
96-
queue.Enqueue(3)
97-
queue.Dequeue()
98-
queue.Dequeue()
99-
queue.Enqueue(4)
100-
queue.Enqueue(5)
101-
queue.Dequeue()
120+
if err := queue.Enqueue(1); err != nil {
121+
t.Error(err)
122+
}
123+
if err := queue.Enqueue(2); err != nil {
124+
t.Error(err)
125+
}
126+
if err := queue.Enqueue(3); err != nil {
127+
t.Error(err)
128+
}
129+
if _, err := queue.Dequeue(); err != nil {
130+
t.Error(err)
131+
}
132+
if _, err := queue.Dequeue(); err != nil {
133+
t.Error(err)
134+
}
135+
if err := queue.Enqueue(4); err != nil {
136+
t.Error(err)
137+
}
138+
if err := queue.Enqueue(5); err != nil {
139+
t.Error(err)
140+
}
141+
if _, err := queue.Dequeue(); err != nil {
142+
t.Error(err)
143+
}
102144

103145
expected := 4
104146
got, err := queue.Peek()
@@ -111,17 +153,25 @@ func TestCircularQueue(t *testing.T) {
111153
})
112154
t.Run("IsFull", func(t *testing.T) {
113155
queue, _ := NewCircularQueue[bool](2)
114-
queue.Enqueue(false)
115-
queue.Enqueue(true)
156+
if err := queue.Enqueue(false); err != nil {
157+
t.Error(err)
158+
}
159+
if err := queue.Enqueue(true); err != nil {
160+
t.Error(err)
161+
}
116162

117163
expected := true
118164
got := queue.IsFull()
119165
if got != expected {
120166
t.Errorf("Expected: %v got: %v\n", expected, got)
121167
}
122168

123-
queue.Dequeue()
124-
queue.Dequeue()
169+
if _, err := queue.Dequeue(); err != nil {
170+
t.Error(err)
171+
}
172+
if _, err := queue.Dequeue(); err != nil {
173+
t.Error(err)
174+
}
125175

126176
expected = false
127177
got = queue.IsFull()
@@ -138,7 +188,9 @@ func TestCircularQueue(t *testing.T) {
138188
t.Errorf("Expected: %v got: %v\n", expected, got)
139189
}
140190

141-
queue.Enqueue(1.0)
191+
if err := queue.Enqueue(1.0); err != nil {
192+
t.Error(err)
193+
}
142194

143195
expected = false
144196
got = queue.IsEmpty()
@@ -150,9 +202,15 @@ func TestCircularQueue(t *testing.T) {
150202
t.Run("Peak", func(t *testing.T) {
151203
queue, _ := NewCircularQueue[rune](10)
152204

153-
queue.Enqueue('a')
154-
queue.Enqueue('b')
155-
queue.Enqueue('c')
205+
if err := queue.Enqueue('a'); err != nil {
206+
t.Error(err)
207+
}
208+
if err := queue.Enqueue('b'); err != nil {
209+
t.Error(err)
210+
}
211+
if err := queue.Enqueue('c'); err != nil {
212+
t.Error(err)
213+
}
156214

157215
expected := 'a'
158216
got, err := queue.Peek()
@@ -171,29 +229,39 @@ func BenchmarkCircularQueue(b *testing.B) {
171229
b.Run("Enqueue", func(b *testing.B) {
172230
queue, _ := NewCircularQueue[int](1000)
173231
for i := 0; i < b.N; i++ {
174-
queue.Enqueue(i)
232+
if err := queue.Enqueue(i); err != nil {
233+
b.Error(err)
234+
}
175235
}
176236
})
177237

178238
b.Run("Dequeue", func(b *testing.B) {
179239
queue, _ := NewCircularQueue[int](1000)
180240
for i := 0; i < 1000; i++ {
181-
queue.Enqueue(i)
241+
if err := queue.Enqueue(i); err != nil {
242+
b.Error(err)
243+
}
182244
}
183245
b.ResetTimer()
184246
for i := 0; i < b.N; i++ {
185-
queue.Dequeue()
247+
if _, err := queue.Dequeue(); err != nil {
248+
b.Error(err)
249+
}
186250
}
187251
})
188252

189253
b.Run("Peek", func(b *testing.B) {
190254
queue, _ := NewCircularQueue[int](1000)
191255
for i := 0; i < 1000; i++ {
192-
queue.Enqueue(i)
256+
if err := queue.Enqueue(i); err != nil {
257+
b.Error(err)
258+
}
193259
}
194260
b.ResetTimer()
195261
for i := 0; i < b.N; i++ {
196-
queue.Peek()
262+
if _, err := queue.Peek(); err != nil {
263+
b.Error(err)
264+
}
197265
}
198266
})
199267
}

0 commit comments

Comments
 (0)