commit 7e75b620f1c63fcc6168fe611eafa97fd709121b
author Paul Kehrer <paul.l.kehrer@gmail.com> Wed Dec 23 19:30:35 2015 -0600
committer Paul Kehrer <paul.l.kehrer@gmail.com> Wed Dec 23 19:30:35 2015 -0600
tree 2a7222914a933ab887815a4995ec50ccf89bb510
parent 1f943ab1a6ed391ef9474152e3f5ccb666cce4c9

address review comments

tests/test_x509.py

diff --git a/tests/test_x509.py b/tests/test_x509.py
index 034e560..ab4d666 100644
--- a/tests/test_x509.py
+++ b/tests/test_x509.py
@@ -6,7 +6,6 @@
 
 import binascii
 import datetime
-import gc
 import ipaddress
 import os
 
@@ -178,17 +177,13 @@
         """
         This test attempts to trigger the crash condition described in
         https://github.com/pyca/cryptography/issues/2557
+        PyPy does gc at its own pace, so it will only be reliable on CPython.
         """
-        crl = _load_cert(
+        revoked = _load_cert(
             os.path.join("x509", "custom", "crl_all_reasons.pem"),
             x509.load_pem_x509_crl,
             backend
-        )
-        revoked = crl[11]
-        crl = "overwritten"
-        # force a gc collection to potentially X509_CRL_free if there are
-        # no references to the X509_CRL left.
-        gc.collect()
+        )[11]
         assert revoked.revocation_date == datetime.datetime(2015, 1, 1, 0, 0)
         assert revoked.serial_number == 11