Skip to content

Commit a81981c

Browse files
committed
Fix test - Evict does not need a goroutine
1 parent a0106f5 commit a81981c

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

cache_test.go

+3-7
Original file line numberDiff line numberDiff line change
@@ -507,7 +507,7 @@ var _ = Describe("ranging over values", func() {
507507
Expect(c.Len()).To(Equal(1))
508508

509509
wg := sync.WaitGroup{}
510-
wg.Add(2)
510+
wg.Add(1)
511511
go func() {
512512
defer wg.Done()
513513
defer GinkgoRecover()
@@ -519,12 +519,8 @@ var _ = Describe("ranging over values", func() {
519519
_, closer, exists := c.Get(key)
520520
Expect(exists).To(BeTrue())
521521
closer.Close()
522-
// Must call Evict in a goroutine.
523-
go func() {
524-
// Must exclude concurrent Evict/Close.
525-
defer wg.Done()
526-
c.Evict("key")
527-
}()
522+
523+
c.Evict("key")
528524
Expect(<-evicted).To(Equal(key))
529525
Expect(c.Len()).To(BeZero())
530526
return true

0 commit comments

Comments
 (0)