Merge branch 'tickets/DM-42217'

lsst · Jan 31, 2024 · 4daa84c · 4daa84c
2 parents 1898adc + e41e238
commit 4daa84c
Showing 1 changed file with 21 additions and 0 deletions.
diff --git a/tests/test_RBTransiNetTask.py b/tests/test_RBTransiNetTask.py
@@ -104,6 +104,27 @@ def test_run(self):
         self.assertIsInstance(result.classifications, lsst.afw.table.BaseCatalog)
         np.testing.assert_array_equal(self.catalog["id"], result.classifications["id"])
 
+    def test_config_butlerblock(self):
+        config = RBTransiNetTask.ConfigClass()
+        config.modelPackageName = "dummy"
+        config.modelPackageStorageMode = "butler"
+        with self.assertRaises(ValueError):
+            config.validate()
+
+    def test_config_butlerpass_none(self):
+        config = RBTransiNetTask.ConfigClass()
+        config.modelPackageName = None
+        config.modelPackageStorageMode = "butler"
+        # Should not raise
+        config.validate()
+
+    def test_config_butlerblock_empty(self):
+        config = RBTransiNetTask.ConfigClass()
+        config.modelPackageName = ""  # Want *only* None for butler packages
+        config.modelPackageStorageMode = "butler"
+        with self.assertRaises(ValueError):
+            config.validate()
+
 
 class MemoryTester(lsst.utils.tests.MemoryTestCase):
     pass