Use skipIf instead of a return when attribute doesn't exist.

2010-12-14 01:22:50 +00:00 · 2010-12-14 01:22:50 +00:00 · 6ecf76ea36
commit 6ecf76ea36
parent 4d55bf9fcc
1 changed files with 2 additions and 2 deletions
--- a/Lib/test/test_time.py
+++ b/Lib/test/test_time.py
@ -126,9 +126,9 @@ class TimeTestCase(unittest.TestCase):
    def test_asctime_bounding_check(self):
        self._bounds_checking(time.asctime)

+    @unittest.skipIf(not hasattr(time, "tzset"),
+        "time module has no attribute tzset")
    def test_tzset(self):
-        if not hasattr(time, "tzset"):
-            return # Can't test this; don't want the test suite to fail

        from os import environ