Add integration tests for parse error handling #105

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.

Sign up for GitHub

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

Jump to bottom

Open

ormelaver-cyber wants to merge 2 commits into master from add-integration-tests-for-parsing-errors

Contributor

ormelaver-cyber commented Jan 29, 2026

Add integration tests for consumer parse error handling

Test callback is not invoked when JSON parsing fails
Test afterProcessMessage hook receives SyntaxError
Test messages are rejected without requeue (verified via messageCount)
Test valid messages continue processing after parse errors
Test empty string with JSON content type as edge case
Test RPC returns SyntaxError to producer on invalid request


          Add integration tests for parse error handling

ormelaver-cyber requested review from ramhr, shamil and yosiat as code owners

January 29, 2026 14:30


          fixed linting issues

8e7f6e0

ormelaver-cyber assigned yosiat, shamil and ramhr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

shamil Awaiting requested review from shamil shamil is a code owner

yosiat Awaiting requested review from yosiat yosiat is a code owner

ramhr Awaiting requested review from ramhr ramhr is a code owner

At least 2 approving reviews are required to merge this pull request.

Labels

None yet