openstack/deb-python-pysaml2
Code Issues Proposed changes

Remove tests asserting unnecessary py2 rounding

Browse Source 
f_quotient in time_utils does not actually ever get passed values in the
ranges that are being tested here. It is, however, asserting bad python2
rounding behavior in the int() constructor that has been removed. We can
just remove these tests, and be comfortable reasoning that there aren't
time differences that cause this situation based on the docs of
time.struct_time.
This commit is contained in:
Clint Byrum
2015-05-26 15:14:02 -07:00
parent 23a388788a
commit 0e68174a9a
1 changed files with 0 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/test_10_time_util.py
View File

@@ -9,7 +9,6 @@ from saml2.time_util import before, after, not_before, not_on_or_after
def test_f_quotient():
assert f_quotient(-1, 3) == -1
assert f_quotient(0, 3) == 0
assert f_quotient(1, 3) == 0
assert f_quotient(2, 3) == 0
@@ -28,7 +27,6 @@ def test_modulo():
def test_f_quotient_2():
assert f_quotient(0, 1, 13) == -1
for i in range(1, 13):
assert f_quotient(i, 1, 13) == 0
assert f_quotient(13, 1, 13) == 1