Clean up config when database is disabled

pyiron_base/state/settings.py

@@ -281,6 +281,21 @@ def _validate_viewer_configuration(config: Dict) -> None:
                 #       I don't actually understand the constraint, I am just making it *explicit* as I refactor. -Liam
                 raise ValueError("Got sql_view arguments, but sql_type is not Postgres")
 
+    @staticmethod
+    def _validate_no_database_configuration(config: Dict) -> None:
+        if "disable_database" in config.keys() and config["disable_database"]:
+            if config["project_check_enabled"]:
+                raise ValueError(
+                    "When the database is disabled 'disable_database=True' the project " +
+                    "check cannot be enabled, so you have to set 'project_check_enabled=False'."
+                )
+            if len(config["project_paths"]) > 0:
+                raise ValueError(
+                    "When the database is disabled 'disable_database=True' the project " +
+                    "paths list should be empty 'project_paths=[]'. Currently it is: " +
+                    str(config["project_paths"])
+                )
+
     def _get_config_from_environment(self) -> Union[Dict, None]:
         config = {}
         for k, v in os.environ.items():
@@ -319,6 +334,7 @@ def _update_from_dict(self, config: Dict, map_: Union[None, Dict] = None) -> Non
         """
         self._validate_sql_configuration(config=config)
         self._validate_viewer_configuration(config=config)
+        self._validate_no_database_configuration(config=config)
 
         for key, value in config.items():
             key = key if map_ is None else map_[key]

tests/state/test_settings.py

@@ -129,6 +129,20 @@ def test_validate_viewer_configuration_completeness(self):
         with self.assertRaises(ValueError):
             s._validate_sql_configuration({"sql_type": "not_a_valid_type"})
 
+    def test_validate_no_database_configuration(self):
+        with self.assertRaises(ValueError):
+            s._validate_no_database_configuration({
+                "disable_database": True,
+                "project_check_enabled": True,
+                "project_paths": [],
+            })
+        with self.assertRaises(ValueError):
+            s._validate_no_database_configuration({
+                "disable_database": True,
+                "project_check_enabled": False,
+                "project_paths": ["/my/path/a"],
+            })
+
     def test_get_config_from_environment(self):
         os.environ["PYIRONFOO"] = "foo"
         os.environ["PYIRONSQLFILE"] = "bar"