Skip to content

Add comprehensive vitest tests for chromatin.ts functions #159

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add comprehensive vitest tests for chromatin.ts functions #159

wants to merge 1 commit into from

Conversation

dvdkouril
Copy link
Member

Summary

  • Add comprehensive vitest tests for core chromatin.ts functions
  • Covers initScene, addStructureToScene, and ViewConfig integration
  • Tests computeSegments logic for chromosome/index continuity handling
  • Uses proper Apache Arrow tableFromArrays API for mock test data

Test Coverage

  • initScene: Tests empty scene creation
  • addStructureToScene: Tests structure addition with default/custom ViewConfigs
  • computeSegments: Tests segmentation logic for continuous/discontinuous data
  • ViewConfig integration: Tests various scale and color configuration options

Test plan

  • All 14 tests passing
  • Proper Apache Arrow API usage with tableFromArrays
  • Mock data structure matches real chromatin data format
  • Tests validate core functionality without DOM/renderer dependencies

🤖 Generated with Claude Code

@dvdkouril @claude
test: add comprehensive vitest tests for chromatin.ts functions
c4948ad 
- Add tests for initScene, addStructureToScene functions
- Add tests for ViewConfig integration with different scale/color options
- Add tests for computeSegments logic with chromosome/index continuity
- Use proper Apache Arrow tableFromArrays API for mock data
- All tests passing with proper test structure and coverage

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Jul 25, 2025

⚠️ No Changeset found

Latest commit: c4948ad

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dvdkouril