Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix 'Error: The client request was invalid...' #65

Merged
merged 2 commits into from
Jan 3, 2023

Conversation

juandspy
Copy link
Collaborator

@juandspy juandspy commented Dec 20, 2022

Verify

Run make testacc with both the mocked server and the real server and see that TestAccRHOASKafka_Error and TestAccRHOASServiceAccount_Error have been fixed.

Explanation

These failures were introduced in 1a502b0. It was just a change in the error strings.

Fixes #66 too.

@jackdelahunt
Copy link
Contributor

LGTM 👍

@jackdelahunt jackdelahunt self-requested a review January 3, 2023 11:33
@jackdelahunt jackdelahunt merged commit 22ded9b into main Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TestAccRHOASTopic_Error produces a different error between the mock and real server
2 participants