On Tue, Apr 16, 2019 at 12:44:26AM +0200, Ján Tomko wrote: > On Mon, Apr 15, 2019 at 02:15:28PM -0400, Cole Robinson wrote: > > On 4/15/19 11:25 AM, Pavel Hrdina wrote: > > > Calling fixtures directly was removed in pytest 4.0, we can change the > > > fixture to be a wrapper around the original function and use the > > > original fixture name. > > Would be nice to call out the direct caller here: > test_connect_node_device_create_xml
Will do before pushing. > > > > > > Signed-off-by: Pavel Hrdina <phrd...@redhat.com> > > > --- > > > tests/libvirttest.py | 15 +++++++++------ > > > 1 file changed, 9 insertions(+), 6 deletions(-) > > > > > Reviewed-by: Ján Tomko <jto...@redhat.com> > > > > diff --git a/tests/libvirttest.py b/tests/libvirttest.py > > > index 14baf5b..a442196 100644 > > > --- a/tests/libvirttest.py > > > +++ b/tests/libvirttest.py > > > @@ -84,13 +84,7 @@ class BaseTestClass(): > > > interface_obj.Create(0) > > > return path, interface_obj > > > > > > - @pytest.fixture > > > def node_device_create(self): > > > - """ Fixture to create dummy node device on the test driver > > > - > > > - This fixture should be used in the setup of every test > > > manipulating > > > - with node devices. > > > - """ > > > # We need a usable parent nodedev: possible candidates are > > > # scsi_host2 (available since libvirt 3.1.0) and > > > # test-scsi-host-vport (available until libvirt 3.0.0). > > > @@ -109,6 +103,15 @@ class BaseTestClass(): > > > path = self.connect.NodeDeviceCreateXML(xml, 0) > > > return path > > > > > > + @pytest.fixture(name="node_device_create") > > > + def fixture_node_device_create(self): > > > + """ Fixture to create dummy node device on the test driver > > > + > > > + This fixture should be used in the setup of every test > > > manipulating > > > + with node devices. > > > + """ > > > + return self.node_device_create() > > > + > > > @pytest.fixture > > > def storage_volume_create(self): > > > """ Fixture to create dummy storage volume on the test driver > > > > > > > Maybe I misunderstand but doesn't this storage_volume_create case > > follow same pattern? > > > > There is neither a direct caller of storage_volume_create nor an > equivalent test_connect_storage_volume_create_xml test. > The difference would be that libvirt doesn't have an event on storage volume > creation. Correct, for now we don't have the same issue here so there is no need for the wrapper. If in the future we will have to call the function directly the patch should rewrite the fixture to this pattern. Thanks, Pavel
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list