feat(ruby): collect Agama logs after the installation

* Save the logs into the installed system.
* Set the proper permissions (700) to the
  /var/log/agama-installation folder.

Author: imobachgs

service/lib/agama/storage/finisher.rb

@@ -28,8 +28,10 @@
 require "agama/helpers"
 require "agama/http"
 require "abstract_method"
+require "fileutils"
 
 Yast.import "Arch"
+Yast.import "Installation"
 
 module Agama
   module Storage
@@ -217,21 +219,31 @@ def label
         end
 
         def run
-          wfm_write("copy_logs_finish")
-          copy_agama_scripts
+          FileUtils.mkdir_p(logs_dir, mode: 0o700)
+          collect_logs
+          copy_scripts
         end
 
       private
 
-        def copy_agama_scripts
+        def copy_scripts
           return unless Dir.exist?(SCRIPTS_DIR)
 
-          Yast.import "Installation"
-          require "fileutils"
-          logs_dir = File.join(Yast::Installation.destdir, "var", "log", "agama-installation")
-          FileUtils.mkdir_p(logs_dir)
           FileUtils.cp_r(SCRIPTS_DIR, logs_dir)
         end
+
+        def collect_logs
+          path = File.join(logs_dir, "logs")
+          Yast::Execute.locally(
+            "agama", "logs", "store", "--destination", path
+          )
+        end
+
+        def logs_dir
+          @logs_dir ||= File.join(
+            Yast::Installation.destdir, "var", "log", "agama-installation"
+          )
+        end
       end
 
       # Executes post-installation scripts

service/test/agama/storage/finisher_test.rb

@@ -141,3 +141,35 @@
 
   include_examples "progress"
 end
+
+describe Agama::Storage::Finisher::CopyLogsStep do
+  let(:logger) { Logger.new($stdout, level: :warn) }
+  let(:scripts_dir) { File.join(tmp_dir, "run", "agama", "scripts") }
+  let(:tmp_dir) { Dir.mktmpdir }
+
+  subject { Agama::Storage::Finisher::CopyLogsStep.new(logger) }
+
+  before do
+    allow(Yast::Installation).to receive(:destdir).and_return(File.join(tmp_dir, "mnt"))
+    allow(Yast::Execute).to receive(:locally)
+    stub_const("Agama::Storage::Finisher::CopyLogsStep::SCRIPTS_DIR",
+      File.join(tmp_dir, "run", "agama", "scripts"))
+  end
+
+  after do
+    FileUtils.remove_entry(tmp_dir)
+  end
+
+  context "when scripts artifacts exist" do
+    before do
+      FileUtils.mkdir_p(scripts_dir)
+      FileUtils.touch(File.join(scripts_dir, "test.sh"))
+    end
+
+    it "copies the artifacts to the installed system" do
+      subject.run
+      expect(File).to exist(File.join(tmp_dir, "mnt", "var", "log", "agama-installation",
+        "scripts"))
+    end
+  end
+end

service/test/agama/storage/manager_test.rb

@@ -361,6 +361,7 @@
       allow(File).to receive(:directory?).with("/iguana").and_return iguana
       allow(copy_files_class).to receive(:new).and_return(copy_files)
       allow(Yast::Execute).to receive(:on_target!)
+      allow(Yast::Execute).to receive(:local)
     end
     let(:copy_files_class) { Agama::Storage::Finisher::CopyFilesStep }
     let(:copy_files) { instance_double(copy_files_class, run?: true, run: true, label: "Copy") }
@@ -378,8 +379,10 @@
       expect(scripts_client).to receive(:run).with("post")
       expect(Yast::Execute).to receive(:on_target!)
         .with("systemctl", "enable", "agama-scripts", allowed_exitstatus: [0, 1])
-      expect(Yast::WFM).to receive(:CallFunction).with("copy_logs_finish", ["Write"])
       expect(Yast::WFM).to receive(:CallFunction).with("umount_finish", ["Write"])
+      expect(Yast::Execute).to receive(:locally).with(
+        "agama", "logs", "store", "--destination", /\/var\/log\/agama-installation\/logs/
+      )
       storage.finish
     end