diff options
author | Roland Hedberg <roland.hedberg@adm.umu.se> | 2014-12-15 15:18:27 +0100 |
---|---|---|
committer | Roland Hedberg <roland.hedberg@adm.umu.se> | 2014-12-15 15:18:27 +0100 |
commit | 6f0ff8644fd0d4325cd2268a98b500b0c2614332 (patch) | |
tree | bc5dbdba988aa97e3c543714a4ee82efb9604e82 | |
parent | 6787ce4dd58a5ce41141e389d9dd08ded39e2c4f (diff) | |
download | pysaml2-6f0ff8644fd0d4325cd2268a98b500b0c2614332.tar.gz |
Better error message
-rw-r--r-- | src/saml2/sigver.py | 2 | ||||
-rw-r--r-- | tests/test_40_sigver.py | 7 |
2 files changed, 5 insertions, 4 deletions
diff --git a/src/saml2/sigver.py b/src/saml2/sigver.py index 0ac1f032..5b83df42 100644 --- a/src/saml2/sigver.py +++ b/src/saml2/sigver.py @@ -856,7 +856,7 @@ class CryptoBackendXmlSec1(CryptoBackend): logger.error( "Signing operation failed :\nstdout : %s\nstderr : %s" % ( stdout, stderr)) - raise SigverError("Signing failed") + raise SigverError(stderr) except DecryptError: raise SigverError("Signing failed") diff --git a/tests/test_40_sigver.py b/tests/test_40_sigver.py index c0d12813..0eefeed6 100644 --- a/tests/test_40_sigver.py +++ b/tests/test_40_sigver.py @@ -1,7 +1,8 @@ #!/usr/bin/env python import base64 -from saml2.sigver import pre_encryption_part, make_temp, XmlsecError +from saml2.sigver import pre_encryption_part, make_temp, XmlsecError, \ + SigverError from saml2.mdstore import MetadataStore from saml2.saml import assertion_from_string, EncryptedAssertion from saml2.samlp import response_from_string @@ -495,7 +496,7 @@ def test_xmlsec_err(): sec.sign_statement(assertion, class_name(assertion), key_file=full_path("tes.key"), node_id=assertion.id) - except XmlsecError as err: # should throw an exception + except (XmlsecError, SigverError) as err: # should throw an exception pass else: assert False @@ -506,4 +507,4 @@ if __name__ == "__main__": # t.setup_class() # t.test_non_verify_2() - test_xbox() + test_xmlsec_err() |