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

Add test for controlsvc.go #1097

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Conversation

AaronH88
Copy link
Contributor

This test pushes unit test coverage from 77% to 83% for this file by adding a fake connection object that RunControlSvc will use.

Copy link

codecov bot commented Jul 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.38%. Comparing base (2e7695d) to head (bd545a0).
Report is 80 commits behind head on devel.

@@            Coverage Diff             @@
##            devel    #1097      +/-   ##
==========================================
+ Coverage   42.62%   43.38%   +0.75%     
==========================================
  Files          42       45       +3     
  Lines        6642     7222     +580     
==========================================
+ Hits         2831     3133     +302     
- Misses       3578     3856     +278     
  Partials      233      233              

see 6 files with indirect coverage changes

Components Coverage Δ
Go 42.86% <ø> (+0.24%) ⬆️
Receptorctl 49.31% <ø> (∅)

Copy link

sonarcloud bot commented Jul 25, 2024

@AaronH88 AaronH88 merged commit 89bd57b into ansible:devel Jul 31, 2024
20 checks passed
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.

3 participants