Skip to content

Commit

Permalink
Incorporate feedback from PR
Browse files Browse the repository at this point in the history
- switch to using `assert.EqualError(..)`

Signed-off-by: Sukhil Suresh <ssuresh@pivotal.io>
  • Loading branch information
sukhil-suresh committed May 8, 2020
1 parent 09939ba commit 127e506
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/dockercreds/access_checker_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ func testAccessChecker(t *testing.T, when spec.G, it spec.S) {
hasAccess, err := HasWriteAccess(testKeychain{}, tagName)
require.Error(t, err)
expectedErrorMessage := fmt.Sprintf("Error validating write permission to %s. GET %s/v2/: unsupported status code 500", tagName, server.URL)
assert.Equal(t, expectedErrorMessage, err.Error())
assert.EqualError(t, err, expectedErrorMessage)
assert.False(t, hasAccess)
})
})
Expand Down

0 comments on commit 127e506

Please sign in to comment.