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
dev: Fix schema validation in devserver #66715
Conversation
@@ -100,7 +100,7 @@ def get_test_stream_processor( | |||
max_poll_interval_ms=None, | |||
enable_dlq=False, | |||
healthcheck_file_path=None, | |||
validate_schema=False, | |||
enforce_schema=True, |
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.
We may as well always enforce this in tests
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #66715 +/- ##
========================================
Coverage 84.29% 84.30%
========================================
Files 5306 5308 +2
Lines 237154 237301 +147
Branches 41016 41043 +27
========================================
+ Hits 199916 200062 +146
- Misses 37019 37020 +1
Partials 219 219
|
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.
LGTM. There seem to be some tests that use invalid messages.
This ensures schema validation is run on every message on topics where a schema is registered in the devserver. The goal of this change is to help catch invalid messages in dev before changes are deployed.