Dan Kenigsberg has submitted this change and it was merged.
Change subject: tests: Add FakeLVM LV properties tests ......................................................................
tests: Add FakeLVM LV properties tests
Building on the example environment in FakeLVMSimpleVGTests we create a single logical volume. We then verify that the properties and attributes of that LV match what is returned by the real lvm module. Some problems were found in FakeLVM and are resolved here.
Since createLV accepts the size in MB, two of the manifest_tests needed to be fixed.
Change-Id: I1bca30db49db84fc5d42ff8736a4c69f17b17247 Signed-off-by: Adam Litke alitke@redhat.com Reviewed-on: https://gerrit.ovirt.org/47859 Continuous-Integration: Jenkins CI Reviewed-by: Nir Soffer nsoffer@redhat.com --- M tests/manifest_tests.py M tests/storagefakelib.py M tests/storagefakelibTests.py 3 files changed, 58 insertions(+), 5 deletions(-)
Approvals: Adam Litke: Verified Nir Soffer: Looks good to me, approved Jenkins CI: Passed CI tests