Skip to content

Commit 0e68174

Browse files
committed
Remove tests asserting unnecessary py2 rounding
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.
1 parent 23a3887 commit 0e68174

File tree

1 file changed

+0
-2
lines changed

1 file changed

+0
-2
lines changed

tests/test_10_time_util.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99

1010

1111
def test_f_quotient():
12-
assert f_quotient(-1, 3) == -1
1312
assert f_quotient(0, 3) == 0
1413
assert f_quotient(1, 3) == 0
1514
assert f_quotient(2, 3) == 0
@@ -28,7 +27,6 @@ def test_modulo():
2827

2928

3029
def test_f_quotient_2():
31-
assert f_quotient(0, 1, 13) == -1
3230
for i in range(1, 13):
3331
assert f_quotient(i, 1, 13) == 0
3432
assert f_quotient(13, 1, 13) == 1

0 commit comments

Comments
 (0)