thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-21 15:12:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add option --show-whitespaces-in-diff to clickhouse-test

Browse Source
This commit is contained in:
vdimir 2023-12-14 15:17:47 +00:00
parent 08ff37f64e
commit 4d7edb4321
No known key found for this signature in database
GPG Key ID: 6EE4CE2BEDC51862
1 changed files with 17 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
tests/clickhouse-test
View File

@ -947,6 +947,8 @@ class TestCase:
else ""
)
self.show_whitespaces_in_diff = args.show_whitespaces_in_diff
# should skip test, should increment skipped_total, skip reason
def should_skip_test(self, suite) -> Optional[FailureReason]:
tags = self.tags
@ -1146,7 +1148,7 @@ class TestCase:
)
if result_is_different:
diff = Popen(
diff_proc = Popen(
[
"diff",
"-U",
@ -1157,7 +1159,14 @@ class TestCase:
encoding="latin-1",
stdout=PIPE,
universal_newlines=True,
).communicate()[0]
)
if self.show_whitespaces_in_diff:
cat_proc = Popen(["cat", "-te"], stdin=diff_proc.stdout, stdout=PIPE)
diff_proc.stdout.close() # Allow diff to receive a SIGPIPE if cat exits.
diff = cat_proc.communicate()[0].decode("utf-8", errors="ignore")
else:
diff = diff_proc.communicate()[0]
if diff.startswith("Binary files "):
diff += "Content of stdout:\n===================\n"
file = open(self.stdout_file, "rb")
@ -2755,6 +2764,12 @@ def parse_args():
help="Total test groups for crc32(test_name) % run_by_hash_total == run_by_hash_num",
)
parser.add_argument(
"--show-whitespaces-in-diff",
action="store_true",
help="Display TAB characters as ^I and trailing spaces as $ in diff",
)
group = parser.add_mutually_exclusive_group(required=False)
group.add_argument(
"--zookeeper",