Skip to content

Conversation

yatbfm
Copy link
Contributor

@yatbfm yatbfm commented Jun 7, 2025

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Release
  • Site / documentation update
  • Demo update
  • Workflow
  • Other (about what?)

🔗 Related issue link

releated #232

🔗 Related PR link

🐞 Bugserver case id

💡 Background and solution

📝 Changelog

Add treemap related test cases. Test cell.color for single and multiple scenarios.

Language Changelog
🇺🇸 English Add treemap related test cases
🇨🇳 Chinese 添加树图相关测试用例

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@github-actions github-actions bot added the vmind label Jun 7, 2025

it('should generate correct basic treemap spec with one color', () => {
const { dataset } = parseCSVData(`Category-0,value
add,593
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

模拟数据输入的时候,可以不用这么多数据

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修改

describe('getChartSpecWithContext', () => {
it('should generate correct basic treemap spec', () => {
const { dataset } = parseCSVData(`Category-0,Category-1,Category-2,Category-3,value
query,methods,add,,593
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

模拟数据输入的时候,可以不用这么多数据

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants