Skip to content

fix(graph): add missing lucide icons and fix SVG test queries in graph-view-coverage - #571

Merged
d-oit merged 1 commit into
mainfrom
feat/graph-view-coverage-fixes
Jul 31, 2026
Merged

fix(graph): add missing lucide icons and fix SVG test queries in graph-view-coverage#571
d-oit merged 1 commit into
mainfrom
feat/graph-view-coverage-fixes

Conversation

@d-oit

@d-oit d-oit commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Fix: Missing lucide icons and SVG test queries in graph-view-coverage

  • Added FileText, FileJson, FileCode, FileArchive, FileLock to lucide-react mock
  • Fixed SVG tabIndex assertion (lowercase tabindex for SVG)
  • Fixed SVG role=img query (jsdom limitation)

All 130 test files pass, 1891 tests.

@vercel

vercel Bot commented Jul 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
do-knowledge-studio Ready Ready Preview, v0 Jul 30, 2026 7:11pm

@github-actions github-actions Bot added config tests Related to automated/manual tests labels Jul 30, 2026
@codacy-production

Copy link
Copy Markdown
Contributor

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@d-oit
d-oit merged commit 23ff512 into main Jul 31, 2026
23 checks passed
@d-oit
d-oit deleted the feat/graph-view-coverage-fixes branch July 31, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config tests Related to automated/manual tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant