do report an exception when running a CLI; test it

WIP, found in bsc#935937
yast · Jun 30, 2015 · 49184d2 · 49184d2
1 parent dbceace
commit 49184d2
Showing 1 changed file with 17 additions and 3 deletions.
diff --git a/library/commandline/src/modules/CommandLine.rb b/library/commandline/src/modules/CommandLine.rb
@@ -84,6 +84,9 @@ def main
             "help" => _(
               "Abort interactive mode without saving the changes"
             )
+          },
+          "please-crash" => {
+            "help" => "Simulates an internal error"
           }
         },
         "options"  => {
@@ -915,6 +918,10 @@ def ProcessSystemCommands(command)
         return true
       end
 
+      if Ops.get_string(command, "command", "") == "please-crash"
+        raise "Crashing as requested"
+      end
+
       if Ops.get_string(command, "command", "") == "help"
         # don't print header when custom help is defined
         PrintHead() if !Builtins.haskey(@modulecommands, "customhelp")
@@ -1599,9 +1606,9 @@ def Run(commandline)
         # disable Reports, we handle them on our own
         Report.Import(
 
-            "messages" => { "show" => false },
-            "warnings" => { "show" => false },
-            "errors"   => { "show" => false }
+          "messages" => { "show" => false },
+          "warnings" => { "show" => false },
+          "errors"   => { "show" => false }
 
         )
 
@@ -1694,6 +1701,13 @@ def Run(commandline)
       Builtins.y2milestone("----------------------------------------")
 
       ret
+    ensure
+      Builtins.y2milestone("Re-enabling reports")
+      Report.Import(
+        "messages" => { "show" => true },
+        "warnings" => { "show" => true },
+        "errors"   => { "show" => true }
+      )
     end
 
     # Ask user, commandline equivalent of Popup::YesNo()