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

bitdancer · bitdancer · commit 6ecf76ea3617 · 2010-12-14T01:22:50.000Z
diff --git a/Lib/test/test_time.py b/Lib/test/test_time.py
@@ -126,9 +126,9 @@ def test_asctime(self):
     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
 
