-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reach 100% coverage #80
Conversation
…rage of tests.attrhooks, tests.cucumbers and tests.utils.
Two bug fixes for pure-Python mode: deleting _p_oid and deleting *any* _p attribute.
Deleted _makeOctets since we have native literals everywhere.
Two of the tests for _p_serial weren't catching what we thought they were.
Remove the old workaround for a broken __cmp__.
There are many fewer 'no cover' lines now. Remove the call to copy_reg.constructor---it doesn't do anything anymore except check that the object is callable, there is no registry. Also make timestamp support PURE_PYTHON.
Do this everywhere for simplicity. It slows PyPy down by 3x, but that should still only be around 4s.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
(The diff is rather on the large side, so I haven't scrutinized it very attentively. I place my trust in the test suite.)
persistent/tests/test_persistence.py
Outdated
def mru(oid): | ||
# Mimic what the real cache does | ||
if oid not in jar._cache._mru: | ||
raise KeyError(oid) | ||
orig_mru(oid) | ||
raise AssertionError("Shold never get here") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shold -> Should
Thanks. For the most part it's very small changes, just quite a few of them. The most potentially disruptive change is in init.py, which I simplified substantially, including taking out an attempted import of I'll add this to CHANGES. |
[skip ci] Update PyPI links in readme. Fix typo in test_persistence.py
In tox only for now, coveralls coming in a followup PR or commit.
There are also two minor pure-python bug fixes I discovered.
Addresses #76.