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

Added mock unit test framework #71

Merged
merged 2 commits into from
Dec 16, 2022
Merged

Conversation

harishp8889
Copy link
Contributor

Signed-off-by: Harish P harish_p4@dell.com

@harishp8889 harishp8889 requested a review from a team as a code owner December 14, 2022 14:43
Copy link
Member

@glimchb glimchb left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link

codecov bot commented Dec 16, 2022

Codecov Report

Merging #71 (e4ba846) into main (65af0be) will decrease coverage by 29.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #71       +/-   ##
==========================================
- Coverage   37.21%   8.09%   -29.13%     
==========================================
  Files           1       1               
  Lines         309     309               
==========================================
- Hits          115      25       -90     
- Misses        186     282       +96     
+ Partials        8       2        -6     
Impacted Files Coverage Δ
goopicsi.go 8.09% <0.00%> (-29.13%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Harish P <harish_p4@dell.com>
Signed-off-by: Harish P <harish_p4@dell.com>
Copy link
Member

@glimchb glimchb left a comment

Choose a reason for hiding this comment

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

lgtm

@glimchb glimchb merged commit 6c24753 into opiproject:main Dec 16, 2022
@harishp8889 harishp8889 deleted the feature/tests branch December 19, 2022 08:23
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants