Closed emmyoop closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.75%. Comparing base (
9a796aa
) to head (dd578bf
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Problem
Tests in tests/unit/test_semantic_layer_nodes_satisfy_protocols.py are a safety check. They mean that in some way, core is not satisfying the protocols that DSI provides. Unfortunately it's a bit harder to write a test to identify whether the failure to satisfy the protocol is breaking or not. Looking at this failure, it seems that we are not providing an optional key/value which is okay (not breaking).
Solution
Mark the failing test as skipped until we can come up with a more stable approach for these tests to not fail when the protocol doesn't match optional fields.
Checklist