Revert "refactor(tidy): remove nested if and else block"

kujtimiihoxha created 1 month ago

This reverts commit a6e99fff52b4ab105df91f595a6e94ff3e8bd2cb.

Change summary

internal/config/config.go | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)

Detailed changes

internal/config/config.go

@@ -546,12 +546,15 @@ func (c *ProviderConfig) TestConnection(resolver VariableResolver) error {
 	if err != nil {
 		return fmt.Errorf("failed to create request for provider %s: %w", c.ID, err)
 	}
-	if c.ID == string(catwalk.InferenceProviderZAI) && b.StatusCode == http.StatusUnauthorized {
-		// for z.ai just check if the http response is not 401
-		return fmt.Errorf("failed to connect to provider %s: %s", c.ID, b.Status)
-	}
-	if b.StatusCode != http.StatusOK {
-		return fmt.Errorf("failed to connect to provider %s: %s", c.ID, b.Status)
+	if c.ID == string(catwalk.InferenceProviderZAI) {
+		if b.StatusCode == http.StatusUnauthorized {
+			// for z.ai just check if the http response is not 401
+			return fmt.Errorf("failed to connect to provider %s: %s", c.ID, b.Status)
+		}
+	} else {
+		if b.StatusCode != http.StatusOK {
+			return fmt.Errorf("failed to connect to provider %s: %s", c.ID, b.Status)
+		}
 	}
 	_ = b.Body.Close()
 	return nil