Ok the commented code is necessary  because it checks that a RuntimeError is not raised. While if we remove the chained context managers we check that the code does not raise any errors. I usually prefer to be explicit even for exception testing but, I do not feel very strong about keeping the test.