refactor: remove unnecessary 'client' fixture and update test duratio…

…ns (#4283)

* Remove unnecessary use of 'client' fixture in test_superuser

* Remove unnecessary use of 'usefixtures' decorator in test files

* Update test fixtures to improve test duration tracking

* Remove unnecessary 'client' fixture from test cases

src/backend/tests/unit/base/tools/test_component_toolkit.py

@@ -81,7 +81,7 @@ def test_component_tool():
 
 
 @pytest.mark.api_key_required
-@pytest.mark.usefixtures("_add_toolkit_output", "client")
+@pytest.mark.usefixtures("_add_toolkit_output")
 def test_component_tool_with_api_key():
     chat_output = ChatOutput()
     openai_llm = OpenAIModelComponent()

src/backend/tests/unit/graph/edge/test_edge_base.py

@@ -6,7 +6,6 @@
 from langflow.graph import Graph
 
 
-@pytest.mark.usefixtures("client")
 def test_edge_raises_error_on_invalid_target_handle():
     template = """Answer the user as if you were a pirate.
 

src/backend/tests/unit/initial_setup/starter_projects/test_memory_chatbot.py

@@ -43,7 +43,6 @@ def memory_chatbot_graph():
     return graph
 
 
-@pytest.mark.usefixtures("client")
 def test_memory_chatbot(memory_chatbot_graph):
     # Now we run step by step
     expected_order = deque(["chat_input", "chat_memory", "prompt", "openai", "chat_output"])

src/backend/tests/unit/test_cli.py

@@ -25,7 +25,6 @@ def test_components_path(runner, default_settings, tmp_path):
     assert str(temp_dir) in settings_service.settings.components_path
 
 
-@pytest.mark.usefixtures("client")
 def test_superuser(runner):
     result = runner.invoke(app, ["superuser"], input="admin\nadmin\n")
     assert result.exit_code == 0, result.stdout

src/backend/tests/unit/test_database.py

@@ -632,7 +632,6 @@ async def test_load_flows(client: TestClient):
     assert response.json()["folder_id"] is not None
 
 
-@pytest.mark.usefixtures("client")
 def test_sqlite_pragmas():
     db_service = get_db_service()