-
Notifications
You must be signed in to change notification settings - Fork 91
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
1860 adding tests for the executor resource deployment files #1893
base: develop
Are you sure you want to change the base?
1860 adding tests for the executor resource deployment files #1893
Conversation
Aravind-psiog
commented
Dec 22, 2023
•
edited
Loading
edited
- I have added the tests to cover my changes.
- I have updated the documentation and CHANGELOG accordingly.
- I have read the CONTRIBUTING document.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #1893 +/- ##
===========================================
+ Coverage 84.52% 85.26% +0.74%
===========================================
Files 295 181 -114
Lines 14488 11267 -3221
Branches 195 0 -195
===========================================
- Hits 12246 9607 -2639
+ Misses 2108 1660 -448
+ Partials 134 0 -134
|
@kessler-frost @araghukas It appears that the test coverage for the cloud_resource_manager is at a satisfactory 96%. However, the coverage for the cli groups are at 55%. The local coverage for this was improved by installing plugins, but these changes have not been pushed to the repository because the CI environment does not install plugins, resulting in test failures. |
…ce-deployment-files