test_basic_without_ssh: use a fixture instead of expecting test_start

"test_start" was never a real test here, turning it into official fixture
makes sure that the VM is started for any test.

There is no other inter-test dependency in this file, this allows to stop
using @pytest.mark.incremental here.

Signed-off-by: Yann Dirson <[email protected]>

Author: ydirson

tests/misc/test_basic_without_ssh.py

@@ -25,11 +25,11 @@ def existing_shared_sr(host):
     assert sr is not None, "A shared SR on the pool is required"
     return sr
 
-@pytest.mark.incremental # tests depend on each other. If one test fails, don't execute the others
 @pytest.mark.multi_vms # run them on a variety of VMs
 @pytest.mark.big_vm # and also on a really big VM ideally
 class TestBasicNoSSH:
-    def test_start(self, imported_vm):
+    @pytest.fixture(scope="class", autouse=True)
+    def vm_start(self, imported_vm):
         vm = imported_vm
         # if VM already running, stop it
         if (vm.is_running()):