Skip to content

Commit

Permalink
Avoid nil pointer dereference in assertGetUserByClaimResponses
Browse files Browse the repository at this point in the history
Update assertGetUserByClaimResponses integration test to verify that
user is not nil.
  • Loading branch information
sudo-sturbia committed Mar 23, 2021
1 parent e685753 commit 999730a
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tests/integration/grpc/userprovider_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,7 @@ var _ = Describe("user providers", func() {
for claim, value := range tests {
user, err := serviceClient.GetUserByClaim(ctx, &userpb.GetUserByClaimRequest{Claim: claim, Value: value})
Expect(err).ToNot(HaveOccurred())
Expect(user.User).ToNot(BeNil())
Expect(user.User.Mail).To(Equal("einstein@example.org"))
}
})
Expand Down

0 comments on commit 999730a

Please sign in to comment.