Updated test

nanograv · Mar 25, 2021 · d46f2e5 · d46f2e5
1 parent f35c2a3
commit d46f2e5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/enterprise/pulsar.py b/enterprise/pulsar.py
@@ -573,5 +573,5 @@ def Pulsar(*args, **kwargs):
             t2pulsar = t2.tempopulsar(relparfile, reltimfile, maxobs=maxobs, ephem=ephem, clk=clk)
             os.chdir(cwd)
             return Tempo2Pulsar(t2pulsar, sort=sort, drop_t2pulsar=drop_t2pulsar, planets=planets)
-    else:
-        logger.error("Unknown arguments {}".format(args))  # pragma: no cover
+
+    raise ValueError("Unknown arguments {}".format(args))
diff --git a/tests/test_pulsar.py b/tests/test_pulsar.py
@@ -142,6 +142,12 @@ def test_wrong_input(self):
             msg = "Cannot find parfile wrong.par or timfile wrong.tim!"
             self.assertTrue(msg in context.exception)
 
+    def test_value_error(self):
+        """Test exception when unknown argument is given"""
+
+        with self.assertRaises(ValueError):
+            Pulsar(datadir + "/B1855+09_NANOGrav_9yv1.gls.par", datadir + "/B1855+09_NANOGrav_9yv1.time")
+
 
 class TestPulsarPint(TestPulsar):
     @classmethod