[IMP] event: improve test class and data to help reusing them

PURPOSE

Allow the user to schedule SMS communication on events

SPECIFICATIONS

In order to add tests this commit performs a quick cleaning of event common
class to speedup and prepare new tests.

Containing
  * move data creation in class and use savepoint;
  * move registration creation in its own method to allow calling it if
    necessary;
  * clean some unnecessary variables.

LINKS

Task 1922187
Part of PR #34705

1 files changed, 9 insertions, 7 deletions

diff --git a/addons/event/tests/test_event_flow.py b/addons/event/tests/test_event_flow.py
index 08ac4654877..7a2364ac4c9 100644
--- a/addons/event/tests/test_event_flow.py
+++ b/addons/event/tests/test_event_flow.py
@@ -19,7 +19,7 @@ class TestEventFlow(TestEventCommon):
     def test_00_basic_event_auto_confirm(self):
         """ Basic event management with auto confirmation """
         # EventUser creates a new event: ok
-        test_event = self.Event.with_user(self.user_eventmanager).create({
+        test_event = self.env['event.event'].with_user(self.user_eventmanager).create({
             'name': 'TestEvent',
             'auto_confirm': True,
             'date_begin': datetime.datetime.now() + relativedelta(days=-1),
@@ -30,13 +30,13 @@ class TestEventFlow(TestEventCommon):
         self.assertEqual(test_event.state, 'confirm', 'Event: auto_confirmation of event failed')
 
         # EventUser create registrations for this event
-        test_reg1 = self.Registration.with_user(self.user_eventuser).create({
+        test_reg1 = self.env['event.registration'].with_user(self.user_eventuser).create({
             'name': 'TestReg1',
             'event_id': test_event.id,
         })
         self.assertEqual(test_reg1.state, 'open', 'Event: auto_confirmation of registration failed')
         self.assertEqual(test_event.seats_reserved, 1, 'Event: wrong number of reserved seats after confirmed registration')
-        test_reg2 = self.Registration.with_user(self.user_eventuser).create({
+        test_reg2 = self.env['event.registration'].with_user(self.user_eventuser).create({
             'name': 'TestReg2',
             'event_id': test_event.id,
         })
@@ -45,7 +45,7 @@ class TestEventFlow(TestEventCommon):
 
         # EventUser create registrations for this event: too much registrations
         with self.assertRaises(ValidationError):
-            self.Registration.with_user(self.user_eventuser).create({
+            self.env['event.registration'].with_user(self.user_eventuser).create({
                 'name': 'TestReg3',
                 'event_id': test_event.id,
             })
@@ -70,7 +70,7 @@ class TestEventFlow(TestEventCommon):
         """ Avanced event flow: no auto confirmation, manage minimum / maximum
         seats, ... """
         # EventUser creates a new event: ok
-        test_event = self.Event.with_user(self.user_eventmanager).create({
+        test_event = self.env['event.event'].with_user(self.user_eventmanager).create({
             'name': 'TestEvent',
             'date_begin': datetime.datetime.now() + relativedelta(days=-1),
             'date_end': datetime.datetime.now() + relativedelta(days=1),
@@ -81,7 +81,7 @@ class TestEventFlow(TestEventCommon):
             'Event: new event should be in draft state, no auto confirmation')
 
         # EventUser create registrations for this event -> no auto confirmation
-        test_reg1 = self.Registration.with_user(self.user_eventuser).create({
+        test_reg1 = self.env['event.registration'].with_user(self.user_eventuser).create({
             'name': 'TestReg1',
             'event_id': test_event.id,
         })
@@ -92,7 +92,7 @@ class TestEventFlow(TestEventCommon):
     def test_event_access_rights(self):
         # EventManager required to create or update events
         with self.assertRaises(AccessError):
-            self.Event.with_user(self.user_eventuser).create({
+            self.env['event.event'].with_user(self.user_eventuser).create({
                 'name': 'TestEvent',
                 'date_begin': datetime.datetime.now() + relativedelta(days=-1),
                 'date_end': datetime.datetime.now() + relativedelta(days=1),
@@ -114,9 +114,11 @@ class TestEventFlow(TestEventCommon):
             event_config.execute()
 
     def test_event_data(self):
+        self.event_0.write({'registration_ids': [(0, 0, {'partner_id': self.user_eventuser.partner_id.id})]})
         self.assertEqual(self.event_0.registration_ids.get_date_range_str(), u'tomorrow')
 
     def test_event_date_range(self):
+        self.event_0.write({'registration_ids': [(0, 0, {'partner_id': self.user_eventuser.partner_id.id})]})
         self.patcher = patch('odoo.addons.event.models.event.fields.Datetime', wraps=Datetime)
         self.mock_datetime = self.patcher.start()