Skip to content

Commit 5682b86

Browse files
committed
responding to PR feedback
1 parent 20e2c61 commit 5682b86

File tree

1 file changed

+15
-4
lines changed

1 file changed

+15
-4
lines changed

test/unit/test_edge_config.py

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -235,10 +235,21 @@ def test_edge_endpoint_config_from_yaml_requires_exactly_one_input():
235235
EdgeEndpointConfig.from_yaml(filename=" ")
236236

237237

238-
def test_edge_endpoint_config_ignores_extra_top_level_fields():
239-
"""Unknown fields are silently ignored for forward compatibility (Postel's Law)."""
240-
config = EdgeEndpointConfig.model_validate({"global_config": {}, "unknown_field": True})
241-
assert config.global_config == GlobalConfig()
238+
def test_edge_endpoint_config_ignores_extra_fields_at_all_levels():
239+
"""Unknown fields are silently ignored at every nesting level for forward compatibility."""
240+
config = EdgeEndpointConfig.model_validate({
241+
"global_config": {"refresh_rate": 30.0, "unknown_global_field": "ignored"},
242+
"edge_inference_configs": {
243+
"default": {"enabled": True, "unknown_inference_field": 42},
244+
},
245+
"detectors": [
246+
{"detector_id": "det_1", "edge_inference_config": "default", "unknown_detector_field": [1, 2]},
247+
],
248+
"unknown_top_level_field": True,
249+
})
250+
assert config.global_config.refresh_rate == 30.0
251+
assert config.edge_inference_configs["default"].enabled is True
252+
assert config.detectors[0].detector_id == "det_1"
242253

243254

244255
def test_model_dump_shape_for_edge_endpoint_config():

0 commit comments

Comments
 (0)