Tahia K <tahia.k...@gmail.com> added the comment:

I'll definitely add that test @Paul. 

Speaking of, what do you guys think of this test:

     def test_c_all(self):
         """Test that __all__ symbols between the c datetime module and
         the python datetime library are equivalent."""

         c_datetime = import_fresh_module('datetime', fresh=['_datetime'])
         py_datetime = import_fresh_module('datetime', blocked=['_datetime'])
         self.assertEqual(c_datetime.__all__, py_datetime.__all__)


I found the import_fresh_module here: 
https://docs.python.org/3/library/test.html#test.support.import_fresh_module - 
super handy! The test currently passes for me locally, but I wanted to check if 
my usage was correct before submitting a PR.

----------

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

Reply via email to