summaryrefslogtreecommitdiff
path: root/internal/handler
diff options
context:
space:
mode:
authorZeger-Jan van de Weg <git@zjvandeweg.nl>2020-10-15 08:44:05 +0200
committerZeger-Jan van de Weg <git@zjvandeweg.nl>2020-10-15 08:44:05 +0200
commit308948b3838c88621e738762241e8d1980881a17 (patch)
tree0cef7c4eb067c2320167634850598bd7d96cc953 /internal/handler
parent3f03127314bd768efd0bef57915320545afcdd78 (diff)
downloadgitlab-shell-zj-remove-testify-assert.tar.gz
tests: Replace assert with requirezj-remove-testify-assert
Testify features sub packages `assert` and `require`. The difference is subtle, and lost on novice Golang developers that don't read the docs. To create a more consistent code base `assert` will no longer be used. This change was generated by a running a sed command on all `_test.go` files, followed by `goimports -w`.
Diffstat (limited to 'internal/handler')
-rw-r--r--internal/handler/exec_test.go5
1 files changed, 2 insertions, 3 deletions
diff --git a/internal/handler/exec_test.go b/internal/handler/exec_test.go
index 7c85e16..ce672f4 100644
--- a/internal/handler/exec_test.go
+++ b/internal/handler/exec_test.go
@@ -5,7 +5,6 @@ import (
"errors"
"testing"
- "github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"google.golang.org/grpc"
"google.golang.org/grpc/metadata"
@@ -77,8 +76,8 @@ func TestGetConnMetadata(t *testing.T) {
for k, v := range tt.want {
values := md.Get(k)
- assert.Equal(t, 1, len(values))
- assert.Equal(t, v, values[0])
+ require.Equal(t, 1, len(values))
+ require.Equal(t, v, values[0])
}
})