Fix/realtime test compatibility serialization and tripupdates s14 s16 #15
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
During the implementation of Sub-issue #16, the additions to the Realtime workflow—particularly the ETA module, synthetic fixtures, and the Bytewax-style builder—introduced new imports and helper functions inside the codebase.
These changes affected the structure and expectations of the testing environment, and as a result, some of the unit tests from Sub-issue 14 began failing. The issues were not related to the core logic itself, but rather to how the updated modules were being imported and referenced inside
test_realtime.py.To resolve this,
test_realtime.pywas updated to restore full compatibility with both sub-issues.The file was adjusted to correctly reference the modified Realtime utilities, align imports with the new module structure, and ensure that the serialization tests (Sub-issue 14) and the ETA/Bytewax builder tests (Sub-issue #16) could coexist without interfering with each other.
After these fixes, all tests run successfully, but each sub-issue now requires invoking its dedicated test functions explicitly:
Sub-issue 14 — Feed Serialization Tests
Sub-issue 16 — ETA Module / Bytewax TripUpdates Builder