[PR #32314] test: add test cases for some base components #33662

Open
opened 2026-02-21 20:53:41 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/32314

State: open
Merged: No


Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

Component File Stmts Branch Funcs Lines
mermaid index.tsx 92.02% 79.67% 95.83% 91.93%
mermaid utils.ts 100% 100% 100% 100%
message-log-modal index.tsx 100% 90.9% 100% 100%
modal index.tsx 100% 100% 100% 100%
modal modal.tsx 100% 91.3% 100% 100%
modal-like-wrap index.tsx 100% 100% 100% 100%
popover index.tsx 100% 93.93% 100% 100%
progress-bar progress-circle.tsx 100% 100% 100% 100%
prompt-log-modal card.tsx 100% 100% 100% 100%
prompt-log-modal index.tsx 100% 87.5% 100% 100%
qrcode index.tsx 96% 75% 100% 95.83%
simple-pie-chart index.module.css 100% 100% 100% 100%
simple-pie-chart index.tsx 100% 100% 100% 100%
svg index.tsx 100% 100% 100% 100%
svg style.module.css 100% 100% 100% 100%
svg-gallery index.tsx 100% 75% 100% 100%

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint and make type-check (backend) and cd web && npx lint-staged (frontend) to appease the lint gods

fixes: #32152

**Original Pull Request:** https://github.com/langgenius/dify/pull/32314 **State:** open **Merged:** No --- > [!IMPORTANT] > > 1. Make sure you have read our [contribution guidelines](https://github.com/langgenius/dify/blob/main/CONTRIBUTING.md) > 1. Ensure there is an associated issue and you have been assigned to it > 1. Use the correct syntax to link this PR: `Fixes #<issue number>`. ## Summary | Component | File | Stmts | Branch | Funcs | Lines | |---|---|---|---|---|---| | mermaid | index.tsx | 92.02% | 79.67% | 95.83% | 91.93% | | mermaid | utils.ts | 100% | 100% | 100% | 100% | | message-log-modal | index.tsx | 100% | 90.9% | 100% | 100% | | modal | index.tsx | 100% | 100% | 100% | 100% | | modal | modal.tsx | 100% | 91.3% | 100% | 100% | | modal-like-wrap | index.tsx | 100% | 100% | 100% | 100% | | popover | index.tsx | 100% | 93.93% | 100% | 100% | | progress-bar | progress-circle.tsx | 100% | 100% | 100% | 100% | | prompt-log-modal | card.tsx | 100% | 100% | 100% | 100% | | prompt-log-modal | index.tsx | 100% | 87.5% | 100% | 100% | | qrcode | index.tsx | 96% | 75% | 100% | 95.83% | | simple-pie-chart | index.module.css | 100% | 100% | 100% | 100% | | simple-pie-chart | index.tsx | 100% | 100% | 100% | 100% | | svg | index.tsx | 100% | 100% | 100% | 100% | | svg | style.module.css | 100% | 100% | 100% | 100% | | svg-gallery | index.tsx | 100% | 75% | 100% | 100% | ## Screenshots | Before | After | |--------|-------| | ... | ... | ## Checklist - [ ] This change requires a documentation update, included: [Dify Document](https://github.com/langgenius/dify-docs) - [x] I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!) - [x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change. - [x] I've updated the documentation accordingly. - [x] I ran `make lint` and `make type-check` (backend) and `cd web && npx lint-staged` (frontend) to appease the lint gods fixes: #32152
yindo added the pull-request label 2026-02-21 20:53:41 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#33662