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.

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

[D&D] Add unit test coverage #1704

Closed
joshuarrrr opened this issue Jun 8, 2022 · 2 comments
Closed

[D&D] Add unit test coverage #1704

joshuarrrr opened this issue Jun 8, 2022 · 2 comments
Labels
medium priority test:unit vis builder

Comments

@joshuarrrr
Copy link
Member

Since the code is forked from the initial POC, most of it has no unit tests. We should initially aim to at least match overall repo coverage levels, and then continue from there.
@joshuarrrr joshuarrrr mentioned this issue Jun 8, 2022
Open
@ashwin-pc ashwin-pc self-assigned this Jul 6, 2022
@kavilla
Copy link
Member

kavilla commented Aug 15, 2022

Will ice-box this for now and rely on functional tests to verify this.

@ashwin-pc
Copy link
Member

Closing this in place of #2380 where i have listed the actual tests to cover

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
medium priority test:unit vis builder
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@joshuarrrr @kavilla @ashwin-pc