From 12c707d9e8ef0d31cbcd64b65473187b924f04e6 Mon Sep 17 00:00:00 2001 From: Chris Fuller Date: Wed, 22 Apr 2026 09:00:20 +0100 Subject: [PATCH 1/3] Fix failing add node tests after category change --- tests/e2e/test_client_e2e.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/tests/e2e/test_client_e2e.py b/tests/e2e/test_client_e2e.py index e250845..22f170c 100644 --- a/tests/e2e/test_client_e2e.py +++ b/tests/e2e/test_client_e2e.py @@ -31,12 +31,10 @@ def test_list_nodes_category(self, e2e_client: Client): Args: e2e_client: A Client instance. """ - respone = e2e_client.list_nodes(category="Basic") + response = e2e_client.list_nodes(category="Mathematical Operators") - # There should only be one node in the demo category - assert len(respone) == 1 - assert respone[0]["category"] == "Basic" - assert respone[0]["id"] == "Add" + assert len(response) > 0 + assert all(node["category"] == "Mathematical Operators" for node in response) def test_queue_node(self, e2e_client: Client): """ From a8ccb88db1df08a510366ba76a5779d6b255acaf Mon Sep 17 00:00:00 2001 From: Chris Fuller Date: Wed, 22 Apr 2026 10:26:29 +0100 Subject: [PATCH 2/3] Add pytest.skip for list_nodes_category test in prod --- tests/e2e/test_client_e2e.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tests/e2e/test_client_e2e.py b/tests/e2e/test_client_e2e.py index 22f170c..1530024 100644 --- a/tests/e2e/test_client_e2e.py +++ b/tests/e2e/test_client_e2e.py @@ -1,3 +1,4 @@ +import os import time from unittest.mock import Mock, patch @@ -24,6 +25,10 @@ def test_list_nodes(self, e2e_client: Client): """ e2e_client.list_nodes() + @pytest.mark.skipif( + os.getenv("UE_ENVIRONMENT") == "prod", + reason=f"Mathematical Operators category is 'Basic' in prod until we deploy there", + ) def test_list_nodes_category(self, e2e_client: Client): """ Verify that nodes can be filtered by category. From 10d4f1f2a75fcaf112de51bc2cba33922c6aefa9 Mon Sep 17 00:00:00 2001 From: Chris Fuller Date: Wed, 22 Apr 2026 10:28:21 +0100 Subject: [PATCH 3/3] Switch from f-string to string in pytest.skip reason --- tests/e2e/test_client_e2e.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/e2e/test_client_e2e.py b/tests/e2e/test_client_e2e.py index 1530024..c6288a3 100644 --- a/tests/e2e/test_client_e2e.py +++ b/tests/e2e/test_client_e2e.py @@ -27,7 +27,7 @@ def test_list_nodes(self, e2e_client: Client): @pytest.mark.skipif( os.getenv("UE_ENVIRONMENT") == "prod", - reason=f"Mathematical Operators category is 'Basic' in prod until we deploy there", + reason="Mathematical Operators category is 'Basic' in prod until we deploy there", ) def test_list_nodes_category(self, e2e_client: Client): """