-
Notifications
You must be signed in to change notification settings - Fork 78
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
Use pytest/bare asserts in tests #49
Open
s-t-e-v-e-n-k
wants to merge
1
commit into
mailgun:master
Choose a base branch
from
s-t-e-v-e-n-k:use-pytest
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -1,127 +1,130 @@ | ||||||||||||||||||||||||||||
from time import sleep | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
from mock import Mock, patch | ||||||||||||||||||||||||||||
from nose.tools import assert_raises, eq_, ok_ | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
from expiringdict import ExpiringDict | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
from pytest import raises | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_create(): | ||||||||||||||||||||||||||||
assert_raises(AssertionError, ExpiringDict, max_len=1, max_age_seconds=-1) | ||||||||||||||||||||||||||||
assert_raises(AssertionError, ExpiringDict, max_len=0, max_age_seconds=1) | ||||||||||||||||||||||||||||
with raises(AssertionError): | ||||||||||||||||||||||||||||
ExpiringDict(max_len=1, max_age_seconds=-1) | ||||||||||||||||||||||||||||
ExpiringDict(max_len=0, max_age_seconds=1) | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
d = ExpiringDict(max_len=3, max_age_seconds=0.01) | ||||||||||||||||||||||||||||
eq_(len(d), 0) | ||||||||||||||||||||||||||||
assert len(d) == 0 | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_basics(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=3, max_age_seconds=0.01) | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
eq_(d.get('a'), None) | ||||||||||||||||||||||||||||
assert d.get('a') is None | ||||||||||||||||||||||||||||
d['a'] = 'x' | ||||||||||||||||||||||||||||
eq_(d.get('a'), 'x') | ||||||||||||||||||||||||||||
assert d.get('a') == 'x' | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
sleep(0.01) | ||||||||||||||||||||||||||||
eq_(d.get('a'), None) | ||||||||||||||||||||||||||||
assert d.get('a') is None | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
d['a'] = 'y' | ||||||||||||||||||||||||||||
eq_(d.get('a'), 'y') | ||||||||||||||||||||||||||||
assert d.get('a') == 'y' | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
ok_('b' not in d) | ||||||||||||||||||||||||||||
assert 'b' not in d | ||||||||||||||||||||||||||||
d['b'] = 'y' | ||||||||||||||||||||||||||||
ok_('b' in d) | ||||||||||||||||||||||||||||
assert 'b' in d | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
sleep(0.01) | ||||||||||||||||||||||||||||
ok_('b' not in d) | ||||||||||||||||||||||||||||
assert 'b' not in d | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
# a is still in expiringdict, next values should expire it | ||||||||||||||||||||||||||||
d['c'] = 'x' | ||||||||||||||||||||||||||||
d['d'] = 'y' | ||||||||||||||||||||||||||||
d['e'] = 'z' | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
# dict if full | ||||||||||||||||||||||||||||
ok_('c' in d) | ||||||||||||||||||||||||||||
ok_('d' in d) | ||||||||||||||||||||||||||||
assert 'c' in d | ||||||||||||||||||||||||||||
assert 'd' in d | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
d['f'] = '1' | ||||||||||||||||||||||||||||
# c should gone after that | ||||||||||||||||||||||||||||
ok_('c' not in d, 'Len of dict is more than max_len') | ||||||||||||||||||||||||||||
assert 'c' not in d, 'Len of dict is more than max_len' | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
# test __delitem__ | ||||||||||||||||||||||||||||
del d['e'] | ||||||||||||||||||||||||||||
ok_('e' not in d) | ||||||||||||||||||||||||||||
assert 'e' not in d | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_pop(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=3, max_age_seconds=0.01) | ||||||||||||||||||||||||||||
d['a'] = 'x' | ||||||||||||||||||||||||||||
eq_('x', d.pop('a')) | ||||||||||||||||||||||||||||
assert 'x' == d.pop('a') | ||||||||||||||||||||||||||||
sleep(0.01) | ||||||||||||||||||||||||||||
eq_(None, d.pop('a')) | ||||||||||||||||||||||||||||
assert d.pop('a') is None | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_repr(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=2, max_age_seconds=0.01) | ||||||||||||||||||||||||||||
d['a'] = 'x' | ||||||||||||||||||||||||||||
eq_(str(d), "ExpiringDict([('a', 'x')])") | ||||||||||||||||||||||||||||
assert str(d) == "ExpiringDict([('a', 'x')])" | ||||||||||||||||||||||||||||
sleep(0.01) | ||||||||||||||||||||||||||||
eq_(str(d), "ExpiringDict([])") | ||||||||||||||||||||||||||||
assert str(d) == "ExpiringDict([])" | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_iter(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=10, max_age_seconds=0.01) | ||||||||||||||||||||||||||||
eq_([k for k in d], []) | ||||||||||||||||||||||||||||
assert [k for k in d] == [] | ||||||||||||||||||||||||||||
d['a'] = 'x' | ||||||||||||||||||||||||||||
d['b'] = 'y' | ||||||||||||||||||||||||||||
d['c'] = 'z' | ||||||||||||||||||||||||||||
eq_([k for k in d], ['a', 'b', 'c']) | ||||||||||||||||||||||||||||
assert [k for k in d] == ['a', 'b', 'c'] | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
eq_([k for k in d.values()], ['x', 'y', 'z']) | ||||||||||||||||||||||||||||
assert [k for k in d.values()] == ['x', 'y', 'z'] | ||||||||||||||||||||||||||||
sleep(0.01) | ||||||||||||||||||||||||||||
eq_([k for k in d.values()], []) | ||||||||||||||||||||||||||||
assert [k for k in d.values()] == [] | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_clear(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=10, max_age_seconds=10) | ||||||||||||||||||||||||||||
d['a'] = 'x' | ||||||||||||||||||||||||||||
eq_(len(d), 1) | ||||||||||||||||||||||||||||
assert len(d) == 1 | ||||||||||||||||||||||||||||
d.clear() | ||||||||||||||||||||||||||||
eq_(len(d), 0) | ||||||||||||||||||||||||||||
assert len(d) == 0 | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_ttl(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=10, max_age_seconds=10) | ||||||||||||||||||||||||||||
d['a'] = 'x' | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
# existent non-expired key | ||||||||||||||||||||||||||||
ok_(0 < d.ttl('a') < 10) | ||||||||||||||||||||||||||||
assert 0 < d.ttl('a') < 10 | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
# non-existent key | ||||||||||||||||||||||||||||
eq_(None, d.ttl('b')) | ||||||||||||||||||||||||||||
assert d.ttl('b') is None | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
# expired key | ||||||||||||||||||||||||||||
with patch.object(ExpiringDict, '__getitem__', | ||||||||||||||||||||||||||||
Mock(return_value=('x', 10**9))): | ||||||||||||||||||||||||||||
eq_(None, d.ttl('a')) | ||||||||||||||||||||||||||||
assert d.ttl('a') is None | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_setdefault(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=10, max_age_seconds=0.01) | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
eq_('x', d.setdefault('a', 'x')) | ||||||||||||||||||||||||||||
eq_('x', d.setdefault('a', 'y')) | ||||||||||||||||||||||||||||
assert d.setdefault('a', 'x') == 'x' | ||||||||||||||||||||||||||||
assert d.setdefault('a', 'y') == 'x' | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
sleep(0.01) | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
eq_('y', d.setdefault('a', 'y')) | ||||||||||||||||||||||||||||
assert d.setdefault('a', 'y') == 'y' | ||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_not_implemented(): | ||||||||||||||||||||||||||||
d = ExpiringDict(max_len=10, max_age_seconds=10) | ||||||||||||||||||||||||||||
assert_raises(NotImplementedError, d.fromkeys) | ||||||||||||||||||||||||||||
assert_raises(NotImplementedError, d.viewitems) | ||||||||||||||||||||||||||||
assert_raises(NotImplementedError, d.viewkeys) | ||||||||||||||||||||||||||||
assert_raises(NotImplementedError, d.viewvalues) | ||||||||||||||||||||||||||||
with raises(NotImplementedError): | ||||||||||||||||||||||||||||
d.fromkeys() | ||||||||||||||||||||||||||||
d.viewitems() | ||||||||||||||||||||||||||||
d.viewkeys() | ||||||||||||||||||||||||||||
d.viewvalues() | ||||||||||||||||||||||||||||
Comment on lines
+123
to
+127
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||
def test_reset_of_key_no_trim(): | ||||||||||||||||||||||||||||
|
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
The format changes starting with Python 3.12:
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.
The empty representation has changed too:
But stepping back, the module itself isn’t compatible with Python 3.12 so I don’t think it’s appropriate to update the tests for that yet.