Issue #19828: Fixed test_site when the whole suite is run with -S.

Also, cleaned up an unused import.

Lib/test/test_site.py

@@ -6,8 +6,7 @@ executing have not been removed.
 """
 import unittest
 import test.support
-from test.support import run_unittest, TESTFN, EnvironmentVarGuard
+from test.support import captured_stderr, TESTFN, EnvironmentVarGuard
-from test.support import captured_stderr
 import builtins
 import os
 import sys
@@ -19,13 +18,13 @@ import subprocess
 import sysconfig
 from copy import copy
 
-# Need to make sure to not import 'site' if someone specified ``-S`` at the
+# These tests are not particularly useful if Python was invoked with -S.
-# command-line.  Detect this by just making sure 'site' has not been imported
+# If you add tests that are useful under -S, this skip should be moved
-# already.
+# to the class level.
-if "site" in sys.modules:
+if sys.flags.no_site:
-    import site
+    raise unittest.SkipTest("Python was invoked with -S")
-else:
+
-    raise unittest.SkipTest("importation of site.py suppressed")
+import site
 
 if site.ENABLE_USER_SITE and not os.path.isdir(site.USER_SITE):
     # need to add user site directory for tests

Misc/NEWS

@@ -130,6 +130,8 @@ IDLE
 Tests
 -----
 
+- Issue #19828: Fixed test_site when the whole suite is run with -S.
+
 - Issue #19928: Implemented a test for repr() of cell objects.
 
 - Issue #19535: Fixed test_docxmlrpc when python is run with -OO.