[ADD] datamodel: add test to check if datamodel env equals the env of the current request

Author: François Degrave

datamodel/tests/test_build_datamodel.py

@@ -5,6 +5,10 @@
 import mock
 from marshmallow_objects.models import Model as MarshmallowModel
 
+from odoo import SUPERUSER_ID, api
+
+from odoo.addons.website.tools import MockRequest
+
 from .. import fields
 from ..core import Datamodel
 from .common import DatamodelRegistryCase, TransactionDatamodelCase
@@ -40,7 +44,7 @@ class Datamodel2(Datamodel):
         self.assertIsInstance(Datamodel2(), MarshmallowModel)
 
     def test_no_name(self):
-        """ Ensure that a datamodel has a _name """
+        """Ensure that a datamodel has a _name"""
 
         class Datamodel1(Datamodel):
             pass
@@ -50,7 +54,7 @@ class Datamodel1(Datamodel):
             Datamodel1._build_datamodel(self.datamodel_registry)
 
     def test_register(self):
-        """ Able to register datamodels in datamodels registry """
+        """Able to register datamodels in datamodels registry"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -67,7 +71,7 @@ class Datamodel2(Datamodel):
         )
 
     def test_inherit_bases(self):
-        """ Check __bases__ of Datamodel with _inherit """
+        """Check __bases__ of Datamodel with _inherit"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -90,7 +94,7 @@ class Datamodel3(Datamodel):
         )
 
     def test_prototype_inherit_bases(self):
-        """ Check __bases__ of Datamodel with _inherit and different _name """
+        """Check __bases__ of Datamodel with _inherit and different _name"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -188,7 +192,7 @@ class Datamodel4(Datamodel):
         )
 
     def test_custom_build(self):
-        """ Check that we can hook at the end of a Datamodel build """
+        """Check that we can hook at the end of a Datamodel build"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -204,7 +208,7 @@ def _complete_datamodel_build(cls):
         self.assertTrue(self.env.datamodels["datamodel1"]._build_done)
 
     def test_inherit_attrs(self):
-        """ Check attributes inheritance of Datamodels with _inherit """
+        """Check attributes inheritance of Datamodels with _inherit"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -236,7 +240,7 @@ def say(self):
         self.assertEqual("foo bar", datamodel2.say())
 
     def test_duplicate_datamodel(self):
-        """ Check that we can't have 2 datamodels with the same name """
+        """Check that we can't have 2 datamodels with the same name"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -250,7 +254,7 @@ class Datamodel2(Datamodel):
             Datamodel2._build_datamodel(self.datamodel_registry)
 
     def test_no_parent(self):
-        """ Ensure we can't _inherit a non-existent datamodel """
+        """Ensure we can't _inherit a non-existent datamodel"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -261,7 +265,7 @@ class Datamodel1(Datamodel):
             Datamodel1._build_datamodel(self.datamodel_registry)
 
     def test_no_parent2(self):
-        """ Ensure we can't _inherit by prototype a non-existent datamodel """
+        """Ensure we can't _inherit by prototype a non-existent datamodel"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -276,7 +280,7 @@ class Datamodel2(Datamodel):
             Datamodel2._build_datamodel(self.datamodel_registry)
 
     def test_add_inheritance(self):
-        """ Ensure we can add a new inheritance """
+        """Ensure we can add a new inheritance"""
 
         class Datamodel1(Datamodel):
             _name = "datamodel1"
@@ -339,7 +343,7 @@ class Datamodel1(Datamodel):
             self.env.datamodels["datamodel1"](field_str="1234")
 
     def test_nested_model(self):
-        """ Test nested model serialization/deserialization"""
+        """Test nested model serialization/deserialization"""
 
         class Parent(Datamodel):
             _name = "parent"
@@ -366,7 +370,7 @@ class Child(Datamodel):
         self.assertEqual(new_instance.child.field_str, instance.child.field_str)
 
     def test_list_nested_model(self):
-        """ Test list model of nested model serialization/deserialization"""
+        """Test list model of nested model serialization/deserialization"""
 
         class Parent(Datamodel):
             _name = "parent"
@@ -469,6 +473,10 @@ class Item(Datamodel):
         self.assertEqual(instance.items[0].env, self.env)
         schema = instance.items[0].get_schema()
         self.assertEqual(schema._env, self.env)
+        another_env = api.Environment(self.env.registry.cursor(), SUPERUSER_ID, {})
+        with MockRequest(another_env):
+            self.assertNotEqual(p.env, self.env)
+            self.assertEqual(p.env, another_env)
 
 
 class TestRegistryAccess(TransactionDatamodelCase):