Code review cleanups

test_runner/fixtures/pg_version.py

@@ -44,16 +44,17 @@ def _missing_(cls, value: object) -> PgVersion | None:
         if not isinstance(value, str):
             return None
 
-        known_values = {v.value for _, v in cls.__members__.items()}
+        known_values = set(cls.__members__.values())
 
         # Allow passing version as v-prefixed string (e.g. "v14")
         if value.lower().startswith("v") and (v := value[1:]) in known_values:
             return cls(v)
+
         # Allow passing version as an int (i.e. both "15" and "150002" matches PgVersion.V15)
-        elif value.isdigit() and (v := value[:2]) in known_values:
+        if value.isdigit() and (v := value[:2]) in known_values:
             return cls(v)
-        else:
-            return None
+
+        return None
 
 
 DEFAULT_VERSION: PgVersion = PgVersion.V16

test_runner/performance/test_physical_replication.py

@@ -237,7 +237,7 @@ def test_replication_start_stop(
             ],
             env=master_env,
         )
-        replica_pgbench: list[subprocess.Popen[Any] | None] = [None for _ in range(num_replicas)]
+        replica_pgbench: list[subprocess.Popen[Any] | None] = [None] * num_replicas
 
         # Use the bits of iconfig to tell us which configuration we are on. For example
         # a iconfig of 2 is 10 in binary, indicating replica 0 is suspended and replica 1 is