bmc-toolbox / bmclib

Library to abstract Baseboard Management Controller interaction
Apache License 2.0
193 stars 36 forks source link

[1/4] Move methods under the redfishwrapper for reuse #369

Closed joelrebel closed 11 months ago

joelrebel commented 11 months ago

What does this PR implement/change/remove?

This is to allow each vendor/OEM provider to re-use and extend these methods.

Checklist

joelrebel commented 11 months ago

All OK but test and lint fail

@ofaurax this is 1/4 PR, the 4th PR https://github.com/bmc-toolbox/bmclib/pull/372 when merged will clear tests and linting

codecov[bot] commented 11 months ago

Codecov Report

Attention: 226 lines in your changes are missing coverage. Please review.

Comparison is base (b565dd4) 44.60% compared to head (ee0f228) 43.87%.

Files Patch % Lines
providers/dell/firmware.go 9.34% 96 Missing and 1 partial :warning:
internal/redfishwrapper/inventory.go 11.76% 45 Missing :warning:
internal/redfishwrapper/inventory_collect.go 10.34% 26 Missing :warning:
providers/supermicro/supermicro.go 30.43% 14 Missing and 2 partials :warning:
internal/redfishwrapper/power.go 0.00% 14 Missing :warning:
client.go 0.00% 7 Missing :warning:
internal/redfishwrapper/task.go 76.19% 3 Missing and 2 partials :warning:
providers/redfish/redfish.go 0.00% 5 Missing :warning:
providers/dell/idrac.go 69.23% 4 Missing :warning:
internal/redfishwrapper/system.go 40.00% 3 Missing :warning:
... and 3 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## devel-rework-redfishwrapper #369 +/- ## =============================================================== - Coverage 44.60% 43.87% -0.74% =============================================================== Files 61 59 -2 Lines 5206 5042 -164 =============================================================== - Hits 2322 2212 -110 + Misses 2628 2608 -20 + Partials 256 222 -34 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.