mirror of
https://github.com/langgenius/dify.git
synced 2026-09-05 00:31:19 +08:00
test(api): align new auth timeout tests with existing docstring style
The existing tests in the same files use sentence-case docstrings
("Test that ...") without backticks and without trailing periods.
The four new tests used a slightly different shape ("X should Y"
with backticks and a period); this commit trims them to match.
This commit is contained in:
parent
24f2779ddb
commit
058ba6ea9b
@ -208,7 +208,7 @@ class TestFirecrawlAuth:
|
||||
assert "timed out" in str(exc_info.value)
|
||||
|
||||
def test_credential_timeout_bounds_connect_phase(self):
|
||||
"""Credential-validation timeout should bound the connect phase to 3.0s."""
|
||||
"""Test that the credential timeout bounds the connect phase to 3.0s"""
|
||||
from services.auth.firecrawl.firecrawl import _CREDENTIAL_TIMEOUT
|
||||
|
||||
assert _CREDENTIAL_TIMEOUT.connect == 3.0
|
||||
@ -216,7 +216,7 @@ class TestFirecrawlAuth:
|
||||
|
||||
@patch("services.auth.firecrawl.firecrawl.httpx.post", autospec=True)
|
||||
def test_post_request_passes_credential_timeout_with_connect(self, mock_post: MagicMock):
|
||||
"""`_post_request` should forward the connect-bounded timeout to httpx.post."""
|
||||
"""Test that _post_request forwards the connect-bounded timeout to httpx.post"""
|
||||
mock_response = MagicMock()
|
||||
mock_response.status_code = 200
|
||||
mock_post.return_value = mock_response
|
||||
|
||||
@ -187,7 +187,7 @@ class TestJinaAuth:
|
||||
assert "super_secret_key_12345" not in str(exc_info.value)
|
||||
|
||||
def test_pooled_http_client_bounds_connect_phase(self):
|
||||
"""Pooled Jina client should have a 3.0s connect timeout on its Timeout object."""
|
||||
"""Test that the pooled Jina client has a 3.0s connect timeout"""
|
||||
from services.auth.jina.jina import _http_client
|
||||
|
||||
assert _http_client.timeout.connect == 3.0
|
||||
|
||||
@ -219,7 +219,7 @@ class TestWatercrawlAuth:
|
||||
assert "timed out" in str(exc_info.value)
|
||||
|
||||
def test_credential_timeout_bounds_connect_phase(self):
|
||||
"""Credential-validation timeout should bound the connect phase to 3.0s."""
|
||||
"""Test that the credential timeout bounds the connect phase to 3.0s"""
|
||||
from services.auth.watercrawl.watercrawl import _CREDENTIAL_TIMEOUT
|
||||
|
||||
assert _CREDENTIAL_TIMEOUT.connect == 3.0
|
||||
@ -227,7 +227,7 @@ class TestWatercrawlAuth:
|
||||
|
||||
@patch("services.auth.watercrawl.watercrawl.httpx.get", autospec=True)
|
||||
def test_get_request_passes_credential_timeout_with_connect(self, mock_get):
|
||||
"""`_get_request` should forward the connect-bounded timeout to httpx.get."""
|
||||
"""Test that _get_request forwards the connect-bounded timeout to httpx.get"""
|
||||
mock_response = MagicMock()
|
||||
mock_response.status_code = 200
|
||||
mock_get.return_value = mock_response
|
||||
|
||||
Loading…
Reference in New Issue
Block a user