Skip to content

Conversation

gaby
Copy link
Collaborator

@gaby gaby commented Mar 26, 2025

  • Improve tests coverage for maker.go from 67.1% to 96.8%

Related #70

@gaby
Copy link
Collaborator Author

gaby commented Mar 26, 2025

@grivera64 Can you take a quick look at this. Separate PR will deal with ifacemaker.go

Copy link
Contributor

@grivera64 grivera64 left a comment

Choose a reason for hiding this comment

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

Aside from a comment on the maker_test.TestMakeDuplicateMethods() unit-test, LGTM! 🚀

@gaby gaby changed the title chore: Improve tests coverage for maker.go chore: Improve tests coverage of maker.go by 30% Mar 27, 2025
@gaby gaby requested a review from grivera64 March 27, 2025 03:23
@gaby gaby merged commit 854fbd5 into master Mar 28, 2025
9 checks passed
@gaby gaby deleted the test-coverage branch April 5, 2025 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants