about_with_statement.py has a boken test on line 98

[denis-roy]

The default placeholder value of __ can be left as is and the test will pass. I might be mistaken but it seems like an unexpected behavior to me..?

[kjc]

The Python 2 version of the same test explicitly asserts that the user-written function does not return None:

def find_line2(self, file_name):
    # Rewrite find_line using the Context Manager.
    pass

def test_finding_lines2(self):
    self.assertEqual(__, self.find_line2("example_file.txt"))
    self.assertNotEqual(None, self.find_line2("example_file.txt"))

I think the idea was that the assertNotEqual would test that the student-written function returns anything, while the assertEqual would test that, in addition, the return value is correct. That would be more clear if they appeared in the opposite order.

PS: This should really be an issue (used for pointing out problems), not a pull request (used for fixes or other improvements). I'll create one now.