From 783f49317394734c8e50e9e8c2aa34427e9e121e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tarek=20Ziad=C3=A9?= Date: Fri, 17 Apr 2009 14:29:56 +0000 Subject: [PATCH] DistutilsSetupError was not raised when one single warning occured --- Lib/distutils/command/check.py | 2 +- Lib/distutils/tests/test_check.py | 17 ++++++++--------- 2 files changed, 9 insertions(+), 10 deletions(-) diff --git a/Lib/distutils/command/check.py b/Lib/distutils/command/check.py index d164f3f5279..7d56dfcf400 100644 --- a/Lib/distutils/command/check.py +++ b/Lib/distutils/command/check.py @@ -72,7 +72,7 @@ class check(Command): # let's raise an error in strict mode, if we have at least # one warning - if self.strict and self._warnings > 1: + if self.strict and self._warnings > 0: raise DistutilsSetupError('Please correct your package.') def check_metadata(self): diff --git a/Lib/distutils/tests/test_check.py b/Lib/distutils/tests/test_check.py index 5e0c453140e..372bae367b1 100644 --- a/Lib/distutils/tests/test_check.py +++ b/Lib/distutils/tests/test_check.py @@ -72,17 +72,16 @@ class CheckTestCase(support.LoggingSilencer, self.assertEquals(cmd._warnings, 1) # let's see if we have an error with strict=1 - cmd = check(dist) - cmd.initialize_options() - cmd.strict = 1 - cmd.ensure_finalized() - self.assertRaises(DistutilsSetupError, cmd.run) + metadata = {'url': 'xxx', 'author': 'xxx', + 'author_email': 'xxx', + 'name': 'xxx', 'version': 'xxx', + 'long_description': broken_rest} + self.assertRaises(DistutilsSetupError, self._run, metadata, + **{'strict': 1, 'restructuredtext': 1}) # and non-broken rest - rest = 'title\n=====\n\ntest' - pkg_info, dist = self.create_dist(long_description=rest) - cmd = check(dist) - cmd.check_restructuredtext() + metadata['long_description'] = 'title\n=====\n\ntest' + cmd = self._run(metadata, strict=1, restructuredtext=1) self.assertEquals(cmd._warnings, 0) def test_check_all(self):