Merge pull request #31 from codecrafters-io/move-exception-capture

CC-1272 Move exception capture
codecrafters-io · Jun 14, 2024 · 24b35f8 · 24b35f8
2 parents e75d94a + 373f0a7
commit 24b35f8
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/internal/utils/codecrafters_client.go b/internal/utils/codecrafters_client.go
@@ -191,10 +191,7 @@ func (c CodecraftersClient) FetchBuild(buildId string) (FetchBuildStatusResponse
 			}
 
 			if fetchBuildResponse.Status != "failure" && fetchBuildResponse.Status != "success" {
-				err = fmt.Errorf("unexpected build status: %s", fetchBuildResponse.Status)
-
-				sentry.CaptureException(err)
-				return err
+				return fmt.Errorf("unexpected build status: %s", fetchBuildResponse.Status)
 			}
 
 			return nil
@@ -207,6 +204,10 @@ func (c CodecraftersClient) FetchBuild(buildId string) (FetchBuildStatusResponse
 	)
 
 	if err != nil {
+		if fetchBuildResponse.Status != "failure" && fetchBuildResponse.Status != "success" {
+			sentry.CaptureException(err)
+		}
+
 		return FetchBuildStatusResponse{}, err
 	}