Removed FakeLVHDSR

it only defined a `__init__self` method.

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>
Add assertion to test_loadvdis

Asserting on the object's state instead of just running the method.

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>
Extract stubout_xapi

Having loads of arguments for a test function which are not used smells
bad. Creating a mixin to embody all the stubs, that are not relevant to
the test.

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>
Extract stubout_journaler_remove

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>
Use generic stubout method instead

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>
Use stubout for non-mocks, remove SMLog impl.

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>
Refactor: use stubs property

To get rid of code dublication around attribute access.

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>
Fix PEP8 violations, remove assertion comment

Shortened comments so they fit in 80 character limit, also removed
assertion message as that's redundant (already at the test's
description)

Signed-off-by: Mate Lakat <mate.lakat@citrix.com>