refactor: improve error handling and increase test coverage

appleboy · appleboy · commit 9218f84bd1d7 · 2024-12-19T18:19:36.000+08:00
- Add the `fmt` package import
- Improve error message when failing to create a new HTTP client

Signed-off-by: appleboy &lt;appleboy.tw@gmail.com&gt;
diff --git a/provider/openai/openai.go b/provider/openai/openai.go
@@ -2,6 +2,7 @@ package openai
 
 import (
 	"context"
+	"fmt"
 	"regexp"
 
 	"github.com/appleboy/CodeGPT/core"
@@ -225,7 +226,7 @@ func New(opts ...Option) (*Client, error) {
 		proxy.WithHeaders(cfg.headers),
 	)
 	if err != nil {
-		return nil, err
+		return nil, fmt.Errorf("can't create a new HTTP client: %w", err)
 	}
 
 	// Set the OpenAI client to use the default configuration with Azure-specific options, if the provider is Azure.

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@ package openai`
`2`	`2`
`3`	`3`	`import (`
`4`	`4`	`"context"`
	`5`	`+ "fmt"`
`5`	`6`	`"regexp"`
`6`	`7`
`7`	`8`	`"github.com/appleboy/CodeGPT/core"`
`@@ -225,7 +226,7 @@ func New(opts ...Option) (*Client, error) {`
`225`	`226`	`proxy.WithHeaders(cfg.headers),`
`226`	`227`	`)`
`227`	`228`	`if err != nil {`
`228`		`- return nil, err`
	`229`	`+ return nil, fmt.Errorf("can't create a new HTTP client: %w", err)`
`229`	`230`	`}`
`230`	`231`
`231`	`232`	`// Set the OpenAI client to use the default configuration with Azure-specific options, if the provider is Azure.`