diff options
author | Ned Batchelder <ned@nedbatchelder.com> | 2016-05-10 12:37:21 -0400 |
---|---|---|
committer | Ned Batchelder <ned@nedbatchelder.com> | 2016-05-10 12:37:21 -0400 |
commit | 15977370873278c9c6b7714747ea2ef6d93b9203 (patch) | |
tree | 85c3978b445ddea1abfbfcf91365173b572ecd2c /coverage/report.py | |
parent | 89d564be706aceac367f9dccb29163fe7ae7904d (diff) | |
download | python-coveragepy-git-15977370873278c9c6b7714747ea2ef6d93b9203.tar.gz |
This didn't need to be an attribute on the object
Diffstat (limited to 'coverage/report.py')
-rw-r--r-- | coverage/report.py | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/coverage/report.py b/coverage/report.py index df34e43f..c3d59af1 100644 --- a/coverage/report.py +++ b/coverage/report.py @@ -24,9 +24,6 @@ class Reporter(object): self.coverage = coverage self.config = config - # The FileReporters to report on. Set by find_file_reporters. - self.file_reporters = [] - # The directory into which to place the report, used by some derived # classes. self.directory = None @@ -36,6 +33,8 @@ class Reporter(object): `morfs` is a list of modules or file names. + Returns a list of FileReporters. + """ reporters = self.coverage._get_file_reporters(morfs) @@ -47,7 +46,7 @@ class Reporter(object): matcher = FnmatchMatcher(prep_patterns(self.config.omit)) reporters = [fr for fr in reporters if not matcher.match(fr.filename)] - self.file_reporters = sorted(reporters) + return sorted(reporters) def report_files(self, report_fn, morfs, directory=None): """Run a reporting function on a number of morfs. @@ -61,16 +60,16 @@ class Reporter(object): `analysis` is the `Analysis` for the morf. """ - self.find_file_reporters(morfs) + file_reporters = self.find_file_reporters(morfs) - if not self.file_reporters: + if not file_reporters: raise CoverageException("No data to report.") self.directory = directory if self.directory and not os.path.exists(self.directory): os.makedirs(self.directory) - for fr in self.file_reporters: + for fr in file_reporters: try: report_fn(fr, self.coverage._analyze(fr)) except NoSource: |