summaryrefslogtreecommitdiff
path: root/coverage/report.py
diff options
context:
space:
mode:
authorNed Batchelder <ned@nedbatchelder.com>2019-06-15 15:32:43 -0400
committerNed Batchelder <ned@nedbatchelder.com>2019-06-15 15:32:43 -0400
commit730ef8b6047bcbe8b92675b5a8bcbe1060585d16 (patch)
treef499f824574ca3afa23f231ccb5534eeea23f52e /coverage/report.py
parentcb1b65a4ad1b8302304706b1775401d165486d02 (diff)
downloadpython-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.py11
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))