langgenius/dify
1
0
Fork 0
mirror of https://github.com/langgenius/dify.git synced 2026-07-21 03:55:25 -04:00
Code Issues 775 Packages Projects Releases 10 Wiki Activity

[PR #32374] test(web): increase test coverage for model-provider-page folder #33698

New Issue
Open
opened 2026-02-21 20:53:44 -05:00 by yindo · 0 comments
yindo commented 2026-02-21 20:53:44 -05:00
Owner
Copy Link
Copy Source

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

State: open
Merged: No

Checklist

  • This change requires a documentation update
  • Five added/updated tests for the behavior covered in this change
  • Tests lint, tests, type check, and coverage for the updated model provider page specs
  • No backward-incompatible code paths were changed

Validation Run

Lint

pnpm lint app/components/header/account-setting/model-provider-page

  • Result: ✅ Passed

Type Check

pnpm type-check:tsgo

  • Result: ✅ Passed

Tests

pnpm test app/components/header/account-setting/model-provider-page

  • Result: ✅ Passed

Coverage

pnpm test:coverage app/components/header/account-setting/model-provider-page

  • Result: ✅ Passed

Coverage Report (12 Target Source Files)

Overall stats:

  • Statements: 97.34%
  • Branches: 89.36%
  • Functions: 98.67%
  • Lines: 97.52%
File Statements Branches Functions Lines
web/app/components/header/account-setting/model-provider-page/index.tsx 93.18% 89.13% 100.00% 95.12%
web/app/components/header/account-setting/model-provider-page/install-from-marketplace.tsx 100.00% 100.00% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/model-selector/deprecated-model-trigger.tsx 100.00% 100.00% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/model-selector/empty-trigger.tsx 100.00% 50.00% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/model-selector/feature-icon.tsx 100.00% 100.00% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/model-selector/index.tsx 100.00% 100.00% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/model-selector/model-trigger.tsx 100.00% 100.00% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/model-selector/popup-item.tsx 100.00% 92.30% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/model-selector/popup.tsx 97.22% 90.47% 100.00% 97.22%
web/app/components/header/account-setting/model-provider-page/provider-icon/index.tsx 100.00% 100.00% 100.00% 100.00%
web/app/components/header/account-setting/model-provider-page/system-model-selector/index.tsx 98.52% 84.21% 100.00% 98.18%
web/app/components/header/account-setting/model-provider-page/utils.ts 95.45% 73.52% 90.90% 95.45%

Fixes Issue - #32357

**Original Pull Request:** https://github.com/langgenius/dify/pull/32374 **State:** open **Merged:** No --- ## Checklist - [ ] This change requires a documentation update - [x] Five added/updated tests for the behavior covered in this change - [x] Tests lint, tests, type check, and coverage for the updated model provider page specs - [x] No backward-incompatible code paths were changed ## Validation Run ### Lint `pnpm lint app/components/header/account-setting/model-provider-page` - Result: ✅ Passed ### Type Check `pnpm type-check:tsgo` - Result: ✅ Passed ### Tests `pnpm test app/components/header/account-setting/model-provider-page` - Result: ✅ Passed ### Coverage `pnpm test:coverage app/components/header/account-setting/model-provider-page` - Result: ✅ Passed ## Coverage Report (12 Target Source Files) Overall stats: - Statements: 97.34% - Branches: 89.36% - Functions: 98.67% - Lines: 97.52% | File | Statements | Branches | Functions | Lines | |---|---:|---:|---:|---:| | web/app/components/header/account-setting/model-provider-page/index.tsx | 93.18% | 89.13% | 100.00% | 95.12% | | web/app/components/header/account-setting/model-provider-page/install-from-marketplace.tsx | 100.00% | 100.00% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/model-selector/deprecated-model-trigger.tsx | 100.00% | 100.00% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/model-selector/empty-trigger.tsx | 100.00% | 50.00% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/model-selector/feature-icon.tsx | 100.00% | 100.00% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/model-selector/index.tsx | 100.00% | 100.00% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/model-selector/model-trigger.tsx | 100.00% | 100.00% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/model-selector/popup-item.tsx | 100.00% | 92.30% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/model-selector/popup.tsx | 97.22% | 90.47% | 100.00% | 97.22% | | web/app/components/header/account-setting/model-provider-page/provider-icon/index.tsx | 100.00% | 100.00% | 100.00% | 100.00% | | web/app/components/header/account-setting/model-provider-page/system-model-selector/index.tsx | 98.52% | 84.21% | 100.00% | 98.18% | | web/app/components/header/account-setting/model-provider-page/utils.ts | 95.45% | 73.52% | 90.90% | 95.45% | Fixes Issue - #32357
yindo added the pull-request label 2026-02-21 20:53:44 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Knowledge Pipeline
Proposal
WIP
cloud
dependencies
feat:webapp
good first issue
outdated
performance
pull-request
Mirrored from GitHub Pull Request
 python:uv
refactor
size:L
stale
status: accepting prs
status: blocked
status: in discussion
stt & tts
vulnerablity
web
wont-fix
🌊 feat:workflow
🌚 invalid
🐞 bug
👻 feat:rag
💪 enhancement
📚 documentation
🔨 feat:tools
🙋‍♂️ question
🙏 help wanted
🤔 cant-reproduce
🤖 feat:agent
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#33698
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?