| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361 |
- package repository
- import (
- "context"
- "io"
- "net/http"
- "net/http/httptest"
- "net/url"
- "testing"
- infraerrors "github.com/Wei-Shaw/sub2api/internal/pkg/errors"
- "github.com/Wei-Shaw/sub2api/internal/pkg/openai"
- "github.com/Wei-Shaw/sub2api/internal/service"
- "github.com/stretchr/testify/require"
- "github.com/stretchr/testify/suite"
- )
- type OpenAIOAuthServiceSuite struct {
- suite.Suite
- ctx context.Context
- srv *httptest.Server
- svc *openaiOAuthService
- received chan url.Values
- }
- func (s *OpenAIOAuthServiceSuite) SetupTest() {
- s.ctx = context.Background()
- s.received = make(chan url.Values, 1)
- }
- func (s *OpenAIOAuthServiceSuite) TearDownTest() {
- if s.srv != nil {
- s.srv.Close()
- s.srv = nil
- }
- }
- func (s *OpenAIOAuthServiceSuite) setupServer(handler http.HandlerFunc) {
- s.srv = newLocalTestServer(s.T(), handler)
- s.svc = &openaiOAuthService{tokenURL: s.srv.URL}
- }
- func (s *OpenAIOAuthServiceSuite) TestExchangeCode_DefaultRedirectURI() {
- errCh := make(chan string, 1)
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- if r.Method != http.MethodPost {
- errCh <- "method mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if err := r.ParseForm(); err != nil {
- errCh <- "ParseForm failed"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("grant_type"); got != "authorization_code" {
- errCh <- "grant_type mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("client_id"); got != openai.ClientID {
- errCh <- "client_id mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("code"); got != "code" {
- errCh <- "code mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("redirect_uri"); got != openai.DefaultRedirectURI {
- errCh <- "redirect_uri mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("code_verifier"); got != "ver" {
- errCh <- "code_verifier mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- wantUA, wantOriginator := service.CodexCanonicalAuthIdentity()
- if got := r.Header.Get("User-Agent"); got != wantUA {
- errCh <- "user-agent mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.Header.Get("originator"); got != wantOriginator {
- errCh <- "originator mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- w.Header().Set("Content-Type", "application/json")
- _, _ = io.WriteString(w, `{"access_token":"at","refresh_token":"rt","token_type":"bearer","expires_in":3600}`)
- }))
- resp, err := s.svc.ExchangeCode(s.ctx, "code", "ver", "", "", "")
- require.NoError(s.T(), err, "ExchangeCode")
- select {
- case msg := <-errCh:
- require.Fail(s.T(), msg)
- default:
- }
- require.Equal(s.T(), "at", resp.AccessToken)
- require.Equal(s.T(), "rt", resp.RefreshToken)
- }
- func (s *OpenAIOAuthServiceSuite) TestRefreshToken_FormFields() {
- errCh := make(chan string, 1)
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- if err := r.ParseForm(); err != nil {
- errCh <- "ParseForm failed"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("grant_type"); got != "refresh_token" {
- errCh <- "grant_type mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("refresh_token"); got != "rt" {
- errCh <- "refresh_token mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("client_id"); got != openai.ClientID {
- errCh <- "client_id mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.PostForm.Get("scope"); got != openai.RefreshScopes {
- errCh <- "scope mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- wantUA, wantOriginator := service.CodexCanonicalAuthIdentity()
- if got := r.Header.Get("User-Agent"); got != wantUA {
- errCh <- "user-agent mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- if got := r.Header.Get("originator"); got != wantOriginator {
- errCh <- "originator mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- w.Header().Set("Content-Type", "application/json")
- _, _ = io.WriteString(w, `{"access_token":"at2","refresh_token":"rt2","token_type":"bearer","expires_in":3600}`)
- }))
- resp, err := s.svc.RefreshToken(s.ctx, "rt", "")
- require.NoError(s.T(), err, "RefreshToken")
- select {
- case msg := <-errCh:
- require.Fail(s.T(), msg)
- default:
- }
- require.Equal(s.T(), "at2", resp.AccessToken)
- require.Equal(s.T(), "rt2", resp.RefreshToken)
- }
- // TestRefreshToken_DefaultsToOpenAIClientID 验证未指定 client_id 时默认使用 OpenAI ClientID,
- // 且只发送一次请求(不再盲猜多个 client_id)。
- func (s *OpenAIOAuthServiceSuite) TestRefreshToken_DefaultsToOpenAIClientID() {
- var seenClientIDs []string
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- if err := r.ParseForm(); err != nil {
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- clientID := r.PostForm.Get("client_id")
- seenClientIDs = append(seenClientIDs, clientID)
- w.Header().Set("Content-Type", "application/json")
- _, _ = io.WriteString(w, `{"access_token":"at","refresh_token":"rt","token_type":"bearer","expires_in":3600}`)
- }))
- resp, err := s.svc.RefreshToken(s.ctx, "rt", "")
- require.NoError(s.T(), err, "RefreshToken")
- require.Equal(s.T(), "at", resp.AccessToken)
- // 只发送了一次请求,使用默认的 OpenAI ClientID
- require.Equal(s.T(), []string{openai.ClientID}, seenClientIDs)
- }
- func (s *OpenAIOAuthServiceSuite) TestRefreshToken_UseProvidedClientID() {
- const customClientID = "custom-client-id"
- var seenClientIDs []string
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- if err := r.ParseForm(); err != nil {
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- clientID := r.PostForm.Get("client_id")
- seenClientIDs = append(seenClientIDs, clientID)
- if clientID != customClientID {
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- w.Header().Set("Content-Type", "application/json")
- _, _ = io.WriteString(w, `{"access_token":"at-custom","refresh_token":"rt-custom","token_type":"bearer","expires_in":3600}`)
- }))
- resp, err := s.svc.RefreshTokenWithClientID(s.ctx, "rt", "", customClientID)
- require.NoError(s.T(), err, "RefreshTokenWithClientID")
- require.Equal(s.T(), "at-custom", resp.AccessToken)
- require.Equal(s.T(), "rt-custom", resp.RefreshToken)
- require.Equal(s.T(), []string{customClientID}, seenClientIDs)
- }
- func (s *OpenAIOAuthServiceSuite) TestNonSuccessStatus_IncludesBody() {
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.WriteHeader(http.StatusBadRequest)
- _, _ = io.WriteString(w, "bad")
- }))
- _, err := s.svc.ExchangeCode(s.ctx, "code", "ver", openai.DefaultRedirectURI, "", "")
- require.Error(s.T(), err)
- require.ErrorContains(s.T(), err, "status 400")
- require.ErrorContains(s.T(), err, "bad")
- }
- func (s *OpenAIOAuthServiceSuite) TestRequestError_ClosedServer() {
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {}))
- s.srv.Close()
- _, err := s.svc.ExchangeCode(s.ctx, "code", "ver", openai.DefaultRedirectURI, "", "")
- require.Error(s.T(), err)
- require.ErrorContains(s.T(), err, "request failed")
- }
- func (s *OpenAIOAuthServiceSuite) TestExchangeCode_RequestErrorWithoutProxyReturnsProxyHint() {
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {}))
- s.srv.Close()
- _, err := s.svc.ExchangeCode(s.ctx, "code", "ver", openai.DefaultRedirectURI, "", "")
- require.Error(s.T(), err)
- require.Equal(s.T(), "OPENAI_OAUTH_PROXY_REQUIRED", infraerrors.Reason(err))
- require.Contains(s.T(), infraerrors.Message(err), "no proxy is configured")
- }
- func (s *OpenAIOAuthServiceSuite) TestContextCancel() {
- started := make(chan struct{})
- block := make(chan struct{})
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- close(started)
- <-block
- }))
- ctx, cancel := context.WithCancel(s.ctx)
- done := make(chan error, 1)
- go func() {
- _, err := s.svc.ExchangeCode(ctx, "code", "ver", openai.DefaultRedirectURI, "", "")
- done <- err
- }()
- <-started
- cancel()
- close(block)
- err := <-done
- require.Error(s.T(), err)
- }
- func (s *OpenAIOAuthServiceSuite) TestExchangeCode_UsesProvidedRedirectURI() {
- want := "http://localhost:9999/cb"
- errCh := make(chan string, 1)
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- _ = r.ParseForm()
- if got := r.PostForm.Get("redirect_uri"); got != want {
- errCh <- "redirect_uri mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- w.Header().Set("Content-Type", "application/json")
- _, _ = io.WriteString(w, `{"access_token":"at","token_type":"bearer","expires_in":1}`)
- }))
- _, err := s.svc.ExchangeCode(s.ctx, "code", "ver", want, "", "")
- require.NoError(s.T(), err, "ExchangeCode")
- select {
- case msg := <-errCh:
- require.Fail(s.T(), msg)
- default:
- }
- }
- func (s *OpenAIOAuthServiceSuite) TestExchangeCode_UseProvidedClientID() {
- wantClientID := "custom-exchange-client-id"
- errCh := make(chan string, 1)
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- _ = r.ParseForm()
- if got := r.PostForm.Get("client_id"); got != wantClientID {
- errCh <- "client_id mismatch"
- w.WriteHeader(http.StatusBadRequest)
- return
- }
- w.Header().Set("Content-Type", "application/json")
- _, _ = io.WriteString(w, `{"access_token":"at","token_type":"bearer","expires_in":1}`)
- }))
- _, err := s.svc.ExchangeCode(s.ctx, "code", "ver", openai.DefaultRedirectURI, "", wantClientID)
- require.NoError(s.T(), err, "ExchangeCode")
- select {
- case msg := <-errCh:
- require.Fail(s.T(), msg)
- default:
- }
- }
- func (s *OpenAIOAuthServiceSuite) TestTokenURL_CanBeOverriddenWithQuery() {
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- _ = r.ParseForm()
- s.received <- r.PostForm
- w.Header().Set("Content-Type", "application/json")
- _, _ = io.WriteString(w, `{"access_token":"at","token_type":"bearer","expires_in":1}`)
- }))
- s.svc.tokenURL = s.srv.URL + "?x=1"
- _, err := s.svc.ExchangeCode(s.ctx, "code", "ver", openai.DefaultRedirectURI, "", "")
- require.NoError(s.T(), err, "ExchangeCode")
- select {
- case <-s.received:
- default:
- require.Fail(s.T(), "expected server to receive request")
- }
- }
- func (s *OpenAIOAuthServiceSuite) TestExchangeCode_SuccessButInvalidJSON() {
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.Header().Set("Content-Type", "application/json")
- w.WriteHeader(http.StatusOK)
- _, _ = io.WriteString(w, "not-valid-json")
- }))
- _, err := s.svc.ExchangeCode(s.ctx, "code", "ver", openai.DefaultRedirectURI, "", "")
- require.Error(s.T(), err, "expected error for invalid JSON response")
- }
- func (s *OpenAIOAuthServiceSuite) TestRefreshToken_NonSuccessStatus() {
- s.setupServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
- w.WriteHeader(http.StatusUnauthorized)
- _, _ = io.WriteString(w, "unauthorized")
- }))
- _, err := s.svc.RefreshToken(s.ctx, "rt", "")
- require.Error(s.T(), err, "expected error for non-2xx status")
- require.ErrorContains(s.T(), err, "status 401")
- }
- func TestNewOpenAIOAuthClient_DefaultTokenURL(t *testing.T) {
- client := NewOpenAIOAuthClient()
- svc, ok := client.(*openaiOAuthService)
- require.True(t, ok)
- require.Equal(t, openai.TokenURL, svc.tokenURL)
- }
- func TestOpenAIOAuthServiceSuite(t *testing.T) {
- suite.Run(t, new(OpenAIOAuthServiceSuite))
- }
|