Remove powermock from ldap & project-role-based plugin - #7658
Conversation
0df72bc to
cc46d5a
Compare
Codecov Report
@@ Coverage Diff @@
## main #7658 +/- ##
============================================
+ Coverage 13.13% 13.14% +0.01%
- Complexity 9147 9163 +16
============================================
Files 2728 2728
Lines 256997 256997
Branches 40067 40067
============================================
+ Hits 33747 33788 +41
+ Misses 219001 218954 -47
- Partials 4249 4255 +6 see 5 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
@blueorangutan package |
|
@vishesh92 a [SF] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 6281 |
|
@blueorangutan test |
|
@vishesh92 a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
[SF] Trillian test result (tid-6805)
|
|
@shwstppr please add an explanation for your approval |
yadvr
left a comment
There was a problem hiding this comment.
LGTM didn't test it, but I think we can rely on automated tests
|
Merging this based on code review, build/pkging checks, simulator/Github actions and smoketests. |
|
Thanks @vishesh92 feel to submit for other areas of codebase in subsequent PRs, thanks for your PR. |
Description
This PR removes powermock from tests & fixes the path for tests of project-role-based plugin.
Pending powermock usage after this PR
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?