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
Test case for lowercase header #919
Test case for lowercase header #919
Conversation
// when | ||
HermesMessage message = hermesMessage(topic.getQualifiedName(), user.asBytes()) | ||
.withContentType(AVRO_BINARY) | ||
.withHeader("schema-version", "1") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will the test pass if you remove this header? hermes frontend uses latest schema version if no header is available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test fails if you remove it with error "Invalid message: Could not deserialize avro message with provided schema"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, and if you remove withSchemaVersionAwareSerialization()
when building topic as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same error like previously.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK thx :)
Test for checking if headers are case-insensitive.