Skip to content

Commit

Permalink
test suite: print single file per line on lots of files (#311)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Kojoley committed Jul 18, 2023
1 parent c653af1 commit dc40b50
Showing 1 changed file with 12 additions and 4 deletions.
16 changes: 12 additions & 4 deletions test/tree.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit dc40b50

Please sign in to comment.