cal/major-domo-database
1
0
Fork 0
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity
273 Commits 9 Branches 16 Tags 43 MiB
main
Commit Graph

3 Commits

Author SHA1 Message Date
Cal Corum
36b962e5d5 fix: update test limit to respect MAX_LIMIT=500 (#110) 
Closes #110

The test was sending limit=999 which exceeds MAX_LIMIT (500), causing
FastAPI to return 422. Changed to limit=500, which is sufficient to
cover all seasons for any player.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
 2026-04-08 12:55:35 +00:00
Cal Corum
c49f91cc19 test: update test_get_nonexistent_play to expect 404 after HTTPException fix
All checks were successful
Build Docker Image / build (pull_request) Successful in 1m3s
Details 
After handle_db_errors no longer catches HTTPException, GET /plays/999999999
correctly returns 404 instead of 500. Update the assertion and docstring
to reflect the fixed behavior.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
 2026-04-02 09:30:39 -05:00
Cal Corum
86f8495284 feat: Add group_by=sbaplayer to batting, pitching, and fielding endpoints
All checks were successful
Build Docker Image / build (pull_request) Successful in 2m32s
Details 
Enables career-total aggregation by real-world player identity (SbaPlayer)
across all seasons. JOINs StratPlay → Player to access Player.sbaplayer FK,
groups by that FK, and excludes players with null sbaplayer. Also refactors
stratplay router from single file into package and adds integration tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
 2026-02-17 17:42:14 -06:00