From dc40b5004fc771b1234ad51872ad369ea58437a4 Mon Sep 17 00:00:00 2001 From: Nikita Kniazev Date: Tue, 18 Jul 2023 05:55:10 +0300 Subject: [PATCH] test suite: print single file per line on lots of files (#311) It is much harder to spot stuff in a long string. When every file is printed on a separate line it is visually much simple to search through it, recognize missing files and wrong prefixes/suffixes. --- test/tree.py | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/test/tree.py b/test/tree.py index 18215d6724..73ade95102 100644 --- a/test/tree.py +++ b/test/tree.py @@ -107,11 +107,19 @@ def ignore_directories(self): self.modified_files = list(filter(not_dir, self.modified_files)) self.touched_files = list(filter(not_dir, self.touched_files)) + @staticmethod + def _pprint_filelist(names): + names = sorted(names) + s = repr(names) + if len(s) < 70: + return s + return "".join(f"\n - {name}" for name in names) + def pprint(self, file=sys.stdout): - file.write("Added files : %s\n" % self.added_files) - file.write("Removed files : %s\n" % self.removed_files) - file.write("Modified files: %s\n" % self.modified_files) - file.write("Touched files : %s\n" % self.touched_files) + file.write("Added files : %s\n" % self._pprint_filelist(self.added_files)) + file.write("Removed files : %s\n" % self._pprint_filelist(self.removed_files)) + file.write("Modified files: %s\n" % self._pprint_filelist(self.modified_files)) + file.write("Touched files : %s\n" % self._pprint_filelist(self.touched_files)) def empty(self): return not (self.added_files or self.removed_files or