Remove ABC subscript type arg to support Python < 3.9.

Author: ajwelch4

client/bqms_run/gcp/gcs.py

@@ -53,7 +53,10 @@ def set_as_default_client(self) -> None:
         cloudpathlib.GSClient._default_client = self  # type: ignore[misc] # pylint: disable=protected-access
 
     def _download_file(
-        self, cloud_path: cloudpathlib.GSPath, local_path: Union[str, os.PathLike[Any]]
+        self,
+        cloud_path: cloudpathlib.GSPath,
+        # Subscript type arg not supported on ABCs in Python < 3.9.
+        local_path: Union[str, os.PathLike],  # type: ignore[type-arg]
     ) -> Path:
         """Override to use bucket.blob instead of bucket.get_blob.
 
@@ -84,7 +87,11 @@ class GSPath(cloudpathlib.GSPath):
 
     client: GSClient
 
-    def download_to(self, destination: Union[str, os.PathLike[Any]]) -> Path:
+    def download_to(
+        self,
+        # Subscript type arg not supported on ABCs in Python < 3.9.
+        destination: Union[str, os.PathLike],  # type: ignore[type-arg]
+    ) -> Path:
         """Download GSPath to local cache."""
         destination = Path(destination)
         if destination.is_dir():

client/noxfile.py

@@ -13,6 +13,7 @@
 # limitations under the License.
 """Dev commands."""
 import pathlib
+import tempfile
 
 import nox
 
@@ -51,38 +52,45 @@ def type_check(session):
 @nox.session(python=["3.7", "3.8", "3.9", "3.10"])
 def tests(session):
     """Execute tests."""
-    session.install("poetry")
-    session.run("poetry", "install")
-    tests_path = pathlib.Path("tests")
-    if "unit" in session.posargs:
-        tests_path = tests_path / "unit"
-    if "integration" in session.posargs:
-        tests_path = tests_path / "integration"
-    if "pytest_verbose" in session.posargs:
-        session.run(
-            "poetry",
-            "run",
-            "coverage",
-            "run",
-            "--source=bqms_run",
-            "-m",
-            "pytest",
-            "-vv",
-            "--log-cli-level=INFO",
-            "--log-cli-format="
-            "%(asctime)s: %(levelname)s: %(threadName)s: "
-            "%(filename)s:%(lineno)s: %(message)s",
-            tests_path.as_posix(),
-        )
-    else:
+    with tempfile.NamedTemporaryFile() as temp_file:
+        session.install("poetry")
         session.run(
             "poetry",
-            "run",
-            "coverage",
-            "run",
-            "--source=bqms_run",
-            "-m",
-            "pytest",
-            tests_path.as_posix(),
+            "export",
+            "--with",
+            "dev",
+            "--without-hashes",
+            "--output",
+            temp_file.name,
         )
-    session.run("poetry", "run", "coverage", "report", "-m")
+        session.install("-r", temp_file.name)
+        session.install(".")
+        tests_path = pathlib.Path("tests")
+        if "unit" in session.posargs:
+            tests_path = tests_path / "unit"
+        if "integration" in session.posargs:
+            tests_path = tests_path / "integration"
+        if "pytest_verbose" in session.posargs:
+            session.run(
+                "coverage",
+                "run",
+                "--source=bqms_run",
+                "-m",
+                "pytest",
+                "-vv",
+                "--log-cli-level=INFO",
+                "--log-cli-format="
+                "%(asctime)s: %(levelname)s: %(threadName)s: "
+                "%(filename)s:%(lineno)s: %(message)s",
+                tests_path.as_posix(),
+            )
+        else:
+            session.run(
+                "coverage",
+                "run",
+                "--source=bqms_run",
+                "-m",
+                "pytest",
+                tests_path.as_posix(),
+            )
+        session.run("coverage", "report", "-m")