fix: wizard formatting + test status string fix

Author: nkanu17

redisvl/migration/wizard.py

@@ -130,7 +130,6 @@ def _filter_staged_adds(
         ]
         return filtered
 
-
     def _apply_staged_changes(
         self,
         source_schema: Dict[str, Any],
@@ -225,9 +224,7 @@ def _build_patch(
                     staged_add_names = {f["name"] for f in changes.add_fields}
                     if field_name in staged_add_names:
                         changes.add_fields = [
-                            f
-                            for f in changes.add_fields
-                            if f["name"] != field_name
+                            f for f in changes.add_fields if f["name"] != field_name
                         ]
                         print(f"Cancelled staged addition of '{field_name}'.")
                     else:

tests/unit/test_migration_wizard.py

@@ -652,8 +652,9 @@ def test_negative_m_ignored(self, monkeypatch):
         wizard = MigrationWizard()
         patch = wizard._build_patch(source_schema)
 
-        update = patch.changes.update_fields[0]
-        assert "m" not in update.attrs  # Negative should be ignored
+        # Negative M is ignored, and since algorithm/datatype/metric are unchanged,
+        # no update should be generated at all
+        assert len(patch.changes.update_fields) == 0
 
     def test_float_m_ignored(self, monkeypatch):
         """Test that float M value is ignored."""
@@ -678,8 +679,9 @@ def test_float_m_ignored(self, monkeypatch):
         wizard = MigrationWizard()
         patch = wizard._build_patch(source_schema)
 
-        update = patch.changes.update_fields[0]
-        assert "m" not in update.attrs
+        # Float M is ignored, and since algorithm/datatype/metric are unchanged,
+        # no update should be generated at all
+        assert len(patch.changes.update_fields) == 0
 
     def test_string_m_ignored(self, monkeypatch):
         """Test that string M value is ignored."""
@@ -704,8 +706,9 @@ def test_string_m_ignored(self, monkeypatch):
         wizard = MigrationWizard()
         patch = wizard._build_patch(source_schema)
 
-        update = patch.changes.update_fields[0]
-        assert "m" not in update.attrs
+        # String M is ignored, and since algorithm/datatype/metric are unchanged,
+        # no update should be generated at all
+        assert len(patch.changes.update_fields) == 0
 
     def test_zero_m_accepted(self, monkeypatch):
         """Test that zero M is accepted (validation happens at schema level)."""