Skip to content

Conversation

@haozang54-source
Copy link
Contributor

@haozang54-source haozang54-source commented Oct 9, 2025

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

#697

背景

  • 新增并完善 Upload 组件单测,覆盖 props、events、slots 与关键分支(选择、校验、进度、成功、失败、预览、删除、禁用、受控/非受控、multiple、accept、children、addContent 等)
  • 使用 Vitest 顶部 vi.mock 方式稳定模拟 ../../_common/js/upload/main../../_common/js/upload/utils,避免 ESM 模块顺序问题
  • 优化测试结构与命名规范,按组件规范放置到 src/upload/__tests__/index.test.tsx
  • 生成并更新测试覆盖率徽章(npm run generate:coverage-badge),Upload 模块整体覆盖率达标
  • Upload 模块整体:行≈98.57%、语句≈98.57%、分支=100%、函数≈95.23%

📝 更新日志

  • fix(Upload): 修复禁用态与 max 边界下的交互行为,确保在达到上限时不再触发新增文件;max=0 时始终展示添加按钮
  • fix(Upload): 无 url 文件时不渲染图片但仍支持删除操作

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