Message352458
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. |
|
Date |
User |
Action |
Args |
2019-09-15 00:33:48 | ta1hia | set | recipients:
+ ta1hia, belopolsky, p-ganssle, corona10 |
2019-09-15 00:33:48 | ta1hia | set | messageid: <1568507628.74.0.0288101628649.issue38155@roundup.psfhosted.org> |
2019-09-15 00:33:48 | ta1hia | link | issue38155 messages |
2019-09-15 00:33:48 | ta1hia | create | |
|