diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2019-06-15 15:32:43 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2019-06-15 15:32:43 -0400 |
commit | 730ef8b6047bcbe8b92675b5a8bcbe1060585d16 (patch) | |
tree | f499f824574ca3afa23f231ccb5534eeea23f52e /coverage/report.py | |
parent | cb1b65a4ad1b8302304706b1775401d165486d02 (diff) | |
download | python-coveragepy-git-730ef8b6047bcbe8b92675b5a8bcbe1060585d16.tar.gz |
Reporter base class shouldn't be responsible for directories
Diffstat (limited to 'coverage/report.py')
-rw-r--r-- | coverage/report.py | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/coverage/report.py b/coverage/report.py index 6f87bbf2..c694d79b 100644 --- a/coverage/report.py +++ b/coverage/report.py @@ -7,7 +7,7 @@ import os import warnings from coverage.files import prep_patterns, FnmatchMatcher -from coverage.misc import CoverageException, NoSource, NotPython, ensure_dir, isolate_module +from coverage.misc import CoverageException, NoSource, NotPython, isolate_module os = isolate_module(os) @@ -25,10 +25,6 @@ class Reporter(object): self.coverage = coverage self.config = config - # The directory into which to place the report, used by some derived - # classes. - self.directory = None - # Our method find_file_reporters used to set an attribute that other # code could read. That's been refactored away, but some third parties # were using that attribute. We'll continue to support it in a noisy @@ -65,7 +61,7 @@ class Reporter(object): self._file_reporters = sorted(reporters) return self._file_reporters - def report_files(self, report_fn, morfs, directory=None): + def report_files(self, report_fn, morfs): """Run a reporting function on a number of morfs. `report_fn` is called for each relative morf in `morfs`. It is called @@ -82,9 +78,6 @@ class Reporter(object): if not file_reporters: raise CoverageException("No data to report.") - self.directory = directory - ensure_dir(self.directory) - for fr in file_reporters: try: report_fn(fr, self.coverage._analyze(fr)) |