jyapayne/langflow
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

refactor: remove unnecessary 'client' fixture and update test durations (#4283)

Browse source 
* 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
This commit is contained in:
Gabriel Luiz Freitas Almeida 2024-10-27 11:16:45 -03:00 • committed by GitHub
commit 73b5cc0098
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 601 additions and 605 deletions
Download patch file Download diff file Expand all files Collapse all files

1200
src/backend/tests/.test_durations
View file

File diff suppressed because it is too large Load diff

2
src/backend/tests/unit/base/tools/test_component_toolkit.py
View file

@ -81,7 +81,7 @@ def test_component_tool():
@pytest.mark.api_key_required @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(): def test_component_tool_with_api_key():
chat_output = ChatOutput() chat_output = ChatOutput()
openai_llm = OpenAIModelComponent() openai_llm = OpenAIModelComponent()

1
src/backend/tests/unit/graph/edge/test_edge_base.py
View file

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

1
src/backend/tests/unit/initial_setup/starter_projects/test_memory_chatbot.py
View file

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

1
src/backend/tests/unit/test_cli.py
View file

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

1
src/backend/tests/unit/test_database.py
View file

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