From 9316f2c2f88ab1b3bc9316ba39d5cc4fc461dd1d Mon Sep 17 00:00:00 2001 From: Giles Odigwe Date: Fri, 24 Apr 2026 11:00:29 -0700 Subject: [PATCH] Re-enable skipped Functions/DurableTask tests and bump timeout to 480s - Remove hard skips from 4 tests in test_11_workflow_parallel.py - Remove hard skip from test_conditional_branching in test_06_dt_multi_agent_orchestration_conditionals.py - Increase pytest --timeout from 360 to 480 for Functions+DurableTask CI job - Updated in both python-merge-tests.yml and python-integration-tests.yml Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .github/workflows/python-integration-tests.yml | 2 +- .github/workflows/python-merge-tests.yml | 2 +- .../tests/integration_tests/test_11_workflow_parallel.py | 4 ---- .../test_06_dt_multi_agent_orchestration_conditionals.py | 1 - 4 files changed, 2 insertions(+), 7 deletions(-) diff --git a/.github/workflows/python-integration-tests.yml b/.github/workflows/python-integration-tests.yml index 82e125acd2..0634c99c1d 100644 --- a/.github/workflows/python-integration-tests.yml +++ b/.github/workflows/python-integration-tests.yml @@ -293,7 +293,7 @@ jobs: -m integration -n logical --dist worksteal -x - --timeout=360 --session-timeout=900 --timeout_method thread + --timeout=480 --session-timeout=900 --timeout_method thread --retries 2 --retry-delay 5 --junitxml=pytest.xml - name: Upload test results diff --git a/.github/workflows/python-merge-tests.yml b/.github/workflows/python-merge-tests.yml index f6c8fe24cb..7a51128b0e 100644 --- a/.github/workflows/python-merge-tests.yml +++ b/.github/workflows/python-merge-tests.yml @@ -422,7 +422,7 @@ jobs: -m integration -n logical --dist worksteal -x - --timeout=360 --session-timeout=900 --timeout_method thread + --timeout=480 --session-timeout=900 --timeout_method thread --retries 2 --retry-delay 5 --junitxml=pytest.xml working-directory: ./python diff --git a/python/packages/azurefunctions/tests/integration_tests/test_11_workflow_parallel.py b/python/packages/azurefunctions/tests/integration_tests/test_11_workflow_parallel.py index bc9fa59bca..683ab7e0be 100644 --- a/python/packages/azurefunctions/tests/integration_tests/test_11_workflow_parallel.py +++ b/python/packages/azurefunctions/tests/integration_tests/test_11_workflow_parallel.py @@ -42,7 +42,6 @@ class TestWorkflowParallel: self.base_url = base_url self.helper = sample_helper - @pytest.mark.skip(reason="Causes timeouts.") def test_parallel_workflow_document_analysis(self) -> None: """Test parallel workflow with a standard document.""" payload = { @@ -71,7 +70,6 @@ class TestWorkflowParallel: assert status["runtimeStatus"] == "Completed" assert "output" in status - @pytest.mark.skip(reason="Causes timeouts.") def test_parallel_workflow_short_document(self) -> None: """Test parallel workflow with a short document.""" payload = { @@ -91,7 +89,6 @@ class TestWorkflowParallel: assert status["runtimeStatus"] == "Completed" assert "output" in status - @pytest.mark.skip(reason="Causes timeouts.") def test_parallel_workflow_technical_document(self) -> None: """Test parallel workflow with a technical document.""" payload = { @@ -115,7 +112,6 @@ class TestWorkflowParallel: status = self.helper.wait_for_orchestration_with_output(data["statusQueryGetUri"], max_wait=300) assert status["runtimeStatus"] == "Completed" - @pytest.mark.skip(reason="Causes timeouts.") def test_workflow_status_endpoint(self) -> None: """Test that the workflow status endpoint works correctly.""" payload = { diff --git a/python/packages/durabletask/tests/integration_tests/test_06_dt_multi_agent_orchestration_conditionals.py b/python/packages/durabletask/tests/integration_tests/test_06_dt_multi_agent_orchestration_conditionals.py index d20c67e20f..177f4ca5f4 100644 --- a/python/packages/durabletask/tests/integration_tests/test_06_dt_multi_agent_orchestration_conditionals.py +++ b/python/packages/durabletask/tests/integration_tests/test_06_dt_multi_agent_orchestration_conditionals.py @@ -52,7 +52,6 @@ class TestMultiAgentOrchestrationConditionals: assert email_agent is not None assert email_agent.name == EMAIL_AGENT_NAME - @pytest.mark.skip(reason="Consistently fails due to orchestration timeouts - needs investigation") def test_conditional_branching(self): """Test that conditional branching works correctly.""" # Test with obvious spam