bsc#1036440: Warning messages shouldn't open UI in command-line mode

library/general/src/modules/Report.rb

@@ -45,6 +45,7 @@ def main
       Yast.import "Mode"
       Yast.import "Popup"
       Yast.import "Summary"
+      Yast.import "CommandLine"
 
       # stored messages
       @errors = []
@@ -481,7 +482,9 @@ def Warning(warning_string)
       Builtins.y2warning(1, "%1", warning_string) if @log_warnings
 
       if @display_warnings
-        if Ops.greater_than(@timeout_warnings, 0)
+        if Mode.commandline
+          CommandLine.Print "Warning: #{warning_string}"
+        elsif Ops.greater_than(@timeout_warnings, 0)
           Popup.TimedWarning(warning_string, @timeout_warnings)
         else
           Popup.Warning(warning_string)
@@ -522,8 +525,7 @@ def Error(error_string)
 
       if @display_errors
         if Mode.commandline
-          Yast.import "CommandLine"
-          CommandLine.Print error_string
+          CommandLine.Print "Error: #{error_string}"
         elsif Ops.greater_than(@timeout_errors, 0)
           Popup.TimedError(error_string, @timeout_errors)
         else

library/general/test/report_test.rb

@@ -0,0 +1,109 @@
+#! /usr/bin/env rspec
+#
+require_relative "test_helper"
+require "yaml"
+
+Yast.import "Report"
+Yast.import "Mode"
+
+describe Yast::Report do
+  before { subject.ClearAll }
+
+  describe ".Warning" do
+    let(:show) { true }
+    let(:message) { "Message" }
+
+    before do
+      allow(Yast::Mode).to receive(:commandline).and_return(commandline?)
+    end
+
+    context "while in command-line mode" do
+      let(:commandline?) { true }
+
+      it "prints the message only on console" do
+        expect(Yast::CommandLine).to receive(:Print)
+          .with(/#{message}/)
+        expect(Yast::Popup).to_not receive(:Warning)
+        expect(Yast::Popup).to_not receive(:TimedWarning)
+        subject.Warning(message)
+      end
+    end
+
+    context "while in UI mode and timeout is disabled" do
+      let(:timeout) { 0 }
+      let(:commandline?) { false }
+
+      before(:each) do
+        subject.DisplayWarnings(show, timeout)
+      end
+
+      it "shows a popup" do
+        expect(Yast::Popup).to receive(:Warning).with(/#{message}/)
+        subject.Warning(message)
+      end
+    end
+
+    context "while in UI mode and timeout is enabled" do
+      let(:timeout) { 1 }
+      let(:commandline?) { false }
+
+      before(:each) do
+        subject.DisplayWarnings(show, timeout)
+      end
+
+      it "shows timed popup" do
+        expect(Yast::Popup).to receive(:TimedWarning).with(/#{message}/, timeout)
+        subject.Warning(message)
+      end
+    end
+  end
+
+  describe ".Error" do
+    let(:show) { true }
+    let(:message) { "Message" }
+
+    before do
+      allow(Yast::Mode).to receive(:commandline).and_return(commandline?)
+    end
+
+    context "while in command-line mode" do
+      let(:commandline?) { true }
+
+      it "prints the message only on console" do
+        expect(Yast::CommandLine).to receive(:Print)
+          .with(/#{message}/)
+        expect(Yast::Popup).to_not receive(:Error)
+        expect(Yast::Popup).to_not receive(:TimedError)
+        subject.Error(message)
+      end
+    end
+
+    context "while in UI mode and timeout is disabled" do
+      let(:timeout) { 0 }
+      let(:commandline?) { false }
+
+      before(:each) do
+        subject.DisplayErrors(show, timeout)
+      end
+
+      it "shows a popup" do
+        expect(Yast::Popup).to receive(:Error).with(/#{message}/)
+        subject.Error(message)
+      end
+    end
+
+    context "while in UI mode and timeout is enabled" do
+      let(:timeout) { 1 }
+      let(:commandline?) { false }
+
+      before(:each) do
+        subject.DisplayErrors(show, timeout)
+      end
+
+      it "shows a timed popup" do
+        expect(Yast::Popup).to receive(:TimedError).with(/#{message}/, timeout)
+        subject.Error(message)
+      end
+    end
+  end
+end

package/yast2.changes

@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Mon May 29 08:19:46 UTC 2017 - gsouza@suse.com
+
+- Warning messages shouldn't open UI in command-line (bsc#1036440)
+  mode.
+- 3.1.155.8
+
 -------------------------------------------------------------------
 Tue May  9 11:44:54 UTC 2017 - mvidner@suse.com
 

package/yast2.spec

@@ -17,7 +17,7 @@
 
 
 Name:           yast2
-Version:        3.1.155.7
+Version:        3.1.155.8
 Release:        0
 Url:            https://github.com/yast/yast-yast2