dream-num/Luckysheet

[Feature request]Add test module

Open

#235 opened on Nov 18, 2020

View on GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (2,236 forks)batch import
enhancementgood first issueproject

Repository metrics

Stars
 (15,390 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Unit testing, etc. It should be that each submission must pass the test before submitting the code

Recommend：Jest

单元测试，在提交代码之前必须通过测试才能提交上来

可以想到的测试模块：

  • 渲染类：sheet页渲染，保证会触发到workbookCreateAfter钩子；公式计算正确；单元格每种格式，富文本渲染正常；冻结行列
  • 界面操作类：滚动条拖动正常，鼠标滚动也正常，页面缩放正常，可正常点击工具栏/sheet页等所有按钮，右击菜单正常，键盘事件正常
  • API：调用正常，钩子触发正常
  • 共享编辑正常
  • 增强功能：图片，图表，批注，条件格式，筛选，sparkline，数据透视表及操作，数据验证
  • 其他

Contributor guide

Research direction
Explore Luckysheet's source code to understand its architecture and existing testing infrastructure. Then, set up Jest and write unit tests for sheet rendering, UI interactions, API calls, shared editing, and enhanced features like images, charts, and data validation.
Tech stack
javascript
Domain
frontend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-2 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
JavaScriptJest
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60