Skip to content

Commit

Permalink
Remove presumably accidental double-call.
Browse files Browse the repository at this point in the history
Looking more closely, I don't believe the double call was intended. The test still passes because the inner call raises the ValueError all the way up, but that means the outer call is never actually used, and it's a bit weird to be testing that passing a datetime.datetime doesn't work (and it would raise a TypeError anyway).
  • Loading branch information
tonyandrewmeyer committed Sep 22, 2023
1 parent 8e1ba55 commit dbf9805
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions test/test_private.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,9 +90,7 @@ def test_parse_rfc3339(self):
timeconv.parse_rfc3339('2021-99-99T04:36:22Z')

with self.assertRaises(ValueError):
timeconv.parse_rfc3339(
timeconv.parse_rfc3339('2021-02-10T04:36:22.118970777x')) # type: ignore
timeconv.parse_rfc3339('2021-02-10T04:36:22.118970777x')

with self.assertRaises(ValueError):
timeconv.parse_rfc3339(
timeconv.parse_rfc3339('2021-02-10T04:36:22.118970777-99:99')) # type: ignore
timeconv.parse_rfc3339('2021-02-10T04:36:22.118970777-99:99')

0 comments on commit dbf9805

Please sign in to comment.