| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968 |
- //go:build unit
- package service
- import (
- "context"
- "errors"
- "sync"
- "sync/atomic"
- "testing"
- "time"
- "github.com/stretchr/testify/require"
- )
- // openAITokenCacheStub implements OpenAITokenCache for testing
- type openAITokenCacheStub struct {
- mu sync.Mutex
- tokens map[string]string
- getErr error
- setErr error
- deleteErr error
- lockAcquired bool
- lockErr error
- releaseLockErr error
- getCalled int32
- setCalled int32
- lockCalled int32
- unlockCalled int32
- simulateLockRace bool
- }
- func newOpenAITokenCacheStub() *openAITokenCacheStub {
- return &openAITokenCacheStub{
- tokens: make(map[string]string),
- lockAcquired: true,
- }
- }
- func (s *openAITokenCacheStub) GetAccessToken(ctx context.Context, cacheKey string) (string, error) {
- atomic.AddInt32(&s.getCalled, 1)
- if s.getErr != nil {
- return "", s.getErr
- }
- s.mu.Lock()
- defer s.mu.Unlock()
- return s.tokens[cacheKey], nil
- }
- func (s *openAITokenCacheStub) SetAccessToken(ctx context.Context, cacheKey string, token string, ttl time.Duration) error {
- atomic.AddInt32(&s.setCalled, 1)
- if s.setErr != nil {
- return s.setErr
- }
- s.mu.Lock()
- defer s.mu.Unlock()
- s.tokens[cacheKey] = token
- return nil
- }
- func (s *openAITokenCacheStub) DeleteAccessToken(ctx context.Context, cacheKey string) error {
- if s.deleteErr != nil {
- return s.deleteErr
- }
- s.mu.Lock()
- defer s.mu.Unlock()
- delete(s.tokens, cacheKey)
- return nil
- }
- func (s *openAITokenCacheStub) AcquireRefreshLock(ctx context.Context, cacheKey string, ttl time.Duration) (bool, error) {
- atomic.AddInt32(&s.lockCalled, 1)
- if s.lockErr != nil {
- return false, s.lockErr
- }
- if s.simulateLockRace {
- return false, nil
- }
- return s.lockAcquired, nil
- }
- func (s *openAITokenCacheStub) ReleaseRefreshLock(ctx context.Context, cacheKey string) error {
- atomic.AddInt32(&s.unlockCalled, 1)
- return s.releaseLockErr
- }
- // openAIAccountRepoStub is a minimal stub implementing only the methods used by OpenAITokenProvider
- type openAIAccountRepoStub struct {
- account *Account
- getErr error
- updateErr error
- getCalled int32
- updateCalled int32
- }
- func (r *openAIAccountRepoStub) GetByID(ctx context.Context, id int64) (*Account, error) {
- atomic.AddInt32(&r.getCalled, 1)
- if r.getErr != nil {
- return nil, r.getErr
- }
- return r.account, nil
- }
- func (r *openAIAccountRepoStub) Update(ctx context.Context, account *Account) error {
- atomic.AddInt32(&r.updateCalled, 1)
- if r.updateErr != nil {
- return r.updateErr
- }
- r.account = account
- return nil
- }
- // openAIOAuthServiceStub implements OpenAIOAuthService methods for testing
- type openAIOAuthServiceStub struct {
- tokenInfo *OpenAITokenInfo
- refreshErr error
- refreshCalled int32
- }
- func (s *openAIOAuthServiceStub) RefreshAccountToken(ctx context.Context, account *Account) (*OpenAITokenInfo, error) {
- atomic.AddInt32(&s.refreshCalled, 1)
- if s.refreshErr != nil {
- return nil, s.refreshErr
- }
- return s.tokenInfo, nil
- }
- func (s *openAIOAuthServiceStub) BuildAccountCredentials(info *OpenAITokenInfo) map[string]any {
- now := time.Now()
- return map[string]any{
- "access_token": info.AccessToken,
- "refresh_token": info.RefreshToken,
- "expires_at": now.Add(time.Duration(info.ExpiresIn) * time.Second).Format(time.RFC3339),
- }
- }
- func TestOpenAITokenProvider_CacheHit(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- account := &Account{
- ID: 100,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "db-token",
- },
- }
- cacheKey := OpenAITokenCacheKey(account)
- cache.tokens[cacheKey] = "cached-token"
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "cached-token", token)
- require.Equal(t, int32(1), atomic.LoadInt32(&cache.getCalled))
- require.Equal(t, int32(0), atomic.LoadInt32(&cache.setCalled))
- }
- func TestOpenAITokenProvider_CacheMiss_FromCredentials(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- // Token expires in far future, no refresh needed
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 101,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "credential-token",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "credential-token", token)
- // Should have stored in cache
- cacheKey := OpenAITokenCacheKey(account)
- require.Equal(t, "credential-token", cache.tokens[cacheKey])
- }
- func TestOpenAITokenProvider_TokenRefresh(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- accountRepo := &openAIAccountRepoStub{}
- oauthService := &openAIOAuthServiceStub{
- tokenInfo: &OpenAITokenInfo{
- AccessToken: "refreshed-token",
- RefreshToken: "new-refresh-token",
- ExpiresIn: 3600,
- },
- }
- // Token expires soon (within refresh skew)
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 102,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "old-token",
- "refresh_token": "old-refresh-token",
- "expires_at": expiresAt,
- },
- }
- accountRepo.account = account
- // We need to directly test with the stub - create a custom provider
- customProvider := &testOpenAITokenProvider{
- accountRepo: accountRepo,
- tokenCache: cache,
- oauthService: oauthService,
- }
- token, err := customProvider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "refreshed-token", token)
- require.Equal(t, int32(1), atomic.LoadInt32(&oauthService.refreshCalled))
- }
- // testOpenAITokenProvider is a test version that uses the stub OAuth service
- type testOpenAITokenProvider struct {
- accountRepo *openAIAccountRepoStub
- tokenCache *openAITokenCacheStub
- oauthService *openAIOAuthServiceStub
- }
- func (p *testOpenAITokenProvider) GetAccessToken(ctx context.Context, account *Account) (string, error) {
- if account == nil {
- return "", errors.New("account is nil")
- }
- if account.Platform != PlatformOpenAI || account.Type != AccountTypeOAuth {
- return "", errors.New("not an openai oauth account")
- }
- cacheKey := OpenAITokenCacheKey(account)
- // 1. Check cache
- if p.tokenCache != nil {
- if token, err := p.tokenCache.GetAccessToken(ctx, cacheKey); err == nil && token != "" {
- return token, nil
- }
- }
- // 2. Check if refresh needed
- expiresAt := account.GetCredentialAsTime("expires_at")
- needsRefresh := expiresAt == nil || time.Until(*expiresAt) <= openAITokenRefreshSkew
- refreshFailed := false
- if needsRefresh && p.tokenCache != nil {
- locked, err := p.tokenCache.AcquireRefreshLock(ctx, cacheKey, 30*time.Second)
- if err == nil && locked {
- defer func() { _ = p.tokenCache.ReleaseRefreshLock(ctx, cacheKey) }()
- // Check cache again after acquiring lock
- if token, err := p.tokenCache.GetAccessToken(ctx, cacheKey); err == nil && token != "" {
- return token, nil
- }
- // Get fresh account from DB
- fresh, err := p.accountRepo.GetByID(ctx, account.ID)
- if err == nil && fresh != nil {
- account = fresh
- }
- expiresAt = account.GetCredentialAsTime("expires_at")
- if expiresAt == nil || time.Until(*expiresAt) <= openAITokenRefreshSkew {
- if p.oauthService == nil {
- refreshFailed = true // 无法刷新,标记失败
- } else {
- tokenInfo, err := p.oauthService.RefreshAccountToken(ctx, account)
- if err != nil {
- refreshFailed = true // 刷新失败,标记以使用短 TTL
- } else {
- newCredentials := p.oauthService.BuildAccountCredentials(tokenInfo)
- for k, v := range account.Credentials {
- if _, exists := newCredentials[k]; !exists {
- newCredentials[k] = v
- }
- }
- account.Credentials = newCredentials
- _ = p.accountRepo.Update(ctx, account)
- expiresAt = account.GetCredentialAsTime("expires_at")
- }
- }
- }
- } else if p.tokenCache.simulateLockRace {
- // Wait and retry cache
- time.Sleep(10 * time.Millisecond) // Short wait for test
- if token, err := p.tokenCache.GetAccessToken(ctx, cacheKey); err == nil && token != "" {
- return token, nil
- }
- }
- }
- accessToken := account.GetOpenAIAccessToken()
- if accessToken == "" {
- return "", errors.New("access_token not found in credentials")
- }
- // 3. Store in cache
- if p.tokenCache != nil {
- ttl := 30 * time.Minute
- if refreshFailed {
- ttl = time.Minute // 刷新失败时使用短 TTL
- } else if expiresAt != nil {
- until := time.Until(*expiresAt)
- if until > openAITokenCacheSkew {
- ttl = until - openAITokenCacheSkew
- } else if until > 0 {
- ttl = until
- } else {
- ttl = time.Minute
- }
- }
- _ = p.tokenCache.SetAccessToken(ctx, cacheKey, accessToken, ttl)
- }
- return accessToken, nil
- }
- func TestOpenAITokenProvider_LockRaceCondition(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.simulateLockRace = true
- accountRepo := &openAIAccountRepoStub{}
- // Token expires soon
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 103,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "race-token",
- "expires_at": expiresAt,
- },
- }
- accountRepo.account = account
- // Simulate another worker already refreshed and cached
- cacheKey := OpenAITokenCacheKey(account)
- go func() {
- time.Sleep(5 * time.Millisecond)
- cache.mu.Lock()
- cache.tokens[cacheKey] = "winner-token"
- cache.mu.Unlock()
- }()
- provider := &testOpenAITokenProvider{
- accountRepo: accountRepo,
- tokenCache: cache,
- }
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- // Should get the token set by the "winner" or the original
- require.NotEmpty(t, token)
- }
- func TestOpenAITokenProvider_NilAccount(t *testing.T) {
- provider := NewOpenAITokenProvider(nil, nil, nil)
- token, err := provider.GetAccessToken(context.Background(), nil)
- require.Error(t, err)
- require.Contains(t, err.Error(), "account is nil")
- require.Empty(t, token)
- }
- func TestOpenAITokenProvider_WrongPlatform(t *testing.T) {
- provider := NewOpenAITokenProvider(nil, nil, nil)
- account := &Account{
- ID: 104,
- Platform: PlatformGemini,
- Type: AccountTypeOAuth,
- }
- token, err := provider.GetAccessToken(context.Background(), account)
- require.Error(t, err)
- require.Contains(t, err.Error(), "not an openai oauth account")
- require.Empty(t, token)
- }
- func TestOpenAITokenProvider_WrongAccountType(t *testing.T) {
- provider := NewOpenAITokenProvider(nil, nil, nil)
- account := &Account{
- ID: 105,
- Platform: PlatformOpenAI,
- Type: AccountTypeAPIKey,
- }
- token, err := provider.GetAccessToken(context.Background(), account)
- require.Error(t, err)
- require.Contains(t, err.Error(), "not an openai oauth account")
- require.Empty(t, token)
- }
- func TestOpenAITokenProvider_NilCache(t *testing.T) {
- // Token doesn't need refresh
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 106,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "nocache-token",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, nil, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "nocache-token", token)
- }
- func TestOpenAITokenProvider_CacheGetError(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.getErr = errors.New("redis connection failed")
- // Token doesn't need refresh
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 107,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "fallback-token",
- "refresh_token": "refresh-token",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- // Should gracefully degrade and return from credentials
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "fallback-token", token)
- }
- func TestOpenAITokenProvider_CacheSetError(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.setErr = errors.New("redis write failed")
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 108,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "still-works-token",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- // Should still work even if cache set fails
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "still-works-token", token)
- }
- func TestOpenAITokenProvider_MissingAccessToken(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 109,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "expires_at": expiresAt,
- // missing access_token
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.Error(t, err)
- require.Contains(t, err.Error(), "access_token not found")
- require.Empty(t, token)
- }
- func TestOpenAITokenProvider_RefreshError(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- accountRepo := &openAIAccountRepoStub{}
- oauthService := &openAIOAuthServiceStub{
- refreshErr: errors.New("oauth refresh failed"),
- }
- // Token expires soon
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 110,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "old-token",
- "refresh_token": "old-refresh-token",
- "expires_at": expiresAt,
- },
- }
- accountRepo.account = account
- provider := &testOpenAITokenProvider{
- accountRepo: accountRepo,
- tokenCache: cache,
- oauthService: oauthService,
- }
- // Now with fallback behavior, should return existing token even if refresh fails
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "old-token", token) // Fallback to existing token
- }
- func TestOpenAITokenProvider_OAuthServiceNotConfigured(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- accountRepo := &openAIAccountRepoStub{}
- // Token expires soon
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 111,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "old-token",
- "expires_at": expiresAt,
- },
- }
- accountRepo.account = account
- provider := &testOpenAITokenProvider{
- accountRepo: accountRepo,
- tokenCache: cache,
- oauthService: nil, // not configured
- }
- // Now with fallback behavior, should return existing token even if oauth service not configured
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "old-token", token) // Fallback to existing token
- }
- func TestOpenAITokenProvider_TTLCalculation(t *testing.T) {
- tests := []struct {
- name string
- expiresIn time.Duration
- }{
- {
- name: "far_future_expiry",
- expiresIn: 1 * time.Hour,
- },
- {
- name: "medium_expiry",
- expiresIn: 10 * time.Minute,
- },
- {
- name: "near_expiry",
- expiresIn: 6 * time.Minute,
- },
- }
- for _, tt := range tests {
- t.Run(tt.name, func(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- expiresAt := time.Now().Add(tt.expiresIn).Format(time.RFC3339)
- account := &Account{
- ID: 200,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "test-token",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- _, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- // Verify token was cached
- cacheKey := OpenAITokenCacheKey(account)
- require.Equal(t, "test-token", cache.tokens[cacheKey])
- })
- }
- }
- func TestOpenAITokenProvider_DoubleCheckAfterLock(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- accountRepo := &openAIAccountRepoStub{}
- oauthService := &openAIOAuthServiceStub{
- tokenInfo: &OpenAITokenInfo{
- AccessToken: "refreshed-token",
- RefreshToken: "new-refresh",
- ExpiresIn: 3600,
- },
- }
- // Token expires soon
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 112,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "old-token",
- "expires_at": expiresAt,
- },
- }
- accountRepo.account = account
- cacheKey := OpenAITokenCacheKey(account)
- // Simulate: first GetAccessToken returns empty, but after lock acquired, cache has token
- originalGet := int32(0)
- cache.tokens[cacheKey] = "" // Empty initially
- provider := &testOpenAITokenProvider{
- accountRepo: accountRepo,
- tokenCache: cache,
- oauthService: oauthService,
- }
- // In a goroutine, set the cached token after a small delay (simulating race)
- go func() {
- time.Sleep(5 * time.Millisecond)
- cache.mu.Lock()
- cache.tokens[cacheKey] = "cached-by-other"
- cache.mu.Unlock()
- }()
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- // Should get either the refreshed token or the cached one
- require.NotEmpty(t, token)
- _ = originalGet // Suppress unused warning
- }
- // Tests for real provider - to increase coverage
- func TestOpenAITokenProvider_Real_LockFailedWait(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockAcquired = false // Lock acquisition fails
- // Token expires soon (within refresh skew) to trigger lock attempt
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 200,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "fallback-token",
- "refresh_token": "refresh-token",
- "expires_at": expiresAt,
- },
- }
- // Set token in cache after lock wait period (simulate other worker refreshing)
- cacheKey := OpenAITokenCacheKey(account)
- go func() {
- time.Sleep(100 * time.Millisecond)
- cache.mu.Lock()
- cache.tokens[cacheKey] = "refreshed-by-other"
- cache.mu.Unlock()
- }()
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- // Should get either the fallback token or the refreshed one
- require.NotEmpty(t, token)
- }
- func TestOpenAITokenProvider_Real_CacheHitAfterWait(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockAcquired = false // Lock acquisition fails
- // Token expires soon
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 201,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "original-token",
- "expires_at": expiresAt,
- },
- }
- cacheKey := OpenAITokenCacheKey(account)
- // Set token in cache immediately after wait starts
- go func() {
- time.Sleep(50 * time.Millisecond)
- cache.mu.Lock()
- cache.tokens[cacheKey] = "winner-token"
- cache.mu.Unlock()
- }()
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.NotEmpty(t, token)
- }
- func TestOpenAITokenProvider_Real_ExpiredWithoutRefreshToken(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockAcquired = false // Prevent entering refresh logic
- // Token with nil expires_at (no expiry set) - should use credentials
- account := &Account{
- ID: 202,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "no-expiry-token",
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- // Without OAuth service, refresh will fail but token should be returned from credentials
- require.NoError(t, err)
- require.Equal(t, "no-expiry-token", token)
- }
- func TestOpenAITokenProvider_Real_WhitespaceToken(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cacheKey := "openai:account:203"
- cache.tokens[cacheKey] = " " // Whitespace only - should be treated as empty
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 203,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "real-token",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "real-token", token) // Should fall back to credentials
- }
- func TestOpenAITokenProvider_Real_LockError(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockErr = errors.New("redis lock failed")
- // Token expires soon (within refresh skew)
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 204,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "fallback-on-lock-error",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "fallback-on-lock-error", token)
- }
- func TestOpenAITokenProvider_Real_WhitespaceCredentialToken(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 205,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": " ", // Whitespace only
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.Error(t, err)
- require.Contains(t, err.Error(), "access_token not found")
- require.Empty(t, token)
- }
- func TestOpenAITokenProvider_Real_NilCredentials(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- expiresAt := time.Now().Add(1 * time.Hour).Format(time.RFC3339)
- account := &Account{
- ID: 206,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "expires_at": expiresAt,
- // No access_token
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.Error(t, err)
- require.Contains(t, err.Error(), "access_token not found")
- require.Empty(t, token)
- }
- func TestOpenAITokenProvider_Real_LockRace_PollingHitsCache(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockAcquired = false // 模拟锁被其他 worker 持有
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 207,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "fallback-token",
- "refresh_token": "refresh-token",
- "expires_at": expiresAt,
- },
- }
- cacheKey := OpenAITokenCacheKey(account)
- go func() {
- time.Sleep(5 * time.Millisecond)
- cache.mu.Lock()
- cache.tokens[cacheKey] = "winner-token"
- cache.mu.Unlock()
- }()
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "winner-token", token)
- }
- func TestOpenAITokenProvider_Real_LockRace_ContextCanceled(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockAcquired = false // 模拟锁被其他 worker 持有
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 208,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "fallback-token",
- "refresh_token": "refresh-token",
- "expires_at": expiresAt,
- },
- }
- ctx, cancel := context.WithCancel(context.Background())
- cancel()
- provider := NewOpenAITokenProvider(nil, cache, nil)
- start := time.Now()
- token, err := provider.GetAccessToken(ctx, account)
- require.Error(t, err)
- require.ErrorIs(t, err, context.Canceled)
- require.Empty(t, token)
- require.Less(t, time.Since(start), 50*time.Millisecond)
- }
- func TestOpenAITokenProvider_RuntimeMetrics_LockWaitHitAndSnapshot(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockAcquired = false
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 209,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "fallback-token",
- "refresh_token": "refresh-token",
- "expires_at": expiresAt,
- },
- }
- cacheKey := OpenAITokenCacheKey(account)
- go func() {
- time.Sleep(10 * time.Millisecond)
- cache.mu.Lock()
- cache.tokens[cacheKey] = "winner-token"
- cache.mu.Unlock()
- }()
- provider := NewOpenAITokenProvider(nil, cache, nil)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- require.Equal(t, "winner-token", token)
- metrics := provider.SnapshotRuntimeMetrics()
- require.GreaterOrEqual(t, metrics.RefreshRequests, int64(1))
- require.GreaterOrEqual(t, metrics.LockContention, int64(1))
- require.GreaterOrEqual(t, metrics.LockWaitSamples, int64(1))
- require.GreaterOrEqual(t, metrics.LockWaitHit, int64(1))
- require.GreaterOrEqual(t, metrics.LockWaitTotalMs, int64(0))
- require.GreaterOrEqual(t, metrics.LastObservedUnixMs, int64(1))
- }
- func TestOpenAITokenProvider_RuntimeMetrics_LockAcquireFailure(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- cache.lockErr = errors.New("redis lock error")
- expiresAt := time.Now().Add(1 * time.Minute).Format(time.RFC3339)
- account := &Account{
- ID: 210,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "fallback-token",
- "refresh_token": "refresh-token",
- "expires_at": expiresAt,
- },
- }
- provider := NewOpenAITokenProvider(nil, cache, nil)
- _, err := provider.GetAccessToken(context.Background(), account)
- require.NoError(t, err)
- metrics := provider.SnapshotRuntimeMetrics()
- require.GreaterOrEqual(t, metrics.LockAcquireFailure, int64(1))
- require.GreaterOrEqual(t, metrics.RefreshRequests, int64(1))
- }
- func TestOpenAITokenProvider_NoRefreshTokenExpired_DisablesAccount(t *testing.T) {
- cache := newOpenAITokenCacheStub()
- repo := &rateLimitAccountRepoStub{}
- expiresAt := time.Now().Add(-time.Minute).UTC().Format(time.RFC3339)
- account := &Account{
- ID: 2881,
- Platform: PlatformOpenAI,
- Type: AccountTypeOAuth,
- Credentials: map[string]any{
- "access_token": "expired-access-token",
- "expires_at": expiresAt,
- },
- }
- cacheKey := OpenAITokenCacheKey(account)
- cache.tokens[cacheKey] = "stale-cached-token"
- // Force the provider past the cache hit branch.
- cache.getErr = errors.New("simulated cache miss")
- provider := NewOpenAITokenProvider(repo, cache, nil)
- blocker := &runtimeBlockRecorder{}
- provider.SetAccountRuntimeBlocker(blocker)
- token, err := provider.GetAccessToken(context.Background(), account)
- require.Error(t, err)
- require.Empty(t, token)
- require.Contains(t, err.Error(), "refresh_token is missing")
- require.Equal(t, 1, repo.setErrorCalls, "account should be disabled via SetError exactly once")
- require.Contains(t, repo.lastErrorMsg, "refresh_token is missing")
- require.Len(t, blocker.accounts, 1)
- require.Equal(t, account.ID, blocker.accounts[0].ID)
- require.Equal(t, "missing_refresh_token", blocker.reasons[0])
- }
|