paper-dynasty-discord/tests/scouting
Cal Corum d569e91905 fix: Address PR review findings — two bugs and cleanup
- Fix int_timestamp() no-arg path returning seconds instead of
  milliseconds, which would silently break the daily scout token cap
  against the real API
- Acknowledge double-click interactions with ephemeral message instead
  of silently returning (Discord requires all interactions to be acked)
- Reorder scout flow: create card copy before consuming token so a
  failure doesn't cost the player a token for nothing
- Move build_scouted_card_list import to top of scout_view.py
- Remove unused asyncio import from helpers/scouting.py
- Fix footer text inconsistency ("One scout per player" everywhere)
- Update tests for new operation order and double-click behavior

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-03-05 03:04:53 +00:00
..
__init__.py refactor: Consolidate scouting utilities, add test suite, use Discord timestamps 2026-03-05 03:04:53 +00:00
conftest.py refactor: Consolidate scouting utilities, add test suite, use Discord timestamps 2026-03-05 03:04:53 +00:00
test_scout_view.py fix: Address PR review findings — two bugs and cleanup 2026-03-05 03:04:53 +00:00
test_scouting_cog.py refactor: Consolidate scouting utilities, add test suite, use Discord timestamps 2026-03-05 03:04:53 +00:00
test_scouting_helpers.py refactor: Consolidate scouting utilities, add test suite, use Discord timestamps 2026-03-05 03:04:53 +00:00