Skip to content

Commit 25143c9

Browse files
move start date back for tests, add logger for test parallelism debugging
1 parent 8a016d0 commit 25143c9

2 files changed

Lines changed: 8 additions & 2 deletions

File tree

tap_circle_ci/client.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,15 @@
55
import requests
66
from requests import session
77
from requests.exceptions import Timeout, ConnectionError, ChunkedEncodingError
8-
from singer import metrics
8+
from singer import metrics, get_logger
99

1010
from .exceptions import (
1111
ERROR_CODE_EXCEPTION_MAPPING, ClientError, Server5xxError,
1212
Http404RequestError, Http429RequestError
1313
)
1414

15+
LOGGER = get_logger()
16+
1517

1618
def raise_for_error(response: requests.Response, endpoint: str = "") -> None:
1719
"""Raises the associated response exception. Takes in a response object,
@@ -123,6 +125,10 @@ def __make_request(self, method: str, endpoint: str, **kwargs) -> Optional[Mappi
123125
try:
124126
raise_for_error(response, endpoint)
125127
except Http404RequestError:
128+
LOGGER.warning(
129+
"Received 404 for endpoint %s, treating as no data. Response body: %s",
130+
endpoint, response.text[:500]
131+
)
126132
return self.default_response
127133
else:
128134
raise ValueError(f"Unsupported method: {method}")

tests/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ def get_credentials(self):
165165
def get_properties(self, original: bool = True):
166166
"""Configuration of properties required for the tap."""
167167
return_value = {
168-
"start_date": "2026-06-16T00:00:00Z",
168+
"start_date": "2026-05-02T00:00:00Z",
169169
"project_slugs": os.getenv("TAP_CIRCLE_CI_PROJECTS"),
170170
}
171171
if original:

0 commit comments

Comments
 (0)