vlogscli/vlogsgenerator/apptest: minor changes to HTTP-related code paths (#567)

Co-authored-by: Aliaksandr Valialkin <[email protected]>

Author: func25

app/vlogscli/main.go

@@ -423,6 +423,7 @@ func getQueryResponse(ctx context.Context, output io.Writer, qStr string, output
 			body = []byte(fmt.Sprintf("cannot read response body: %s", err))
 		}
 		fmt.Fprintf(output, "unexpected status code: %d; response body:\n%s\n", resp.StatusCode, body)
+		_ = resp.Body.Close()
 		return nil
 	}
 

app/vlogsgenerator/main.go

@@ -209,7 +209,7 @@ func generateAndPushLogs(cfg *workerConfig, workerID int) {
 	defer resp.Body.Close()
 
 	if resp.StatusCode/100 != 2 {
-		logger.Fatalf("unexpected status code got from %q: %d; want 2xx", cfg.url, err)
+		logger.Fatalf("unexpected status code got from %q: %d; want 2xx", cfg.url, resp.StatusCode)
 	}
 
 	// Wait until all the generateLogs goroutine is finished.

apptest/client.go

@@ -109,7 +109,7 @@ func readAllAndClose(t *testing.T, responseBody io.ReadCloser) string {
 	defer responseBody.Close()
 	b, err := io.ReadAll(responseBody)
 	if err != nil {
-		t.Fatalf("could not read response body: %d", err)
+		t.Fatalf("could not read response body: %s", err)
 	}
 	return string(b)
 }