Clean up of the new plugin method. Thanks, Emil Madsen

2 files changed, 8 insertions, 3 deletions

diff --git a/coverage/control.py b/coverage/control.py
index 1f79e2f0..4c43ee38 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 43f291b3..3e0e4836 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 []