Add unit tests

library/system/test/execute_test.rb

@@ -10,63 +10,137 @@
     expect(Cheetah.default_options[:logger]).to eq Yast::Y2Logger.instance
   end
 
-  describe ".locally" do
+  describe "#locally" do
+    it "returns a chaining object if no argumens are given" do
+      expect(subject.locally).to be_a(Yast::Execute)
+    end
+
     it "passes arguments directly to cheetah" do
       expect(Cheetah).to receive(:run).with("ls", "-a")
 
-      Yast::Execute.locally("ls", "-a")
+      subject.locally("ls", "-a")
     end
 
     it "report error if command execution failed" do
       expect(Yast::Report).to receive(:Error)
-      Yast::Execute.locally("false")
+      subject.locally("false")
     end
 
     it "returns nil if command execution failed" do
-      expect(Yast::Execute.locally("false")).to eq nil
+      expect(subject.locally("false")).to eq nil
     end
   end
 
-  describe ".locally!" do
+  describe "#locally!" do
+    it "returns a chaining object if no argumens are given" do
+      expect(subject.locally).to be_a(Yast::Execute)
+    end
+
     it "passes arguments directly to cheetah" do
       expect(Cheetah).to receive(:run).with("ls", "-a")
 
-      Yast::Execute.locally("ls", "-a")
+      subject.locally("ls", "-a")
     end
 
     it "raises Cheetah::ExecutionFailed if command execution failed" do
-      expect { Yast::Execute.locally!("false") }.to raise_error(Cheetah::ExecutionFailed)
+      expect { subject.locally!("false") }.to raise_error(Cheetah::ExecutionFailed)
     end
   end
 
-  describe ".on_target" do
+  describe "#on_target" do
+    it "returns a chaining object if no argumens are given" do
+      expect(subject.on_target).to be_a(Yast::Execute)
+    end
+
     it "adds to passed arguments chroot option if scr chrooted" do
       allow(Yast::WFM).to receive(:scr_root).and_return("/mnt")
       expect(Cheetah).to receive(:run).with("ls", "-a", chroot: "/mnt")
 
-      Yast::Execute.on_target("ls", "-a")
+      subject.on_target("ls", "-a")
     end
 
     it "report error if command execution failed" do
       expect(Yast::Report).to receive(:Error)
-      Yast::Execute.on_target("false")
+      subject.on_target("false")
     end
 
     it "returns nil if command execution failed" do
-      expect(Yast::Execute.on_target("false")).to eq nil
+      expect(subject.on_target("false")).to eq nil
     end
   end
 
-  describe ".on_target!" do
+  describe "#on_target!" do
+    it "returns a chaining object if no argumens are given" do
+      expect(subject.on_target!).to be_a(Yast::Execute)
+    end
+
     it "adds to passed arguments chroot option if scr chrooted" do
       allow(Yast::WFM).to receive(:scr_root).and_return("/mnt")
       expect(Cheetah).to receive(:run).with("ls", "-a", chroot: "/mnt")
 
-      Yast::Execute.on_target("ls", "-a")
+      subject.on_target("ls", "-a")
     end
 
     it "raises Cheetah::ExecutionFailed if command execution failed" do
-      expect { Yast::Execute.on_target!("false") }.to raise_error(Cheetah::ExecutionFailed)
+      expect { subject.on_target!("false") }.to raise_error(Cheetah::ExecutionFailed)
+    end
+  end
+
+  describe "#stdout" do
+    it "returns a chaining object if no argumens are given" do
+      expect(subject.stdout).to be_a(Yast::Execute)
+    end
+
+    it "captures stdout of the command" do
+      expect(Cheetah).to receive(:run).with("ls", "-a", stdout: :capture)
+
+      subject.stdout("ls", "-a")
+    end
+
+    it "returns an empty string if command execution failed" do
+      expect(subject.stdout("false")).to eq("")
+    end
+
+    context "when chaining with #locally" do
+      it "report error if command execution failed" do
+        expect(Yast::Report).to receive(:Error)
+        subject.locally.stdout("false")
+      end
+
+      it "returns nil if command execution failed" do
+        expect(subject.locally.stdout("false")).to be_nil
+      end
+    end
+
+    context "when chaining with #on_target" do
+      it "report error if command execution failed" do
+        expect(Yast::Report).to receive(:Error)
+        subject.on_target.stdout("false")
+      end
+
+      it "returns nil if command execution failed" do
+        expect(subject.on_target.stdout("false")).to be_nil
+      end
+    end
+
+    context "when chaining with #locally!" do
+      it "does not raise an exception if command execution failed" do
+        expect { subject.locally!.stdout("false") }.to_not raise_error
+      end
+
+      it "returns an empty string if command execution failed" do
+        expect(subject.locally!.stdout("false")).to eq("")
+      end
+    end
+
+    context "when chaining with #on_target!" do
+      it "does not raise an exception if command execution failed" do
+        expect { subject.on_target!.stdout("false") }.to_not raise_error
+      end
+
+      it "returns an empty string if command execution failed" do
+        expect(subject.on_target!.stdout("false")).to eq("")
+      end
     end
   end
 end