Skip to content

Commit

Permalink
fix: Define a constant instead of duplicating this literal "Bearer {{…
Browse files Browse the repository at this point in the history
….login.data.access_token}}" 3 times. (#62)
wongearl authored May 8, 2023
1 parent 6d58233 commit bb74504
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions pkg/server/remote_server_test.go
Original file line number Diff line number Diff line change
@@ -12,6 +12,10 @@ import (
"github.com/stretchr/testify/assert"
)

const (
BearerToken = "Bearer {{.login.data.access_token}}"
)

func TestRemoteServer(t *testing.T) {
server := NewRemoteServer()
_, err := server.Run(context.TODO(), &TestTask{
@@ -70,7 +74,7 @@ func TestFindParentTestCases(t *testing.T) {
testcase: &atesting.TestCase{
Request: atesting.Request{
Header: map[string]string{
"Authorization": "Bearer {{.login.data.access_token}}",
"Authorization": BearerToken,
},
},
},
@@ -114,7 +118,7 @@ func TestFindParentTestCases(t *testing.T) {
Request: atesting.Request{
API: "/users/{{(index .users 0).name}}{{randomKubernetesName}}",
Header: map[string]string{
"Authorization": "Bearer {{.login.data.access_token}}",
"Authorization": BearerToken,
},
},
},
@@ -131,7 +135,7 @@ func TestFindParentTestCases(t *testing.T) {
Request: atesting.Request{
API: "/users/{{(index .users 0).name}}",
Header: map[string]string{
"Authorization": "Bearer {{.login.data.access_token}}",
"Authorization": BearerToken,
},
},
}},

0 comments on commit bb74504

Please sign in to comment.