server_environment: running_env default to test (fix #44)

server_environment/server_env.py

@@ -58,14 +58,16 @@
  "off": False,
 }
 
-if not system_base_config.get("running_env", False):
- raise Exception(
-  "The parameter 'running_env' has not be set neither in base config "
-  "file option -c or in openerprc.\n"
+
+if not system_base_config.get("running_env"):
+ _logger.warning("`running_env` not found. Using default = `test`.")
+ _logger.warning(
  "We strongly recommend against using the rc file but instead use an "
- "explicit config file with this content:\n"
- "[options]\nrunning_env = dev"
+ "explicit config file or env variable."
  )
+ # safe default
+ system_base_config["running_env"] = "test"
+
 
 ck_path = None
 if _dir:

server_environment/tests/common.py

@@ -13,10 +13,14 @@
 
 
 class ServerEnvironmentCase(common.SavepointCase):
+
+ _test_case_running_env = "testing"
+
  def setUp(self):
  super().setUp()
  self._original_running_env = config.get("running_env")
- config["running_env"] = "testing"
+ if self._test_case_running_env:
+ config["running_env"] = self._test_case_running_env
 
  def tearDown(self):
  super().tearDown()

server_environment/tests/test_environment_variable.py

@@ -2,6 +2,8 @@
 # License GPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 
+from odoo.tools.config import config
+
 from odoo.addons.server_environment import server_env
 
 from .common import ServerEnvironmentCase
@@ -25,3 +27,11 @@ def test_env_variables_override(self):
  parser = server_env._load_config()
  val = parser.get("external_service.ftp", "user")
  self.assertEqual(val, "foo")
+
+
+class TestRunningEnv(ServerEnvironmentCase):
+
+ _test_case_running_env = ""
+
+ def test_running_env_default(self):
+ self.assertEqual(config["running_env"], "test")