Merge pull request #772 from alimcmaster1/test-cov-lat

Add test for invalid separator in isoparser
dateutil · Jun 20, 2018 · ef6cddd · ef6cddd
2 parents 6dde5d6 + fb77107
commit ef6cddd
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/changelog.d/772.misc.rst b/changelog.d/772.misc.rst
@@ -0,0 +1 @@
+Added tests that an error is raised for an incorrect separator character in a strict isoparser. Contribution by @alimcmaster1 (gh pr #772).
diff --git a/dateutil/test/test_isoparser.py b/dateutil/test/test_isoparser.py
@@ -264,12 +264,15 @@ def test_iso_raises(isostr, exception):
         isoparse(isostr)
 
 
-@pytest.mark.parametrize('sep_act,valid_sep', [
-    ('C', 'T'),
-    ('T', 'C')
+@pytest.mark.parametrize('sep_act, valid_sep, exception', [
+    ('T', 'C', ValueError),
+    ('C', 'T', ValueError),
 ])
-def test_iso_raises_sep(sep_act, valid_sep):
+def test_iso_with_sep_raises(sep_act, valid_sep, exception):
+    parser = isoparser(sep=valid_sep)
     isostr = '2012-04-25' + sep_act + '01:25:00'
+    with pytest.raises(exception):
+        parser.isoparse(isostr)
 
 
 @pytest.mark.xfail()