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 | d61be94db1fc5883a06f3a0eed262ffb1432ddda (patch) | |
tree | 11a6cfa9a50b760af3aac38e06ae6164e6139297 /coverage/report.py | |
parent | adf5c4705f1fb97b0c5f84ed5af40458b38f039d (diff) | |
download | python-coveragepy-d61be94db1fc5883a06f3a0eed262ffb1432ddda.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 df34e43..c3d59af 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: |