Merge PR #235 into 17.0

Signed-off-by simahawk

Author: OCA-git-bot

server_environment/models/server_env_mixin.py

@@ -407,6 +407,7 @@ def _server_env_add_default_field(self, base_field):
                     "sparse": "server_env_defaults",
                     "automatic": True,
                     "string": fieldlabel,
+                    "default": base_field.default,
                 }
             )
 

server_environment/server_env.py

@@ -308,11 +308,11 @@ def get_view(self, view_id=None, view_type="form", **options):
     @api.model
     def _is_secret(self, key):
         """
-        This method is intended to be inherited to defined which keywords
-        should be secret.
-        :return: list of secret keywords
+        This method is intended to be inherited to define which values
+        should be kept secret.
+        :return: True if information has to be protected, False otherwise
         """
-        secret_keys = ["passw", "key", "secret", "token"]
+        secret_keys = ["_pass", "passw", "key", "secret", "token"]
         return any(secret_key in key for secret_key in secret_keys)
 
     @api.model

server_environment/tests/common.py

@@ -10,6 +10,12 @@
 import odoo.addons.server_environment.models.server_env_mixin as server_env_mixin
 from odoo.addons.server_environment import server_env
 
+CLEAN_ENV = {
+    var: value
+    for (var, value) in os.environ.items()
+    if var not in ("RUNNING_ENV", "ODOO_STAGE")
+}
+
 
 class ServerEnvironmentCase(common.TransactionCase):
     @contextmanager
@@ -24,23 +30,34 @@ def set_config_dir(self, path):
             server_env._dir = original_dir
 
     @contextmanager
-    def set_env_variables(self, public=None, secret=None):
-        newkeys = {}
+    def set_env_variables(self, public=None, secret=None, **env_vars):
+        newkeys = {**CLEAN_ENV, **env_vars}
         if public:
             newkeys["SERVER_ENV_CONFIG"] = public
         if secret:
             newkeys["SERVER_ENV_CONFIG_SECRET"] = secret
-        with patch.dict("os.environ", newkeys):
+        with patch.dict("os.environ", newkeys, clear=True):
             yield
 
     @contextmanager
-    def load_config(self, public=None, secret=None, serv_config_class=server_env_mixin):
+    def load_config(
+        self,
+        public=None,
+        secret=None,
+        config_dir=None,
+        serv_config_class=server_env_mixin,
+    ):
         original_serv_config = serv_config_class.serv_config
         try:
-            with self.set_config_dir(None), self.set_env_variables(public, secret):
+            with (
+                self.set_config_dir(config_dir),
+                self.set_env_variables(public, secret),
+            ):
                 parser = server_env._load_config()
                 serv_config_class.serv_config = parser
+                server_env.serv_config = parser
                 yield
 
         finally:
             serv_config_class.serv_config = original_serv_config
+            server_env.serv_config = original_serv_config

server_environment/tests/test_environment_variable.py

@@ -1,7 +1,6 @@
 # Copyright 2018 Camptocamp (https://www.camptocamp.com).
 # License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
 
-
 from unittest.mock import patch
 
 from odoo.tools.config import config as odoo_config
@@ -11,8 +10,11 @@
 
 
 class TestRunningEnvDefault(ServerEnvironmentCase):
+    @patch.dict(odoo_config.options, {"running_env": None})
     def test_running_env_default(self):
         """When var is not provided it defaults to `test`."""
+        with self.set_env_variables():
+            server_env._load_running_env()
         self.assertEqual(odoo_config["running_env"], "test")
 
 

server_environment/tests/test_server_environment.py

@@ -9,6 +9,17 @@
 from .. import server_env
 from . import common
 
+NO_DEFAULT = [
+    "id",
+    "create_uid",
+    "create_date",
+    "write_uid",
+    "write_date",
+    "display_name",
+    "config",
+    "__last_update",
+]
+
 
 class TestEnv(common.ServerEnvironmentCase):
     def test_view(self):
@@ -20,17 +31,20 @@ def _test_default(self, hidden_pwd=False):
         model = self.env["server.config"]
         rec = model.create({})
         fields = model.fields_get()
-        self.assertTrue(fields)
         defaults = rec.default_get(list(fields))
-        self.assertTrue(defaults)
+        fields_with_default = {fld for fld in fields if fld not in NO_DEFAULT}
+        self.assertTrue(fields_with_default)
         self.assertIsInstance(defaults, dict)
+        self.assertEqual(fields_with_default, set(defaults))
+        # Check secrets
         pass_checked = False
         for default in defaults:
-            if "passw" in default:
+            if "_pass" in default:
                 check = self.assertEqual if hidden_pwd else self.assertNotEqual
                 check(defaults[default], "**********")
                 pass_checked = True
         self.assertTrue(pass_checked)
+        return defaults
 
     @patch.dict(odoo_config.options, {"running_env": "dev"})
     def test_default_dev(self):
@@ -54,10 +68,23 @@ def test_odoosh_dev_from_environ(self):
         self._test_default()
 
     @patch.dict(odoo_config.options, {"running_env": "testing"})
-    def test_value_retrival(self):
+    def test_value_retrieval(self):
         with self.set_config_dir("testfiles"):
             parser = server_env._load_config()
             val = parser.get("external_service.ftp", "user")
             self.assertEqual(val, "testing")
             val = parser.get("external_service.ftp", "host")
             self.assertEqual(val, "sftp.example.com")
+
+    @patch.dict(odoo_config.options, {"running_env": "testing"})
+    def test_default_hidden_password(self):
+        with self.load_config(config_dir="testfiles"):
+            model = self.env["server.config"]
+            model._add_columns()
+            self.env.registry.setup_models(self.env.cr)
+        defaults = self._test_default(hidden_pwd=True)
+
+        self.assertIn("odoo_I_admin_passwd", defaults)
+        self.assertIn("odoo_I_db_password", defaults)
+        self.assertIn("odoo_I_smtp_password", defaults)
+        self.assertIn("outgoing_mail_provider_promail_I_smtp_pass", defaults)

server_environment/tests/testfiles/testing/outmail.conf

@@ -0,0 +1,6 @@
+[outgoing_mail.provider_promail]
+smtp_encryption	= ssl
+smtp_host = email.server.invalid
+smtp_pass = THISISNOTPUBLIC
+smtp_port = 912
+smtp_user = user_abc