use better argument type

library/general/src/lib/installation/auto_client.rb

@@ -46,7 +46,7 @@ def run
       when "SetModified"
         modified
       else
-        raise "Invalid action for auto client '#{func.inspect}'"
+        raise ArgumentError, "Invalid action for auto client '#{func.inspect}'"
       end
     end
 

library/general/src/lib/installation/finish_client.rb

@@ -55,7 +55,7 @@ def run
       when "Write"
         write
       else
-        raise "Invalid action for proposal '#{func.inspect}'"
+        raise ArgumentError, "Invalid action for proposal '#{func.inspect}'"
       end
     end
 

library/general/src/lib/installation/proposal_client.rb

@@ -64,7 +64,7 @@ def run
       when "Write"
         write(param)
       else
-        raise "Invalid action for proposal '#{func.inspect}'"
+        raise ArgumentError, "Invalid action for proposal '#{func.inspect}'"
       end
     end
 

library/general/test/auto_client_test.rb

@@ -17,9 +17,9 @@ def import(args)
 describe ::Installation::AutoClient do
   subject { ::TestAuto }
   describe ".run" do
-    it "raise exception if unknown first argument is passed" do
+    it "raise ArgumentError exception if unknown first argument is passed" do
       allow(Yast::WFM).to receive(:Args).and_return(["Unknown", {}])
-      expect{::Installation::AutoClient.run}.to raise_error
+      expect{::Installation::AutoClient.run}.to raise_error(ArgumentError)
     end
 
     context "first client argument is Import" do

library/general/test/finish_client_test.rb

@@ -17,9 +17,9 @@ def write
 describe ::Installation::FinishClient do
   subject { ::TestFinish }
   describe ".run" do
-    it "raise exception if unknown first argument is passed" do
+    it "raise ArgumentError exception if unknown first argument is passed" do
       allow(Yast::WFM).to receive(:Args).and_return(["Unknown", {}])
-      expect{::Installation::ProposalClient.run}.to raise_error
+      expect{::Installation::FinishClient.run}.to raise_error(ArgumentError)
     end
 
     context "first client argument is Info" do

library/general/test/proposal_client_test.rb

@@ -25,9 +25,9 @@ def write(args)
 describe ::Installation::ProposalClient do
   subject { ::TestProposal }
   describe ".run" do
-    it "raise exception if unknown first argument is passed" do
+    it "raise ArgumentError exception if unknown first argument is passed" do
       allow(Yast::WFM).to receive(:Args).and_return(["Unknown", {}])
-      expect{::Installation::ProposalClient.run}.to raise_error
+      expect{::Installation::ProposalClient.run}.to raise_error(ArgumentError)
     end
 
     context "first client argument is MakeProposal" do