Dan Kenigsberg has submitted this change and it was merged. Change subject: tests: add HostStatsThread._getCpuCoreStats test ......................................................................
tests: add HostStatsThread._getCpuCoreStats test Next patch wants to refactor HostStatsThread._getCpuCoreStats into a free function, so this patch adds very basic test coverage. This test needs to be refactored as well, and will be done alongside the method refactoring. Change-Id: I6c6a82c77054e76d7d08322ad83d086302c2bbad Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/41372 Reviewed-by: Dan Kenigsberg <[email protected]> Continuous-Integration: Jenkins CI --- M tests/samplingTests.py M tests/vmfakelib.py 2 files changed, 54 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/41372 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c6a82c77054e76d7d08322ad83d086302c2bbad Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
