Skip to content

Track expect sequences by entity#33

Merged
bruth merged 1 commit intomainfrom
fix/model-mixed-entities
Dec 8, 2025
Merged

Track expect sequences by entity#33
bruth merged 1 commit intomainfrom
fix/model-mixed-entities

Conversation

@bruth
Copy link
Copy Markdown
Collaborator

@bruth bruth commented Dec 6, 2025

The previous assumption was that a single model would encapsulate the same entity type. This could technically be overridden by setting explicit Expect values on each event manually, but that would require external sequence tracking. Wrapping with Model can do this implicitly and now correctly tracks sequence by entity.

Note, memory growth will be proportionate to the number of entities being represented in the model.

Also, updated the test utility to always use a temp directory to ensure it is isolated and cleaned up properly.

The previous assumption was that a single model would
encapsulate the same entity type. This could technically
be overridden by setting explicit Expect values on each
event manually, but that would require external sequence
tracking. Wrapping with Model can do this implicitly and
now correctly tracks sequence by entity.

Note, memory growth will be proportionate to the number of
entities being represented in the model.

Signed-off-by: Byron Ruth <byron@synadia.com>
@bruth bruth requested a review from jordan-rash December 6, 2025 13:25
@bruth bruth merged commit d763b22 into main Dec 8, 2025
4 checks passed
@bruth bruth deleted the fix/model-mixed-entities branch December 8, 2025 21:58
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.

2 participants