Removed piping results to shasum to avoid test passing incorrectly

[techraf]

I understand that this was supposed to be some sort of optimisation, but because h1.exit_status and h2.exit_status will always inherit the exit code of the latter openssl sha -sha512, the test would pass if both: the certificate and the key were invalid or the files were non-existent.

For example the following check always passed:

describe x509_private_key('/some/non-existent/key/file') do
    it { should have_matching_certificate('/etc/passwd') }
end

Removing the pipe solves the problem, but I guess the correct way for such cases should be decided for the whole project. For example to make run_command to behave like bash with 'set -o pipefail'.

[mizzy]

Thanks!

Removing the pipe solves the problem, but I guess the correct way for such cases should be decided for the whole project. For example to make run_command to behave like bash with 'set -o pipefail'.

Thanks for your advice. I'll check it.

[mizzy]

Released as v2.29.1.