Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Add unit tests #11

Closed
minghinmatthewlam opened this issue Aug 28, 2023 · 0 comments · Fixed by #21
Closed

Add unit tests #11

minghinmatthewlam opened this issue Aug 28, 2023 · 0 comments · Fixed by #21
Assignees
@geoff-vball
Labels
enhancement New feature or request tests Improving test coverage through unit/e2e tests, etc.

Comments

@minghinmatthewlam
Copy link

Context and scope
Unit test coverage is sparse. We should strive for full test coverage in everything below the application entry point.

Open questions
For mocks, should we try to use mockgen or any other standardized approach for this?
@minghinmatthewlam minghinmatthewlam added enhancement New feature or request tests Improving test coverage through unit/e2e tests, etc. labels Aug 28, 2023
@minghinmatthewlam minghinmatthewlam moved this from 📋 Backlog to 🏗 In progress in Avalanche Warp Messaging Aug 29, 2023
@minghinmatthewlam minghinmatthewlam mentioned this issue Aug 30, 2023
Merged
@minghinmatthewlam minghinmatthewlam moved this from 🏗 In progress to 👀 In review in Avalanche Warp Messaging Aug 30, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Avalanche Warp Messaging Sep 22, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@geoff-vball geoff-vball

Labels
enhancement New feature or request tests Improving test coverage through unit/e2e tests, etc.
Projects
Platform Engineering Group
Archived in project
Avalanche Warp Messaging
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Unit tests ava-labs/icm-services
2 participants
@geoff-vball @minghinmatthewlam