Skip to content

test: add unit tests for SymbolLegend mark (#172)#375

Merged
gka merged 3 commits intosvelteplot:mainfrom
ljodea:test/symbollegend-172
Feb 18, 2026
Merged

test: add unit tests for SymbolLegend mark (#172)#375
gka merged 3 commits intosvelteplot:mainfrom
ljodea:test/symbollegend-172

Conversation

@ljodea
Copy link
Copy Markdown
Collaborator

@ljodea ljodea commented Feb 15, 2026

Summary

  • Adds 9 unit tests for the SymbolLegend mark, covering all 6 code branches
  • Tests verify rendering behavior, label text, SVG path generation, distinct symbol shapes, color/fill/stroke logic based on colorSymbolRedundant and hasFilledDotMarks flags

Test plan

  • All 9 new tests pass
  • Full test suite (460 tests) still passing
  • Lint clean

Closes #172

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 15, 2026

Deploy Preview for svelteplot failed.

Name Link
🔨 Latest commit 4cb69d3
🔍 Latest deploy log https://app.netlify.com/projects/svelteplot/deploys/6994cb15fd4ec60008b49c86

@gka gka merged commit a90b1f7 into svelteplot:main Feb 18, 2026
5 of 9 checks passed
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.

Add unit tests for SymbolLegend mark

2 participants