Skip to content

Commit d9cd412

Browse files
authored
fix: Add missing error nil check to log msg (#73)
1 parent f92d500 commit d9cd412

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

internal/analysis/analysis.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -509,7 +509,9 @@ func (a *analysisOrchestrator) RunTest(ctx context.Context, orgId string, b bund
509509
parsedResponse, err := testApi.ParseCreateTestResponse(resp)
510510
defer func() {
511511
closeErr := resp.Body.Close()
512-
a.logger.Err(closeErr).Msg("failed to close response body")
512+
if closeErr != nil {
513+
a.logger.Err(closeErr).Msg("failed to close response body")
514+
}
513515
}()
514516
if err != nil {
515517
a.logger.Debug().Msg(err.Error())
@@ -574,7 +576,9 @@ func (a *analysisOrchestrator) retrieveTestURL(ctx context.Context, client *test
574576
}
575577
defer func() {
576578
closeErr := httpResponse.Body.Close()
577-
a.logger.Err(closeErr).Msg("failed to close response body")
579+
if closeErr != nil {
580+
a.logger.Err(closeErr).Msg("failed to close response body")
581+
}
578582
}()
579583

580584
parsedResponse, err := testApi.ParseGetTestResultResponse(httpResponse)

0 commit comments

Comments
 (0)