refactor(tests): rename test_5_snapshots and reduce the number of cycles

Rename test to `test_cycled_snapshot_restore` and add a better comment
to better explain what the test does. Also reduce number of cycles to 3
to speed test a bit.

Signed-off-by: Egor Lazarchuk <[email protected]>

Author: ShadowCurse

tests/integration_tests/functional/test_snapshot_basic.py

@@ -113,7 +113,7 @@ def test_snapshot_current_version(uvm_nano):
 # TODO: Multiple microvm sizes must be tested in the async pipeline.
 @pytest.mark.parametrize("snapshot_type", [SnapshotType.DIFF, SnapshotType.FULL])
 @pytest.mark.parametrize("use_snapshot_editor", [False, True])
-def test_5_snapshots(
+def test_cycled_snapshot_restore(
     bin_vsock_path,
     tmp_path,
     microvm_factory,
@@ -124,10 +124,14 @@ def test_5_snapshots(
     cpu_template_any,
 ):
     """
-    Create and load 5 snapshots.
+    Run a cycle of VM restoration and VM snapshot creation where new VM is
+    restored from a snapshot of the previous one.
     """
+    # This is an arbitrary selected value. It is big enough to test the
+    # functionality, but small enough to not be annoying long to run.
+    cycles = 3
+
     logger = logging.getLogger("snapshot_sequence")
-    seq_len = 5
     diff_snapshots = snapshot_type == SnapshotType.DIFF
 
     vm = microvm_factory.build(guest_kernel, rootfs)
@@ -155,7 +159,7 @@ def test_5_snapshots(
     vm.kill()
 
     for microvm in microvm_factory.build_n_from_snapshot(
-        snapshot, seq_len, incremental=True, use_snapshot_editor=use_snapshot_editor
+        snapshot, cycles, incremental=True, use_snapshot_editor=use_snapshot_editor
     ):
         # FIXME: This and the sleep below reduce the rate of vsock/ssh connection
         # related spurious test failures, although we do not know why this is the case.