Do not remove an empty subvolumes list when cloning

src/modules/AutoinstPartition.rb

@@ -279,7 +279,7 @@ def parsePartition(part)
         newPart["subvolumes"] = part["subvolumes"].reject do |subvolume|
           subvolume["path"].start_with?(".snapshots")
         end
-      else
+      elsif newPart["filesystem"] != :btrfs
         newPart = Builtins.remove(newPart, "subvolumes")
       end
       if !Builtins.isempty(Ops.get_string(part, "used_pool", ""))

test/AutoinstPartition_test.rb

@@ -0,0 +1,50 @@
+#!/usr/bin/env rspec
+
+require_relative "test_helper"
+
+Yast.import "AutoinstPartition"
+
+describe "Yast::AutoinstPartition" do
+  subject { Yast::AutoinstPartition }
+
+  describe "#parsePartition" do
+    let(:filesystem) { :btrfs }
+    let(:partition) do
+      {
+        "filesystem" => filesystem,
+        "subvolumes" => [
+          { "path" => ".snapshots/1" },
+          { "path" => "var/lib/machines" }
+        ]
+      }
+    end
+
+    it "filters out snapper snapshots" do
+      parsed = subject.parsePartition(partition)
+
+      expect(parsed["subvolumes"]).to eq(
+        [{ "path" => "var/lib/machines" }]
+      )
+    end
+
+    context "when there are no Btrfs subvolumes" do
+      let(:partition) { { "subvolumes" => [] } }
+
+      it "exports them as an empty array" do
+        parsed = subject.parsePartition(partition)
+
+        expect(parsed["subvolumes"]).to eq([])
+      end
+
+      context "and filesystem is not Btrfs" do
+        let(:filesystem) { :ext4 }
+
+        it "does not export the subvolumes list" do
+          parsed = subject.parsePartition(partition)
+
+          expect(parsed["subvolumes"]).to eq([])
+        end
+      end
+    end
+  end
+end

test/Makefile.am

@@ -9,6 +9,7 @@ TESTS = \
     AutoinstConfig_tests.rb \
     AutoinstFunctions_test.rb \
     AutoinstGeneral_test.rb \
+    AutoinstPartition_test.rb \
     AutoinstPartPlan_test.rb \
     AutoinstSoftware_test.rb \
     profile_test.rb \