From: Eric Smith <eric@trueblade.com>
Date: Wed, 27 Jan 2010 01:21:15 +0000 (+0000)
Subject: Switch to test_support.get_attribute.
X-Git-Tag: v2.7a3~78
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=59529e1ef661fc2588f3436ec11909b238006116;p=python

Switch to test_support.get_attribute.
---

diff --git a/Lib/test/test_sys.py b/Lib/test/test_sys.py
index 1e28ccfc6a..e05b0daf30 100644
--- a/Lib/test/test_sys.py
+++ b/Lib/test/test_sys.py
@@ -204,29 +204,30 @@ class SysModuleTest(unittest.TestCase):
         sys.setrecursionlimit(oldlimit)
 
     def test_getwindowsversion(self):
-        if hasattr(sys, "getwindowsversion"):
-            v = sys.getwindowsversion()
-            self.assertEqual(len(v), 5)
-            self.assertIsInstance(v[0], int)
-            self.assertIsInstance(v[1], int)
-            self.assertIsInstance(v[2], int)
-            self.assertIsInstance(v[3], int)
-            self.assertIsInstance(v[4], str)
-            self.assertRaises(IndexError, operator.getitem, v, 5)
-            self.assertIsInstance(v.major, int)
-            self.assertIsInstance(v.minor, int)
-            self.assertIsInstance(v.build, int)
-            self.assertIsInstance(v.platform, int)
-            self.assertIsInstance(v.service_pack, str)
-            self.assertEqual(v[0], v.major)
-            self.assertEqual(v[1], v.minor)
-            self.assertEqual(v[2], v.build)
-            self.assertEqual(v[3], v.platform)
-            self.assertEqual(v[4], v.service_pack)
-
-            # This is how platform.py calls it. Make sure tuple
-            #  still has 5 elements
-            maj, min, buildno, plat, csd = sys.getwindowsversion()
+        # Raise SkipTest if sys doesn't have getwindowsversion attribute
+        test.test_support.get_attribute(sys, "getwindowsversion")
+        v = sys.getwindowsversion()
+        self.assertEqual(len(v), 5)
+        self.assertIsInstance(v[0], int)
+        self.assertIsInstance(v[1], int)
+        self.assertIsInstance(v[2], int)
+        self.assertIsInstance(v[3], int)
+        self.assertIsInstance(v[4], str)
+        self.assertRaises(IndexError, operator.getitem, v, 5)
+        self.assertIsInstance(v.major, int)
+        self.assertIsInstance(v.minor, int)
+        self.assertIsInstance(v.build, int)
+        self.assertIsInstance(v.platform, int)
+        self.assertIsInstance(v.service_pack, str)
+        self.assertEqual(v[0], v.major)
+        self.assertEqual(v[1], v.minor)
+        self.assertEqual(v[2], v.build)
+        self.assertEqual(v[3], v.platform)
+        self.assertEqual(v[4], v.service_pack)
+
+        # This is how platform.py calls it. Make sure tuple
+        #  still has 5 elements
+        maj, min, buildno, plat, csd = sys.getwindowsversion()
 
     def test_dlopenflags(self):
         if hasattr(sys, "setdlopenflags"):