From: Serhiy Storchaka <storchaka@gmail.com>
Date: Sun, 3 Sep 2023 12:21:43 +0000 (+0300)
Subject: Reorder some test's decorators (GH-108804)
X-Git-Tag: v3.13.0a1~633
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=509bb61977cc8a4487efd3f9cdd63d9f7b86be62;p=thirdparty%2FPython%2Fcpython.git

Reorder some test's decorators (GH-108804)

For example, do not demand the 'cpu' resource if the test cannot be run
due to non-working threads.
---

diff --git a/Lib/test/test_io.py b/Lib/test/test_io.py
index 26ae40d93c84..6976a64bf77e 100644
--- a/Lib/test/test_io.py
+++ b/Lib/test/test_io.py
@@ -1541,8 +1541,8 @@ class BufferedReaderTest(unittest.TestCase, CommonBufferedTests):
 
         self.assertEqual(b"abcdefg", bufio.read())
 
-    @support.requires_resource('cpu')
     @threading_helper.requires_working_threading()
+    @support.requires_resource('cpu')
     def test_threads(self):
         try:
             # Write out many bytes with exactly the same number of 0's,
@@ -1930,8 +1930,8 @@ class BufferedWriterTest(unittest.TestCase, CommonBufferedTests):
                 f.truncate()
                 self.assertEqual(f.tell(), buffer_size + 2)
 
-    @support.requires_resource('cpu')
     @threading_helper.requires_working_threading()
+    @support.requires_resource('cpu')
     def test_threads(self):
         try:
             # Write out many bytes from many threads and test they were
diff --git a/Lib/test/test_site.py b/Lib/test/test_site.py
index 9e701fd847ac..20a96169e8be 100644
--- a/Lib/test/test_site.py
+++ b/Lib/test/test_site.py
@@ -465,10 +465,10 @@ class ImportSideEffectTests(unittest.TestCase):
             else:
                 self.fail("sitecustomize not imported automatically")
 
-    @test.support.requires_resource('network')
-    @test.support.system_must_validate_cert
     @unittest.skipUnless(hasattr(urllib.request, "HTTPSHandler"),
                          'need SSL support to download license')
+    @test.support.requires_resource('network')
+    @test.support.system_must_validate_cert
     def test_license_exists_at_url(self):
         # This test is a bit fragile since it depends on the format of the
         # string displayed by license in the absence of a LICENSE file.