diff options
| author | Raphael Collet <rco@odoo.com> | 2019-06-17 09:45:09 +0000 |
|---|---|---|
| committer | Raphael Collet <rco@odoo.com> | 2019-07-04 11:32:22 +0000 |
| commit | b7fd679a6c41dbb694c6fb2ce4911798de0424b1 (patch) | |
| tree | cb4e75910deb2516d5397925e96ab5e680eed9e1 /addons/maintenance/tests | |
| parent | 1e6c3bec2c5ca621344b09e58dbed535e80fb343 (diff) | |
[FIX] *: sudo() -> with_user()
Diffstat (limited to 'addons/maintenance/tests')
| -rw-r--r-- | addons/maintenance/tests/test_maintenance.py | 6 | ||||
| -rw-r--r-- | addons/maintenance/tests/test_maintenance_multicompany.py | 40 |
2 files changed, 23 insertions, 23 deletions
diff --git a/addons/maintenance/tests/test_maintenance.py b/addons/maintenance/tests/test_maintenance.py index de756185210..82909496691 100644 --- a/addons/maintenance/tests/test_maintenance.py +++ b/addons/maintenance/tests/test_maintenance.py @@ -39,7 +39,7 @@ class TestEquipment(TransactionCase): def test_10_equipment_request_category(self): # Create a new equipment - equipment_01 = self.equipment.sudo(self.manager).create({ + equipment_01 = self.equipment.with_user(self.manager).create({ 'name': 'Samsung Monitor "15', 'category_id': self.ref('maintenance.equipment_monitor'), 'technician_user_id': self.ref('base.user_root'), @@ -54,7 +54,7 @@ class TestEquipment(TransactionCase): assert equipment_01, "Equipment not created" # Create new maintenance request - maintenance_request_01 = self.maintenance_request.sudo(self.user).create({ + maintenance_request_01 = self.maintenance_request.with_user(self.user).create({ 'name': 'Resolution is bad', 'user_id': self.user.id, 'owner_user_id': self.user.id, @@ -71,7 +71,7 @@ class TestEquipment(TransactionCase): self.assertEquals(maintenance_request_01.stage_id.id, self.ref('maintenance.stage_0')) # I check that change the maintenance_request stage on click statusbar - maintenance_request_01.sudo(self.user).write({'stage_id': self.ref('maintenance.stage_1')}) + maintenance_request_01.with_user(self.user).write({'stage_id': self.ref('maintenance.stage_1')}) # I check that maintenance request is in the "In Progress" stage self.assertEquals(maintenance_request_01.stage_id.id, self.ref('maintenance.stage_1')) diff --git a/addons/maintenance/tests/test_maintenance_multicompany.py b/addons/maintenance/tests/test_maintenance_multicompany.py index 39b4be27a8d..f535e7f4ce3 100644 --- a/addons/maintenance/tests/test_maintenance_multicompany.py +++ b/addons/maintenance/tests/test_maintenance_multicompany.py @@ -58,52 +58,52 @@ class TestEquipmentMulticompany(TransactionCase): }) # create a maintenance team for company A user - team = MaintenanceTeam.sudo(equipment_manager).create({ + team = MaintenanceTeam.with_user(equipment_manager).create({ 'name': 'Metrology', 'company_id': company_a.id, }) # create a maintenance team for company B user - teamb = MaintenanceTeam.sudo(equipment_manager).with_context(allowed_company_ids=cids).create({ + teamb = MaintenanceTeam.with_user(equipment_manager).with_context(allowed_company_ids=cids).create({ 'name': 'Subcontractor', 'company_id': company_b.id, }) # User should not able to create equipment category. with self.assertRaises(AccessError): - Category.sudo(user).create({ + Category.with_user(user).create({ 'name': 'Software', 'company_id': company_b.id, 'technician_user_id': user.id, }) # create equipment category for equipment manager - category_1 = Category.sudo(equipment_manager).with_context(allowed_company_ids=cids).create({ + category_1 = Category.with_user(equipment_manager).with_context(allowed_company_ids=cids).create({ 'name': 'Monitors', 'company_id': company_b.id, 'technician_user_id': equipment_manager.id, }) # create equipment category for equipment manager - Category.sudo(equipment_manager).with_context(allowed_company_ids=cids).create({ + Category.with_user(equipment_manager).with_context(allowed_company_ids=cids).create({ 'name': 'Computers', 'company_id': company_b.id, 'technician_user_id': equipment_manager.id, }) # create equipment category for equipment user - Category.sudo(equipment_manager).create({ + Category.with_user(equipment_manager).create({ 'name': 'Phones', 'company_id': company_a.id, 'technician_user_id': equipment_manager.id, }) # Check category for user equipment_manager and user - self.assertEquals(Category.sudo(equipment_manager).with_context(allowed_company_ids=cids).search_count([]), 3) - self.assertEquals(Category.sudo(user).search_count([]), 2) + self.assertEquals(Category.with_user(equipment_manager).with_context(allowed_company_ids=cids).search_count([]), 3) + self.assertEquals(Category.with_user(user).search_count([]), 2) # User should not able to create equipment. with self.assertRaises(AccessError): - Equipment.sudo(user).create({ + Equipment.with_user(user).create({ 'name': 'Samsung Monitor 15', 'category_id': category_1.id, 'assign_date': time.strftime('%Y-%m-%d'), @@ -111,7 +111,7 @@ class TestEquipmentMulticompany(TransactionCase): 'owner_user_id': user.id, }) - Equipment.sudo(equipment_manager).with_context(allowed_company_ids=cids).create({ + Equipment.with_user(equipment_manager).with_context(allowed_company_ids=cids).create({ 'name': 'Acer Laptop', 'category_id': category_1.id, 'assign_date': time.strftime('%Y-%m-%d'), @@ -120,7 +120,7 @@ class TestEquipmentMulticompany(TransactionCase): }) # create an equipment for user - Equipment.sudo(equipment_manager).with_context(allowed_company_ids=cids).create({ + Equipment.with_user(equipment_manager).with_context(allowed_company_ids=cids).create({ 'name': 'HP Laptop', 'category_id': category_1.id, 'assign_date': time.strftime('%Y-%m-%d'), @@ -128,21 +128,21 @@ class TestEquipmentMulticompany(TransactionCase): 'owner_user_id': equipment_manager.id, }) # Now there are total 2 equipments created and can view by equipment_manager user - self.assertEquals(Equipment.sudo(equipment_manager).with_context(allowed_company_ids=cids).search_count([]), 2) + self.assertEquals(Equipment.with_user(equipment_manager).with_context(allowed_company_ids=cids).search_count([]), 2) # And there is total 1 equipment can be view by Normal User ( Which user is followers) - self.assertEquals(Equipment.sudo(user).search_count([]), 1) + self.assertEquals(Equipment.with_user(user).search_count([]), 1) # create an equipment team BY user with self.assertRaises(AccessError): - MaintenanceTeam.sudo(user).create({ + MaintenanceTeam.with_user(user).create({ 'name': 'Subcontractor', 'company_id': company_b.id, }) # create an equipment category BY user with self.assertRaises(AccessError): - Category.sudo(user).create({ + Category.with_user(user).create({ 'name': 'Computers', 'company_id': company_b.id, 'technician_user_id': user.id, @@ -150,12 +150,12 @@ class TestEquipmentMulticompany(TransactionCase): # create an maintenance stage BY user with self.assertRaises(AccessError): - self.env['maintenance.stage'].sudo(user).create({ + self.env['maintenance.stage'].with_user(user).create({ 'name': 'identify corrective maintenance requirements', }) # Create an maintenance request for ( User Follower ). - MaintenanceRequest.sudo(user).create({ + MaintenanceRequest.with_user(user).create({ 'name': 'Some keys are not working', 'company_id': company_b.id, 'user_id': user.id, @@ -163,7 +163,7 @@ class TestEquipmentMulticompany(TransactionCase): }) # Create an maintenance request for equipment_manager (Admin Follower) - MaintenanceRequest.sudo(equipment_manager).create({ + MaintenanceRequest.with_user(equipment_manager).create({ 'name': 'Battery drains fast', 'company_id': company_a.id, 'user_id': equipment_manager.id, @@ -171,5 +171,5 @@ class TestEquipmentMulticompany(TransactionCase): }) # Now here is total 1 maintenance request can be view by Normal User - self.assertEquals(MaintenanceRequest.sudo(equipment_manager).with_context(allowed_company_ids=cids).search_count([]), 2) - self.assertEquals(MaintenanceRequest.sudo(user).search_count([]), 1) + self.assertEquals(MaintenanceRequest.with_user(equipment_manager).with_context(allowed_company_ids=cids).search_count([]), 2) + self.assertEquals(MaintenanceRequest.with_user(user).search_count([]), 1) |
