Repository navigation
Test every report and chat command; fix first-run crashes - #16
Merged
Merged
Conversation
…ound - tests/common.py: shared mocked fixture (no network); test_reports.py covers each chat command, chart, prepare_* function, cron report and podcast helper; test_finbot.py keeps plumbing checks - fix .price top with no period (date_range_english(None)), rating.py unbound emoji when nothing changed, trades.py and .watchlist crashing when their state file does not exist yet Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
…no period (defaults to 7 days) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
…s they found - tests/test_providers.py: Yahoo quote/detail/chart and Sharesight portfolios/trades/performance parsed from doc/ samples, plus HTTP errors, retries, exclusions, Telegram send failures and error reporting - tests/test_util.py: table-driven helper tests; tests/test_edges.py: empty/broken upstream data, bad arguments, rejected requests - every command test now asserts the worker did not report a crash - fixes: yahoo.fetch returned None (callers iterate), fetch_detail TypeError on result=null, util.get_holdings_and_watchlist crashed with no watchlist file, .marketcap with no data Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
…ed labels on zoomed charts - tests/test_charts.py: 1280x800, not blank, nothing on the image edge; blue/red by sign, row order, titles, report sort order; one tolerant golden image per chart type (UPDATE_GOLDEN=1 to refresh) - fix: zoomed bar charts (.buy/.sell ratings) with few rows or wide labels drew their labels over the left edge of the image Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
robdevops
force-pushed
the
claude/kind-newton-ad0z2q
branch
from
October 7, 2026 05:01
df59a69 to
ce98beb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
tests/common.py: shared mocked fixture (no network)tests/test_reports.py: every chat command, chart,prepare_*function, cron report and podcast helpertests/test_finbot.py: plumbing checks (chunking, timezones, WSGI handler).price topwith no period,rating.pyunboundemoji,trades.pyand.watchlistcrashing when their state file doesn't exist yet🤖 Generated with Claude Code
https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
Generated by Claude Code