mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-11-22 07:31:57 +00:00
Consider whole check failure in bugfix validate check as an error
This commit is contained in:
parent
30148972ed
commit
d2bea265c9
@ -36,6 +36,18 @@ def process_result(file_path: Path) -> Tuple[bool, TestResults]:
|
||||
test_results = [] # type: TestResults
|
||||
state, report_url, description = post_commit_status_from_file(file_path)
|
||||
prefix = file_path.parent.name
|
||||
if description.strip() in [
|
||||
"Invalid check_status.tsv",
|
||||
"Not found test_results.tsv",
|
||||
"Empty test_results.tsv",
|
||||
]:
|
||||
status = (
|
||||
f'Check failed (<a href="{report_url}">Report</a>)'
|
||||
if report_url != "null"
|
||||
else "Check failed"
|
||||
)
|
||||
return False, [TestResult(f"{prefix}: {description}", status)]
|
||||
|
||||
is_ok = state == "success"
|
||||
if is_ok and report_url == "null":
|
||||
return is_ok, test_results
|
||||
|
Loading…
Reference in New Issue
Block a user