Cleanups identified by transition to Ruff.

Author: freakboy3742

pth-tester/pth_tester.py

@@ -2,7 +2,7 @@
 has_socket = False
 
 
-# The pth_tester module should be initalized by processing the `.pth` file
+# The pth_tester module should be initialized by processing the `.pth` file
 # created on installation.
 def init():
     global initialized

tests/test_common.py

@@ -51,9 +51,9 @@ def test_bootstrap_modules():
         except ModuleNotFoundError:
             missing.append(module)
 
-    assert (
-        len(missing) == 0
-    ), f"Missing bootstrap modules: {', '.join(str(m) for m in missing)}"
+    assert len(missing) == 0, (
+        f"Missing bootstrap modules: {', '.join(str(m) for m in missing)}"
+    )
 
 
 def test_stdlib_modules():
@@ -139,9 +139,9 @@ def test_stdlib_modules():
         except ModuleNotFoundError:
             missing.append(module)
 
-    assert (
-        len(missing) == 0
-    ), f"Missing stdlib modules: {', '.join(str(m) for m in missing)}"
+    assert len(missing) == 0, (
+        f"Missing stdlib modules: {', '.join(str(m) for m in missing)}"
+    )
 
 
 def test_sysconfig_data():
@@ -284,9 +284,9 @@ def test_hashlib():
             digest_args = []
         msg = getattr(hashlib, algorithm)()
         msg.update(b"Hello world")
-        assert (
-            msg.hexdigest(*digest_args) == expected
-        ), f"{algorithm} digest was {msg.hexdigest(*digest_args)}"
+        assert msg.hexdigest(*digest_args) == expected, (
+            f"{algorithm} digest was {msg.hexdigest(*digest_args)}"
+        )
 
 
 def test_sqlite3():

tests/test_macOS.py

@@ -81,6 +81,6 @@ def test_stdlib_modules():
         except ModuleNotFoundError:
             missing.append(module)
 
-    assert (
-        len(missing) == 0
-    ), f"Missing stdlib modules: {', '.join(str(m) for m in missing)}"
+    assert len(missing) == 0, (
+        f"Missing stdlib modules: {', '.join(str(m) for m in missing)}"
+    )

tests/test_thirdparty.py

@@ -11,7 +11,7 @@
 
 def xfail_if_not_installed(package_name):
     """A test decorator that xfails a test if the named package isn't installed.
-    The third-party tests are dependant on packages being built. During pre-release some
+    The third-party tests are dependent on packages being built. During pre-release some
     packages won't be compilable. So - the pyproject.toml installs third party packages
     with some conditional gating.
 
@@ -114,7 +114,7 @@ def test_cryptography():
 
     cert = x509.load_pem_x509_certificate(cert_pem, default_backend())
     domain = cert.subject.get_attributes_for_oid(NameOID.COMMON_NAME)[0].value
-    assert "www.android.com" == domain
+    assert domain == "www.android.com"
 
 
 @xfail_if_not_installed("pyspamsum")
@@ -176,7 +176,7 @@ def test_numpy():
     from numpy import array
 
     # Numpy is the thousand pound gorilla packaging test.
-    assert [4, 7] == (array([1, 2]) + array([3, 5])).tolist()
+    assert (array([1, 2]) + array([3, 5])).tolist() == [4, 7]
 
 
 @xfail_if_not_installed("pandas")
@@ -191,10 +191,10 @@ def test_pandas():
 
     # Pandas 1.5 changed the API for to_csv()
     if tuple(int(v) for v in __version__.split(".")[:2]) < (1, 5):
-        kwargs = dict(line_terminator="\n")
+        kwargs = {"line_terminator": "\n"}
     else:
-        kwargs = dict(lineterminator="\n")
+        kwargs = {"lineterminator": "\n"}
 
-    assert (
-        ",Letter,Number\n" "0,alpha,1\n" "1,bravo,2\n" "2,charlie,3\n"
-    ) == df.to_csv(**kwargs)
+    assert df.to_csv(**kwargs) == (
+        ",Letter,Number\n0,alpha,1\n1,bravo,2\n2,charlie,3\n"
+    )

tests/testbed.py

@@ -32,7 +32,7 @@ def run_tests():
     # macOS, output the log sentinel multiple times to decrease the chance of
     # this happening.
     if sys.platform in {"darwin", "ios"}:
-        for i in range(0, 6):
+        for _ in range(0, 6):
             time.sleep(0.5)
             print(f">>>>>>>>>> EXIT {returncode} <<<<<<<<<<")
     else: