oauth 和 openai 包的 SessionStore.Stop() 直接调用 close(stopCh), 重复调用会导致 panic。使用 sync.Once 包裹确保幂等安全。 新增单元测试覆盖连续调用和 50 goroutine 并发调用场景。 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
44 lines
588 B
Go
44 lines
588 B
Go
package oauth
|
|
|
|
import (
|
|
"sync"
|
|
"testing"
|
|
"time"
|
|
)
|
|
|
|
func TestSessionStore_Stop_Idempotent(t *testing.T) {
|
|
store := NewSessionStore()
|
|
|
|
store.Stop()
|
|
store.Stop()
|
|
|
|
select {
|
|
case <-store.stopCh:
|
|
// ok
|
|
case <-time.After(time.Second):
|
|
t.Fatal("stopCh 未关闭")
|
|
}
|
|
}
|
|
|
|
func TestSessionStore_Stop_Concurrent(t *testing.T) {
|
|
store := NewSessionStore()
|
|
|
|
var wg sync.WaitGroup
|
|
for range 50 {
|
|
wg.Add(1)
|
|
go func() {
|
|
defer wg.Done()
|
|
store.Stop()
|
|
}()
|
|
}
|
|
|
|
wg.Wait()
|
|
|
|
select {
|
|
case <-store.stopCh:
|
|
// ok
|
|
case <-time.After(time.Second):
|
|
t.Fatal("stopCh 未关闭")
|
|
}
|
|
}
|