From eaaeec5f1c745aaddb70b2b4112d64b8e25a3dd7 Mon Sep 17 00:00:00 2001 From: Andy Grunwald Date: Sun, 3 May 2020 15:45:24 +0200 Subject: [PATCH] chore(test): Remove unit testing log output for success cases (#293) During running the unit tests we see output like === RUN TestIssueService_DeleteAttachment TestIssueService_DeleteAttachment: issue_test.go:587: Attachment deleted TestIssueService_DeleteAttachment: issue_test.go:592: No error This log can be removed, because this mostly confirms the success case. The opposite case is throwing a testing error, which is visible as well. Hence, there is no need to log this messages durcing testing --- issue_test.go | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/issue_test.go b/issue_test.go index ba2526f1..3b562d48 100644 --- a/issue_test.go +++ b/issue_test.go @@ -583,13 +583,10 @@ func TestIssueService_DeleteAttachment(t *testing.T) { if resp.StatusCode == 404 { t.Error("Attachment not found") } - } else { - t.Log("Attachment deleted") } + if err != nil { t.Errorf("Error given: %s", err) - } else { - t.Log("No error") } }