From db04ffdd2c7305243728cd662c11d49714068b8f Mon Sep 17 00:00:00 2001 From: Ned Batchelder Date: Tue, 28 Mar 2017 06:38:02 -0400 Subject: Clean up of the new plugin method. Thanks, Emil Madsen --- coverage/control.py | 3 ++- coverage/plugin.py | 8 ++++++-- 2 files changed, 8 insertions(+), 3 deletions(-) (limited to 'coverage') diff --git a/coverage/control.py b/coverage/control.py index 1f79e2f..4c43ee3 100644 --- a/coverage/control.py +++ b/coverage/control.py @@ -857,6 +857,7 @@ class Coverage(object): self.data.add_run_info(note=self.config.note) def _find_plugin_files(self, src_dir): + """Get executable files from the plugins.""" for plugin in self.plugins: for x_file in plugin.find_executable_files(src_dir): yield x_file, plugin._coverage_plugin_name @@ -868,7 +869,7 @@ class Coverage(object): and add them as unexecuted files in `self.data`. """ - py_files = ((py_file, None) for py_file in files.find_python_files(src_dir)) + py_files = ((py_file, None) for py_file in find_python_files(src_dir)) plugin_files = self._find_plugin_files(src_dir) for file_path, plugin_name in itertools.chain(py_files, plugin_files): diff --git a/coverage/plugin.py b/coverage/plugin.py index 43f291b..3e0e483 100644 --- a/coverage/plugin.py +++ b/coverage/plugin.py @@ -89,12 +89,16 @@ class CoveragePlugin(object): """ _needs_to_implement(self, "file_reporter") - def find_executable_files(self, src_dir): - """Yield all of the executable files in `dirname`, recursively. + def find_executable_files(self, src_dir): # pylint: disable=unused-argument + """Yield all of the executable files in `src_dir`, recursively. Executability is a plugin-specific property, but generally means files which would have been considered for coverage analysis, had they been included automatically. + + Returns or yields a sequence of strings, the paths to files that could + have been executed, including files that had been executed. + """ return [] -- cgit v1.2.1