Fix exit code returned by client

Fixes a problem introduced with 5c9068a.
os-autoinst · Apr 25, 2020 · ae46054 · ae46054
1 parent b3f6ea4
commit ae46054
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/script/client b/script/client
@@ -181,6 +181,7 @@ if ($options{form} && $options{'json-data'}) {
     exit(2);
 }
 
-exit OpenQA::Client::run(\%options, @ARGV);
+my $ret = OpenQA::Client::run(\%options, @ARGV);
+exit(defined $ret ? 0 : 1);
 
 1;
diff --git a/t/31-client.t b/t/31-client.t
@@ -85,9 +85,9 @@ $code = 404;
 $code_mock->{error} = {message => 'Not Found'};
 sub wrong_call { $ret = OpenQA::Client::run(\%options, qw(unknown)) }
 stderr_like \&wrong_call, qr/$code.*Not Found/, 'Error reported';
-is $ret, 1, 'exit code shows error';
+is $ret, undef, 'undef shows error';
 $options{json} = 1;
 stderr_like \&wrong_call, qr/$code.*Not Found/, 'Error reported for undocumented "json" parameter';
-is $ret, 1, 'exit code shows error';
+is $ret, undef, 'undef shows error for undocumented parameter';
 
 done_testing();