Skip to content

fix: repair frontend test mocks for Watchlist & Charts (fin-a70r)#3

Open
team2human wants to merge 2 commits intoed-donner:mainfrom
team2human:polecat/quartz-mnyqpcxl
Open

fix: repair frontend test mocks for Watchlist & Charts (fin-a70r)#3
team2human wants to merge 2 commits intoed-donner:mainfrom
team2human:polecat/quartz-mnyqpcxl

Conversation

@team2human
Copy link
Copy Markdown

Summary

  • Fix PortfolioPage test mocks for lightweight-charts v5 and jsdom compatibility
  • Add missing browser globals (ResizeObserver, EventSource) to test setup
  • Add CLAUDE.md to .gitignore

Context

Issue fin-a70r: Frontend Watchlist Panel, Sparklines & Charts — part of the FinAlly build.

Test plan

  • cd frontend && npm test passes

🤖 Generated with Claude Code

quartz and others added 2 commits April 14, 2026 16:54
…dom (fin-a70r)

Update lightweight-charts mock to use addSeries/LineSeries (v5 API) instead of
the removed addLineSeries/LineType. Add addEventListener to EventSource mock and
ResizeObserver polyfill to test setup so all 37 tests pass.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Auto-generated Gas Town infrastructure file should not be tracked.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant