test_5_snapshots: clarify variable usage

Only keep a single "snapshot" variable outside the loop

Signed-off-by: Patrick Roy <[email protected]>

Author: roypat

tests/integration_tests/functional/test_snapshot_basic.py

@@ -157,7 +157,6 @@ def test_5_snapshots(
     # Create a snapshot from a microvm.
     start_guest_echo_server(vm)
     snapshot = vm.make_snapshot(snapshot_type)
-    base_snapshot = snapshot
     vm.kill()
 
     for i in range(seq_len):
@@ -183,20 +182,20 @@ def test_5_snapshots(
 
         time.sleep(2)
         logger.info("Create snapshot %s #%d.", snapshot_type, i + 1)
-        snapshot = microvm.make_snapshot(snapshot_type)
+        new_snapshot = microvm.make_snapshot(snapshot_type)
 
         # If we are testing incremental snapshots we must merge the base with
         # current layer.
         if snapshot.is_diff:
-            logger.info("Base: %s, Layer: %s", base_snapshot.mem, snapshot.mem)
-            snapshot = snapshot.rebase_snapshot(
-                base_snapshot, use_snapshot_editor=use_snapshot_editor
+            logger.info("Base: %s, Layer: %s", snapshot.mem, new_snapshot.mem)
+            new_snapshot = new_snapshot.rebase_snapshot(
+                snapshot, use_snapshot_editor=use_snapshot_editor
             )
 
         microvm.kill()
         copied_snapshot.delete()
         # Update the base for next iteration.
-        base_snapshot = snapshot
+        snapshot = new_snapshot
 
 
 def test_patch_drive_snapshot(uvm_nano, microvm_factory):