Skip to content

Commit 4d92daa

Browse files
committed
leap: replace assertFalse with assertIs because the test passes if the tested method returns None, replace assertTrue with assertIs for consistency.
1 parent 575df84 commit 4d92daa

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

exercises/leap/leap_test.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,16 +7,16 @@
77

88
class YearTest(unittest.TestCase):
99
def test_year_not_divisible_by_4(self):
10-
self.assertFalse(is_leap_year(2015))
10+
self.assertIs(is_leap_year(2015), False)
1111

1212
def test_year_divisible_by_4_not_divisible_by_100(self):
13-
self.assertTrue(is_leap_year(2016))
13+
self.assertIs(is_leap_year(2016), True)
1414

1515
def test_year_divisible_by_100_not_divisible_by_400(self):
16-
self.assertFalse(is_leap_year(2100))
16+
self.assertIs(is_leap_year(2100), False)
1717

1818
def test_year_divisible_by_400(self):
19-
self.assertTrue(is_leap_year(2000))
19+
self.assertIs(is_leap_year(2000), True)
2020

2121

2222
if __name__ == '__main__':

0 commit comments

Comments
 (0)