diff options
| author | Nick Thomas <nick@gitlab.com> | 2020-10-15 11:12:14 +0000 |
|---|---|---|
| committer | Nick Thomas <nick@gitlab.com> | 2020-10-15 11:12:14 +0000 |
| commit | 25305f74cc194e45e41974f0815b6f22c29e54dd (patch) | |
| tree | 0cef7c4eb067c2320167634850598bd7d96cc953 /client/httpsclient_test.go | |
| parent | d44dd84c568b840e98e1fbe434ea23cc752f0553 (diff) | |
| parent | ad56a97c108e1883ddfbc439af5d136ee6dc241b (diff) | |
| download | gitlab-shell-25305f74cc194e45e41974f0815b6f22c29e54dd.tar.gz | |
Merge branch 'zj-remove-testify-assert' into 'master'
tests: Remove testify assert in favour of require
See merge request gitlab-org/gitlab-shell!419
Diffstat (limited to 'client/httpsclient_test.go')
| -rw-r--r-- | client/httpsclient_test.go | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/client/httpsclient_test.go b/client/httpsclient_test.go index 0cf77e3..d76890b 100644 --- a/client/httpsclient_test.go +++ b/client/httpsclient_test.go @@ -8,7 +8,6 @@ import ( "path" "testing" - "github.com/stretchr/testify/assert" "github.com/stretchr/testify/require" "gitlab.com/gitlab-org/gitlab-shell/client/testserver" "gitlab.com/gitlab-org/gitlab-shell/internal/testhelper" @@ -51,8 +50,8 @@ func TestSuccessfulRequests(t *testing.T) { defer response.Body.Close() responseBody, err := ioutil.ReadAll(response.Body) - assert.NoError(t, err) - assert.Equal(t, string(responseBody), "Hello") + require.NoError(t, err) + require.Equal(t, string(responseBody), "Hello") }) } } @@ -84,7 +83,7 @@ func TestFailedRequests(t *testing.T) { _, err := client.Get(context.Background(), "/hello") require.Error(t, err) - assert.Equal(t, err.Error(), "Internal API unreachable") + require.Equal(t, err.Error(), "Internal API unreachable") }) } } |