From 326d664d3e27a0d1cec23b41ab428d5977f1239e Mon Sep 17 00:00:00 2001 From: woutdenolf Date: Tue, 6 Oct 2026 14:46:17 +0200 Subject: [PATCH] Log and emit a warning when merged workflow outputs have name collisions --- CHANGELOG.md | 4 ++++ src/ewoksppf/tests/test_workflow_outputs.py | 5 +++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 7435391..ed33044 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ## [Unreleased] +### Changed + +- Log and emit a warning when merged workflow outputs have name collisions. + ## [3.1.0rc2] - 2026-10-01 ### Added diff --git a/src/ewoksppf/tests/test_workflow_outputs.py b/src/ewoksppf/tests/test_workflow_outputs.py index e2a1007..4be946d 100644 --- a/src/ewoksppf/tests/test_workflow_outputs.py +++ b/src/ewoksppf/tests/test_workflow_outputs.py @@ -48,11 +48,12 @@ def test_default_outputs(ppf_log_config): @pytest.mark.filterwarnings( - "ignore:Merged workflow results.*are overwritten by actor.*:UserWarning" + "ignore:This process .* is multi-threaded:DeprecationWarning" ) def test_all_outputs_merged(ppf_log_config): """'task6' is the last task with outputs so it takes precedence.""" - result = execute_graph(_create_graph(), outputs=[{"all": True}]) + with pytest.warns(UserWarning, match="are overwritten by actor"): + result = execute_graph(_create_graph(), outputs=[{"all": True}]) assert result == {"inputs": {"a": 10, "b": 6}, "result": 16, "label": "task6"}