Skip to content

Commit

Permalink
Ban require.NotEqualValues (ava-labs#1470)
Browse files Browse the repository at this point in the history
  • Loading branch information
dhrubabasu authored May 6, 2023
1 parent 8903335 commit 1b64bbe
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ linters-settings:
- 'require\.Error$(# ErrorIs should be used instead)?'
- 'require\.ErrorContains$(# ErrorIs should be used instead)?'
- 'require\.EqualValues$(# Equal should be used instead)?'
- 'require\.NotEqualValues$(# NotEqual should be used instead)?'
exclude_godoc_examples: false
# https://golangci-lint.run/usage/linters#gosec
gosec:
Expand Down
2 changes: 1 addition & 1 deletion network/peer/ip_signer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,5 +51,5 @@ func TestIPSigner(t *testing.T) {
require.NoError(err)
require.Equal(dynIP.IPPort(), signedIP3.IPPort)
require.Equal(uint64(11), signedIP3.Timestamp)
require.NotEqualValues(signedIP2.Signature, signedIP3.Signature)
require.NotEqual(signedIP2.Signature, signedIP3.Signature)
}
2 changes: 1 addition & 1 deletion utils/sampler/uniform_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ func TestSeeding(t *testing.T) {
s1.Seed(0)
v, err := s2.Next()
require.NoError(err)
require.NotEqualValues(s1Val, v)
require.NotEqual(s1Val, v)

s1.ClearSeed()

Expand Down

0 comments on commit 1b64bbe

Please sign in to comment.