Skip to content

Commit

Permalink
chore: Clean up go warning
Browse files Browse the repository at this point in the history
Error strings should not be capitalized
Test variable should not be public as sarama only has one package
  • Loading branch information
Gorgonx7 committed Oct 16, 2023
1 parent a824226 commit bcfe70d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions broker_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ func TestBrokerFailedRequest(t *testing.T) {
}
}

var ErrTokenFailure = errors.New("Failure generating token")
var errTokenFailure = errors.New("failure generating token")

type TokenProvider struct {
accessToken *AccessToken
Expand Down Expand Up @@ -268,7 +268,7 @@ func TestSASLOAuthBearer(t *testing.T) {
mockSASLAuthResponse: NewMockSaslAuthenticateResponse(t),
expectClientErr: true,
expectedBrokerError: ErrNoError,
tokProvider: newTokenProvider(&AccessToken{Token: "access-token-123"}, ErrTokenFailure),
tokProvider: newTokenProvider(&AccessToken{Token: "access-token-123"}, errTokenFailure),
},
{
name: "SASL/OAUTHBEARER invalid extension",
Expand Down

0 comments on commit bcfe70d

Please sign in to comment.