Skip to content

Commit 007b524

Browse files
committed
fix logging
1 parent a365d26 commit 007b524

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

python/lib/sift_client/_internal/low_level_wrappers/test_results.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1151,7 +1151,7 @@ def _iter_log_data_lines(
11511151
continue
11521152
match = line_pattern.match(line)
11531153
if not match:
1154-
continue
1154+
raise ValueError(f"Invalid log line: {line}")
11551155
request_type = match.group(1)
11561156
if request_type == "LogTracking":
11571157
continue

python/lib/sift_client/_tests/resources/test_test_results.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -705,7 +705,7 @@ async def test_malformed_log_line_skipped(self, tmp_path):
705705
)
706706

707707
client = TestResultsLowLevelClient(grpc_client=MagicMock())
708-
with pytest.raises(ValueError, match="No CreateTestReport found"):
708+
with pytest.raises(ValueError, match="Invalid log line: this is not a valid log lin"):
709709
await client.import_log_file(log_file)
710710

711711
@pytest.mark.asyncio

0 commit comments

Comments
 (0)