Test: update comment

davedoesdev · Oct 28, 2021 · 21713b5 · 21713b5
1 parent 54f20e3
commit 21713b5
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/test/alg_none_verification_vows.py b/test/alg_none_verification_vows.py
@@ -42,7 +42,9 @@ def token_should_not_verify(self, r):
             expect([
                 'Verification failed for all signatures[\'Failed: [InvalidJWSSignature(\\\'Verification failed {InvalidSignature(\\\\\\\'The "none" signature cannot be verified\\\\\\\',)}\\\',)]\']',
                 'Verification failed for all signatures[\'Failed: [InvalidJWSSignature(\\\'Verification failed {InvalidSignature(\\\\\\\'The "none" signature cannot be verified\\\\\\\')}\\\')]\']',
-                'Verification failed for all signatures["Failed: [InvalidJWSSignature(\'Verification failed\')]"]', # python3 uses raise from and doesn't put reason in exception
+                # On Python3, jwcrypto uses raise from and doesn't put the reason in exception.
+                # It also uses repr() on the exception so loses the cause attribute.
+                'Verification failed for all signatures["Failed: [InvalidJWSSignature(\'Verification failed\')]"]',
                 'Verification failed for all signatures["Failed: [InvalidJWSSignature(\'Verification failed\',)]"]'
             ]).to_include(str(r))