From 4e081154b0685b2008b47887bf19046473f99a73 Mon Sep 17 00:00:00 2001 From: Dmitry Shachnev Date: Wed, 29 Jun 2016 13:15:16 +0300 Subject: [PATCH] Make coverage plugin compatible with Coverage.py 4.1 According to the Coverage.py 4.1 changelog: - The `Coverage.report` function had two parameters with non-None defaults, which have been changed. `show_missing` used to default to True, but now defaults to None. If you had been calling `Coverage.report` without specifying `show_missing`, you'll need to explicitly set it to True to keep the same behavior. Without that option, four tests in nose fail: - test_coverage_plugin.TestCoverageMinPercentagePlugin - test_coverage_plugin.TestCoverageMinPercentageSinglePackagePlugin - test_coverage_plugin.TestCoverageMinPercentageSinglePackageWithBranchesPlugin - test_coverage_plugin.TestCoveragePlugin --- nose/plugins/cover.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nose/plugins/cover.py b/nose/plugins/cover.py index 4bae004e..fc3321cd 100644 --- a/nose/plugins/cover.py +++ b/nose/plugins/cover.py @@ -202,7 +202,7 @@ def report(self, stream): if self.wantModuleCoverage(name, module)] log.debug("Coverage report will cover modules: %s", modules) if self.coverPrint: - self.coverInstance.report(modules, file=stream) + self.coverInstance.report(modules, file=stream, show_missing=True) import coverage if self.coverHtmlDir: @@ -222,7 +222,7 @@ def report(self, stream): # make sure we have minimum required coverage if self.coverMinPercentage: f = StringIO.StringIO() - self.coverInstance.report(modules, file=f) + self.coverInstance.report(modules, file=f, show_missing=True) multiPackageRe = (r'-------\s\w+\s+\d+\s+\d+(?:\s+\d+\s+\d+)?' r'\s+(\d+)%\s+\d*\s{0,1}$')