Skip to content
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.

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

feat: added a unit test to compute with store ids #48

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wwwehr
Copy link
Member

@wwwehr wwwehr commented Dec 12, 2024

This was the unit test I prepared to work out a fabric error from our FE's app. Looks like @pablojhl may have created another unit test that covers this case.

Given this could be a low value dupe, feel free to close w/o merge - I just want to use it to test.

@wwwehr
Copy link
Member Author

wwwehr commented Dec 12, 2024

@Envoy-VC FYI

Copy link

Coverage Report for ./client-vms

Status Category Percentage Covered / Total
🔵 Lines 88.05% 2028 / 2303
🔵 Statements 88.05% 2028 / 2303
🔵 Functions 81.39% 210 / 258
🔵 Branches 93.83% 365 / 389
File CoverageNo changed files found.
Generated in workflow #63 for commit eeb8662 by the Vitest Coverage Report Action

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