[freeipa PR#5541][opened] ipa cert-remove-hold <invalid_cert_id> returns an incorrect error message

[flo-renaud]

   URL: https://github.com/freeipa/freeipa/pull/5541
Author: flo-renaud
 Title: #5541:  ipa cert-remove-hold <invalid_cert_id> returns an incorrect error
message
Action: opened

PR body:
"""
### cert plugin: propagate the error for non-existent cert

ipa cert-show, ipa cert-revoke and ipa cert-remove-hold do not
print meaningful info when called on a non-existent cert id:
Certificate operation cannot be completed: Unable to communicate
with CMS

Propagate the reason from the HTTP message in order to print
'Certificate ID 0x.. not found'

Fixes: https://pagure.io/freeipa/issue/8704

### xmlrpc tests: add a test for cert-remove-hold
Add tests for the ipa cert-remove-hold command.
Scenario 1:
add host entry, request cert, revoke cert with "hold" reason, remove hold

Scenario 2:
call ipa cert-move-hold with a non-existent cert ID and ensure that
the exception mentions 'Certificate ID .. not found'

Related: https://pagure.io/freeipa/issue/8704
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/5541/head:pr5541
git checkout pr5541

[freeipa PR#5541][closed] ipa cert-remove-hold <invalid_cert_id> returns an incorrect error message

[flo-renaud]

   URL: https://github.com/freeipa/freeipa/pull/5541
Author: flo-renaud
 Title: #5541:  ipa cert-remove-hold <invalid_cert_id> returns an incorrect error
message
Action: closed

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/5541/head:pr5541
git checkout pr5541