Skip to content

Commit 677cda2

Browse files
committed
IDEV-2013: Add tests.
1 parent 9ec9638 commit 677cda2

File tree

5 files changed

+598
-549
lines changed

5 files changed

+598
-549
lines changed

domaintools/utils.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,8 +177,12 @@ def validate_feeds_parameters(params):
177177
after = params.get("after")
178178
before = params.get("before")
179179
if not (sessionID or after or before):
180-
raise ValueError("sessionID or after or before must be defined")
180+
raise ValueError("sessionID or after or before must be provided")
181181

182182
format = params.get("output_format")
183-
if params.get("endpoint") == Endpoint.DOWNLOAD.value and format == OutputFormat.CSV.value:
183+
endpoint = params.get("endpoint")
184+
if endpoint == Endpoint.DOWNLOAD.value and format == OutputFormat.CSV.value:
184185
raise ValueError(f"{format} format is not available in {Endpoint.DOWNLOAD.value} API.")
186+
187+
if endpoint == Endpoint.DOWNLOAD.value and params.get("header_authentication", True):
188+
raise ValueError(f"{Endpoint.DOWNLOAD.value} API does not support header authentication. Provide api_key in the parameter")

tests/conftest.py

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,14 @@
11
"""Configuration for test environment"""
2+
3+
import pytest
4+
5+
6+
@pytest.fixture
7+
def test_feeds_params():
8+
return {
9+
"sessionID": "test-session-id",
10+
"after": -60,
11+
"before": -120,
12+
"output_format": "csv",
13+
"endpoint": "download",
14+
}

tests/settings.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,5 +49,11 @@ def filter_patch_parameters(request):
4949
api = API(
5050
os.getenv("TEST_USER", "test"),
5151
os.getenv("TEST_KEY", "test"),
52+
)
53+
54+
feeds_api = API(
55+
os.getenv("TEST_USER", "test"),
56+
os.getenv("TEST_KEY", "test"),
5257
always_sign_api_key=False,
58+
rate_limit=False,
5359
)

0 commit comments

Comments
 (0)