diff --git a/tests/integration/test_stratplay_routes.py b/tests/integration/test_stratplay_routes.py index bcc9abf..9a0dfea 100644 --- a/tests/integration/test_stratplay_routes.py +++ b/tests/integration/test_stratplay_routes.py @@ -81,9 +81,9 @@ class TestRouteRegistration: for route, methods in EXPECTED_PLAY_ROUTES.items(): assert route in paths, f"Route {route} missing from OpenAPI schema" for method in methods: - assert ( - method in paths[route] - ), f"Method {method.upper()} missing for {route}" + assert method in paths[route], ( + f"Method {method.upper()} missing for {route}" + ) def test_play_routes_have_plays_tag(self, api): """All play routes should be tagged with 'plays'.""" @@ -96,9 +96,9 @@ class TestRouteRegistration: for method, spec in paths[route].items(): if method in ("get", "post", "patch", "delete"): tags = spec.get("tags", []) - assert ( - "plays" in tags - ), f"{method.upper()} {route} missing 'plays' tag, has {tags}" + assert "plays" in tags, ( + f"{method.upper()} {route} missing 'plays' tag, has {tags}" + ) @pytest.mark.post_deploy @pytest.mark.skip( @@ -124,9 +124,9 @@ class TestRouteRegistration: ]: params = paths[route]["get"].get("parameters", []) param_names = [p["name"] for p in params] - assert ( - "sbaplayer_id" in param_names - ), f"sbaplayer_id parameter missing from {route}" + assert "sbaplayer_id" in param_names, ( + f"sbaplayer_id parameter missing from {route}" + ) # --------------------------------------------------------------------------- @@ -493,10 +493,9 @@ class TestPlayCrud: assert result["id"] == play_id def test_get_nonexistent_play(self, api): - """GET /plays/999999999 returns an error (wrapped by handle_db_errors).""" + """GET /plays/999999999 returns 404 Not Found.""" r = requests.get(f"{api}/api/v3/plays/999999999", timeout=10) - # handle_db_errors wraps HTTPException as 500 with detail message - assert r.status_code == 500 + assert r.status_code == 404 assert "not found" in r.json().get("detail", "").lower() @@ -575,9 +574,9 @@ class TestGroupBySbaPlayer: ) assert r_seasons.status_code == 200 season_pas = [s["pa"] for s in r_seasons.json()["stats"]] - assert career_pa >= max( - season_pas - ), f"Career PA ({career_pa}) should be >= max season PA ({max(season_pas)})" + assert career_pa >= max(season_pas), ( + f"Career PA ({career_pa}) should be >= max season PA ({max(season_pas)})" + ) @pytest.mark.post_deploy def test_batting_sbaplayer_short_output(self, api):