We test in 3.4. So either this code is ignored in that test or it works pre 3.5

Does the testing raise an exception? Is the exception raised a NameError or a RecursionError or a RuntimeError?

The failure is from master branch, not this PR.

It looks like the exception is used at the bottom to get a nicer print when a particular failure happened so in the normal case we never exercise the 'except' branch. This is fine, but might be better to remove the try block where the exception is used

Thanks for finding and fixing all of these issues @cclauss !

@jklymak Yep, if it never raises, then the wrong name isn't a problem:

>>> try:
...     1
... except NotARealExceptionError:
...     pass
... 
1

vs.

>>> try:
...     a
... except NotARealExceptionError:
...     pass
... 
Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
NameError: name 'a' is not defined

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 3, in <module>
NameError: name 'NotARealExceptionError' is not defined