]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
[3.12] gh-116491: Improve `test_win32_ver` (GH-116506) (#116708)
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Wed, 13 Mar 2024 07:29:13 +0000 (08:29 +0100)
committerGitHub <noreply@github.com>
Wed, 13 Mar 2024 07:29:13 +0000 (07:29 +0000)
gh-116491: Improve `test_win32_ver` (GH-116506)
(cherry picked from commit ee0dbbc04504e0e0f1455e2bab8801ce0a682afd)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Lib/test/test_platform.py

index 216973350319fe69dd69319839ceeb0fb93aa2d6..b62a9e3897797cdf31b53d59c112799ea420a6fc 100644 (file)
@@ -322,8 +322,36 @@ class PlatformTest(unittest.TestCase):
         if sys.platform == 'java':  # Is never actually checked in CI
             self.assertTrue(all(res))
 
+    @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()