]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
gh-116491: Improve `test_win32_ver` (#116506)
authorNikita Sobolev <mail@sobolevn.me>
Wed, 13 Mar 2024 06:46:48 +0000 (09:46 +0300)
committerGitHub <noreply@github.com>
Wed, 13 Mar 2024 06:46:48 +0000 (09:46 +0300)
Lib/test/test_platform.py

index bbd0c06efed2c98417be0e678bf5318793174d73..9f8aeeea25731120b64823e800b33f5daee477e4 100644 (file)
@@ -326,8 +326,36 @@ class PlatformTest(unittest.TestCase):
             res = platform.java_ver()
         self.assertEqual(len(res), 4)
 
+    @unittest.skipUnless(support.MS_WINDOWS, 'This test only makes sense on Windows')
     def test_win32_ver(self):
-        res = platform.win32_ver()
+        release1, version1, csd1, ptype1 = 'a', 'b', 'c', 'd'
+        res = platform.win32_ver(release1, version1, csd1, ptype1)
+        self.assertEqual(len(res), 4)
+        release, version, csd, ptype = res
+        if release:
+            # Currently, release names always come from internal dicts,
+            # but this could change over time. For now, we just check that
+            # release is something different from what we have passed.
+            self.assertNotEqual(release, release1)
+        if version:
+            # It is rather hard to test explicit version without
+            # going deep into the details.
+            self.assertIn('.', version)
+            for v in version.split('.'):
+                int(v)  # should not fail
+        if csd:
+            self.assertTrue(csd.startswith('SP'), msg=csd)
+        if ptype:
+            if os.cpu_count() > 1:
+                self.assertIn('Multiprocessor', ptype)
+            else:
+                self.assertIn('Uniprocessor', ptype)
+
+    @unittest.skipIf(support.MS_WINDOWS, 'This test only makes sense on non Windows')
+    def test_win32_ver_on_non_windows(self):
+        release, version, csd, ptype = 'a', '1.0', 'c', 'd'
+        res = platform.win32_ver(release, version, csd, ptype)
+        self.assertSequenceEqual(res, (release, version, csd, ptype), seq_type=tuple)
 
     def test_mac_ver(self):
         res = platform.mac_ver()