Accept more error messages in test_tls_ext_noca

tiran · tiran · commit 8fb725d1a005 · 2017-12-04T12:50:53.000+01:00
OpenSSL 1.0, 1.1, and NSS return different error messages for untrusted certificate and missing CA. Closes: #87 Signed-off-by: Christian Heimes <cheimes@redhat.com>
diff --git a/Tests/t_cext.py b/Tests/t_cext.py
@@ -818,9 +818,15 @@ def test_tls_ext_noca(self):
         l.set_option(_ldap.OPT_PROTOCOL_VERSION, _ldap.VERSION3)
         with self.assertRaises(_ldap.CONNECT_ERROR) as e:
             l.start_tls_s()
-        # some platforms return '(unknown error code)' as reason
-        if '(unknown error code)' not in str(e.exception):
-            self.assertIn('not trusted', str(e.exception))
+        # known resaons:
+        # Ubuntu on Travis: '(unknown error code)'
+        # OpenSSL 1.1: error:1416F086:SSL routines:\
+        #    tls_process_server_certificate:certificate verify failed
+        # NSS: TLS error -8172:Peer's certificate issuer has \
+        #    been marked as not trusted by the user.
+        msg = str(e.exception)
+        if 'certificate' not in msg and msg != '(unknown error code)':
+            self.fail(msg)
 
     @requires_tls(skip_nss=True)
     def test_tls_ext_clientcert(self):
