Berker Peksag added the comment:

LGTM.

A minor comment:

+    def test_mock_open_reuse_issue_21750(self):

We can also add an assert to check the data is actually "data" (e.g. 
assertEqual(f1.read(), 'data')).

----------
stage: patch review -> commit review

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue21750>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to