Fix handling empty vm_type in Store.last_buildroot

osc/store.py

@@ -119,7 +119,7 @@ def write_list(self, fn, value, subdir=None):
         if not isinstance(value, (list, tuple)):
             msg = f"The argument `value` should be list, not {type(value).__name__}"
             raise TypeError(msg)
-        value = "".join((f"{line}\n" for line in value))
+        value = "".join((f"{line or ''}\n" for line in value))
         self.write_file(fn, value, subdir=subdir)
 
     def read_string(self, fn, subdir=None):
@@ -291,6 +291,8 @@ def last_buildroot(self):
         if items is not None and len(items) != 3:
             msg = f"Package '{self.path}' contains _last_buildroot metadata that doesn't contain 3 lines: [repo, arch, vm_type]"
             raise oscerr.NoWorkingCopy(msg)
+        if items[2] in ("", "None"):
+            items[2] = None
         return items
 
     @last_buildroot.setter

tests/test_git_scm_store.py

@@ -36,6 +36,14 @@ def test_last_buildroot(self):
         store = GitStore(self.tmpdir)
         self.assertEqual(store.last_buildroot, ("repo", "arch", "vm_type"))
 
+    def test_last_buildroot_empty_vm_type(self):
+        store = GitStore(self.tmpdir)
+        self.assertEqual(store.last_buildroot, None)
+        store.last_buildroot = ("repo", "arch", None)
+
+        store = GitStore(self.tmpdir)
+        self.assertEqual(store.last_buildroot, ("repo", "arch", None))
+
     def test_scmurl(self):
         store = GitStore(self.tmpdir)
         self.assertEqual(store.scmurl, "https://example.com/packages/my-package.git")

tests/test_store.py

@@ -203,6 +203,12 @@ def test_last_buildroot(self):
         store2 = Store(self.tmpdir)
         self.assertEqual(store2.last_buildroot, ["repo", "arch", "vm_type"])
 
+        self.store.last_buildroot = "repo", "arch", None
+        self.fileEquals("_last_buildroot", "repo\narch\n\n")
+
+        store2 = Store(self.tmpdir)
+        self.assertEqual(store2.last_buildroot, ["repo", "arch", None])
+
     def test_meta_node(self):
         self.store.write_string(
             "_meta",