From 6689ed458c7cb357392a476ca82420533ea61ab1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Roberto=20Abdelkader=20Mart=C3=ADnez=20P=C3=A9rez?= Date: Thu, 3 Oct 2019 14:06:04 +0200 Subject: [PATCH] Better names for tests --- http/reason_test.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/http/reason_test.go b/http/reason_test.go index 6a1521e..42d6388 100644 --- a/http/reason_test.go +++ b/http/reason_test.go @@ -5,35 +5,35 @@ import ( "testing" ) -func TestEmptyStringIsEmptyReason(t *testing.T) { +func TestEmptyReasonWhenEmptyString(t *testing.T) { r := &nethttp.Response{Status: ""} if GetReason(r) != "" { t.Errorf("We consider an empty status line to have an empty reason") } } -func TestOnlyCodeIsEmptyReason(t *testing.T) { +func TestEmptyReasonWhenOnlyCode(t *testing.T) { r := &nethttp.Response{Status: "200"} if GetReason(r) != "" { t.Errorf("We consider an status line with just the status code to have an empty reason") } } -func TestOnlyCodePlusSpaceIsEmptyReason(t *testing.T) { +func TestEmptyReasonWhenOnlyCodePlusSpace(t *testing.T) { r := &nethttp.Response{Status: "200 "} if GetReason(r) != "" { t.Errorf("We consider an status line with just the status code to have an empty reason") } } -func TestOneWordReason(t *testing.T) { +func TestReasonOfOneWord(t *testing.T) { r := &nethttp.Response{Status: "200 FOO"} if GetReason(r) != "FOO" { t.Errorf("Unexpected reason found") } } -func TestMultiWordReason(t *testing.T) { +func TestReasonOfMultipleWords(t *testing.T) { r := &nethttp.Response{Status: "200 FOO BAR BAZ"} if GetReason(r) != "FOO BAR BAZ" { t.Errorf("Unexpected reason found")