fix a bug where AdvanceNext unlocks twice (#10)

edwingeng · edwingeng · web-flow · commit d4dbd838d090 · 2024-11-20T10:41:56.000+04:00
Co-authored-by: edwingeng &lt;edwingeng@outlook.com&gt;
diff --git a/mock.go b/mock.go
@@ -349,6 +349,7 @@ func (m *Mock) AdvanceNext() (time.Duration, AdvanceWaiter) {
 		defer close(w.ch)
 		defer m.mu.Unlock()
 		m.tb.Error("cannot AdvanceNext because there are no timers or tickers running")
+		return 0, w
 	}
 	d := m.nextTime.Sub(m.cur)
 	m.cur = m.nextTime

Original file line number	Diff line number	Diff line change
`@@ -349,6 +349,7 @@ func (m *Mock) AdvanceNext() (time.Duration, AdvanceWaiter) {`
`349`	`349`	`defer close(w.ch)`
`350`	`350`	`defer m.mu.Unlock()`
`351`	`351`	`m.tb.Error("cannot AdvanceNext because there are no timers or tickers running")`
	`352`	`+ return 0, w`
`352`	`353`	`}`
`353`	`354`	`d := m.nextTime.Sub(m.cur)`
`354`	`355`	`m.cur = m.nextTime`