Merge pull request #938 from lsst/tickets/DM-42416

DM-42416: Modify without_datastore test to use file config
lsst · Jan 9, 2024 · d8d7886 · d8d7886
2 parents 307ac87 + 550f2e0
commit d8d7886
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/tests/test_butler.py b/tests/test_butler.py
@@ -2397,13 +2397,18 @@ def tearDown(self) -> None:
 
     def test_fallback(self) -> None:
         # Read the butler config and mess with the datastore section.
-        bad_config = Config(os.path.join(self.root, "butler.yaml"))
+        config_path = os.path.join(self.root, "butler.yaml")
+        bad_config = Config(config_path)
         bad_config["datastore", "cls"] = "lsst.not.a.datastore.Datastore"
+        bad_config.dumpToUri(config_path)
 
         with self.assertRaises(RuntimeError):
-            Butler.from_config(bad_config)
+            Butler(self.root, without_datastore=False)
 
-        butler = Butler.from_config(bad_config, writeable=True, without_datastore=True)
+        with self.assertRaises(RuntimeError):
+            Butler.from_config(self.root, without_datastore=False)
+
+        butler = Butler.from_config(self.root, writeable=True, without_datastore=True)
         self.assertIsInstance(butler._datastore, NullDatastore)
 
         # Check that registry is working.