fix: TODOs

Author: Justintime50

.github/workflows/ci.yml

@@ -19,11 +19,10 @@ jobs:
       - name: Lint
         run: make lint
   run-tests:
-    # TODO: Use `latest` once we drop support for Python 3.7
-    runs-on: ubuntu-22.04
+    runs-on: ubuntu-latest
     strategy:
       matrix:
-        pythonversion: ['3.7', '3.8', '3.9', '3.10', '3.11', '3.12', '3.13']
+        pythonversion: ['3.8', '3.9', '3.10', '3.11', '3.12', '3.13']
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5

easypost/util.py

@@ -102,7 +102,7 @@ def get_lowest_stateless_rate(
         ):
             continue
 
-        if lowest_rate is None or float(rate.rate) < float(lowest_rate.rate):
+        if lowest_rate is None or float(rate["rate"]) < float(lowest_rate["rate"]):
             lowest_rate = rate
 
     if lowest_rate is None:

setup.py

@@ -9,18 +9,17 @@
 ]
 
 DEV_REQUIREMENTS = [
-    "bandit==1.7.5",
-    "black==23.*",
-    "build==1.0.*;python_version>='3.8'",  # TODO: remove python pin when 3.7 is dropped
-    "urllib3==1.*",  # TODO: Pinned because vcrpy did a dumb and didn't pin urllib3
-    "flake8==5.*",  # TODO: flake8 v6 requires Python 3.8.1+
-    "isort==5.*",
-    "mypy==1.4.*",  # TODO: mypy v1.5 requires Python 3.8+
-    "pdoc==13.*",  # TODO: pdoc v14 requires Python 3.8+
-    "pytest-cov==4.*",
+    "bandit==1.8.*",
+    "black==25.*",
+    "build==1.2.*",
+    "flake8==6.*",
+    "isort==6.*",
+    "mypy==1.15.*",
+    "pdoc==14.*",  # v15 requires Python 3.9
+    "pytest-cov==5.*",  # v6 requires Python 3.9
     "pytest-vcr==1.*",
-    "pytest==7.*",
-    "vcrpy==4.*",  # TODO: vcrpy v5 requires Python 3.8+
+    "pytest==8.*",
+    "vcrpy==6.*",  # v7 requires Python 3.9
 ]
 
 with open("README.md", encoding="utf-8") as f:
@@ -55,17 +54,17 @@
         "Tracker": "https://github.com/EasyPost/easypost-python/issues",
         "Source": "https://github.com/EasyPost/easypost-python",
     },
-    python_requires=">=3.7, <4",
+    python_requires=">=3.8, <4",
     classifiers=[
         "Development Status :: 5 - Production/Stable",
         "Environment :: Web Environment",
         "Programming Language :: Python",
-        "Programming Language :: Python :: 3.7",
         "Programming Language :: Python :: 3.8",
         "Programming Language :: Python :: 3.9",
         "Programming Language :: Python :: 3.10",
         "Programming Language :: Python :: 3.11",
         "Programming Language :: Python :: 3.12",
+        "Programming Language :: Python :: 3.13",
         "Intended Audience :: Developers",
         "Operating System :: OS Independent",
         "License :: OSI Approved :: MIT License",

tests/test_address.py

@@ -36,8 +36,7 @@ def test_address_create_verify(incorrect_address, test_client):
 
     # Delivery verification assertions
     assert address.verifications.delivery.success is False
-    # TODO: details is not deserializing correctly, related to the larger "double EasyPostObject" wrapping issue
-    # assert address.verifications.delivery.details == {}
+    assert address.verifications.delivery.details.to_dict() == {}
     assert address.verifications.delivery.errors[0].code == "E.ADDRESS.NOT_FOUND"
     assert address.verifications.delivery.errors[0].field == "address"
     assert address.verifications.delivery.errors[0].suggestion is None