[FIX] SavepointCase tests

analytic_contract_value/tests/test_analytic_contract_value.py

@@ -8,8 +8,10 @@
 
 
 class TestAnalyticSchedule(common.SavepointCase):
-    def setUp(cls):
-        super(TestAnalyticSchedule, cls).setUp()
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestAnalyticSchedule, cls).setUpClass()
         cls.project_project = cls.env["project.project"]
         cls.project = cls.project_project.create(
             {"name": "Test project", "code": "ACV0001"}
@@ -49,5 +51,5 @@ def setUp(cls):
             }
         )
 
-    def test_contarct_value(cls):
-        cls.assertEquals(cls.account_id.contract_value, 100000.0, "Bad value")
+    def test_contarct_value(self):
+        self.assertEquals(self.account_id.contract_value, 100000.0, "Bad value")

analytic_cost_revenue/tests/test_analytic_cost_revenue.py

@@ -5,8 +5,10 @@
 
 
 class TestAnalyticCostRevenue(common.SavepointCase):
-    def setUp(cls):
-        super(TestAnalyticCostRevenue, cls).setUp()
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestAnalyticCostRevenue, cls).setUpClass()
         cls.project_project = cls.env["project.project"]
         cls.project = cls.project_project.create(
             {"name": "Test project", "code": "ACV0001"}
@@ -78,8 +80,8 @@ def setUp(cls):
             }
         )
 
-    def test_cost_revenue(cls):
-        cls.assertEqual(cls.account_id.material_cost, 100.0)
-        cls.assertEqual(cls.account_id.labor_cost, 0.0)
-        cls.assertEqual(cls.account_id.revenue, 200.0)
-        cls.assertEqual(cls.account_id.gross_profit, 100.0)
+    def test_cost_revenue(self):
+        self.assertEqual(self.account_id.material_cost, 100.0)
+        self.assertEqual(self.account_id.labor_cost, 0.0)
+        self.assertEqual(self.account_id.revenue, 200.0)
+        self.assertEqual(self.account_id.gross_profit, 100.0)

analytic_plan_cost_revenue/tests/test_analytic_plan_cost_revenue.py

@@ -6,8 +6,10 @@
 
 
 class TestAnalyticCostRevenue(common.SavepointCase):
-    def setUp(cls):
-        super(TestAnalyticCostRevenue, cls).setUp()
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestAnalyticCostRevenue, cls).setUpClass()
         cls.project_project = cls.env["project.project"]
         cls.project = cls.project_project.create(
             {"name": "Test project", "code": "ACV0001"}
@@ -99,8 +101,8 @@ def setUp(cls):
             }
         )
 
-    def test_cost_revenue(cls):
-        cls.assertEqual(cls.account_id.material_cost_plan, -100.0)
-        cls.assertEqual(cls.account_id.labor_cost_plan, -0.0)
-        cls.assertEqual(cls.account_id.revenue_plan, 200.0)
-        cls.assertEqual(cls.account_id.gross_profit_plan, 100.0)
+    def test_cost_revenue(self):
+        self.assertEqual(self.account_id.material_cost_plan, -100.0)
+        self.assertEqual(self.account_id.labor_cost_plan, -0.0)
+        self.assertEqual(self.account_id.revenue_plan, 200.0)
+        self.assertEqual(self.account_id.gross_profit_plan, 100.0)

analytic_resource_plan/tests/test_analytic_resource_plan.py

@@ -10,8 +10,9 @@
 
 class TestAnalyticResourcePlan(common.SavepointCase):
 
-    def setUp(cls):
-        super(TestAnalyticResourcePlan, cls).setUp()
+    @classmethod
+    def setUpClass(cls):
+        super(TestAnalyticResourcePlan, cls).setUpClass()
         cls.project = cls.env['project.project'].create(
             {'name': 'Test project',
              'code': 'X0001'})
@@ -60,14 +61,14 @@ def _create_account(self, acc_type, name, code, company):
         })
         return account
 
-    def test_plan(cls):
-        cls.resource_plan_line.action_button_confirm()
-        cls.assertEqual(cls.resource_plan_line.state, 'confirm')
-        plan_line = cls.env['account.analytic.line.plan'].search(
-            [('resource_plan_id', '=', cls.resource_plan_line.id)])
-        cls.assertEqual(len(plan_line), 1, 'Wrong plan lines number')
-        cls.resource_plan_line.action_button_draft()
-        cls.assertEqual(cls.resource_plan_line.state, 'draft')
-        plan_line = cls.env['account.analytic.line.plan'].search(
-            [('resource_plan_id', '=', cls.resource_plan_line.id)])
-        cls.assertEqual(len(plan_line), 0, 'Plan line not deleted')
+    def test_plan(self):
+        self.resource_plan_line.action_button_confirm()
+        self.assertEqual(self.resource_plan_line.state, 'confirm')
+        plan_line = self.env['account.analytic.line.plan'].search(
+            [('resource_plan_id', '=', self.resource_plan_line.id)])
+        self.assertEqual(len(plan_line), 1, 'Wrong plan lines number')
+        self.resource_plan_line.action_button_draft()
+        self.assertEqual(self.resource_plan_line.state, 'draft')
+        plan_line = self.env['account.analytic.line.plan'].search(
+            [('resource_plan_id', '=', self.resource_plan_line.id)])
+        self.assertEqual(len(plan_line), 0, 'Plan line not deleted')

analytic_resource_plan_mrp/tests/test_analytic_resource_plan_mrp.py

@@ -9,8 +9,10 @@
 class TestAnalyticResourcePlanMrp(
     test_analytic_resource_plan_stock.TestAnalyticResourcePlanStock
 ):
-    def setUp(cls):
-        super(TestAnalyticResourcePlanMrp, cls).setUp()
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestAnalyticResourcePlanMrp, cls).setUpClass()
         cls.analytic_account_obj = cls.env["account.analytic.account"]
         cls.analytic_resource_plan_obj = cls.env["analytic.resource.plan.line"]
         cls.mrp_bom_obj = cls.env["mrp.bom"]
@@ -37,14 +39,13 @@ def setUp(cls):
         cls.location_id = cls.env.ref("stock.stock_location_stock")
         cls.product_id.write({"expense_analytic_plan_journal_id": 1})
 
-        cls.bom = cls._create_bom(cls.product_id)
+        cls.bom = cls._create_bom(cls.env, cls.product_id)
         cls.resource_plan_line_wbom_id = cls._create_analytic_resource_plan(
-            cls.product_id
-        )
+            cls.env, cls.product_id)
         cls.account_id.bom_id = cls.bom.id
 
-    def _create_bom(cls, product):
-        test_bom = cls.mrp_bom_obj.create(
+    def _create_bom(self, product):
+        test_bom = self.mrp_bom_obj.create(
             {
                 "product_id": product.id,
                 "product_tmpl_id": product.product_tmpl_id.id,
@@ -53,7 +54,7 @@ def _create_bom(cls, product):
                 "type": "normal",
             }
         )
-        cls.mrp_bom_line_obj.create(
+        self.mrp_bom_line_obj.create(
             {
                 "bom_id": test_bom.id,
                 "product_id": product.id,
@@ -63,9 +64,9 @@ def _create_bom(cls, product):
         return test_bom
 
     def _create_analytic_account(
-        cls, name, partner, analytic_plan_version, location
+        self, name, partner, analytic_plan_version, location
     ):
-        return cls.analytic_account_obj.create(
+        return self.analytic_account_obj.create(
             {
                 "name": name,
                 "partner_id": partner.id,
@@ -74,53 +75,53 @@ def _create_analytic_account(
             }
         )
 
-    def _create_analytic_resource_plan(cls, product):
-        return cls.analytic_resource_plan_obj.create(
+    def _create_analytic_resource_plan(self, product):
+        return self.analytic_resource_plan_obj.create(
             {
                 "name": product.name,
                 "product_id": product.id,
                 "unit_amount": 2.0,
                 "product_uom_id": product.uom_id.id,
-                "account_id": cls.account_id.id,
+                "account_id": self.account_id.id,
                 "resource_type": "procurement",
-                "bom_id": cls.bom.id,
+                "bom_id": self.bom.id,
             }
         )
 
-    def test_resource_plan_line(cls):
-        cls.resource_plan_line_wbom_id.action_button_confirm()
-        child = cls.resource_plan_line_wbom_id.child_ids
-        plan_lines = cls.env["account.analytic.line.plan"].search(
+    def test_resource_plan_line(self):
+        self.resource_plan_line_wbom_id.action_button_confirm()
+        child = self.resource_plan_line_wbom_id.child_ids
+        plan_lines = self.env["account.analytic.line.plan"].search(
             [("resource_plan_id", "=", child.id)]
         )
-        cls.assertEqual(len(plan_lines), 1, "bad plan lines")
-        cls.assertEqual(child.state, "confirm", "child not confirm")
-        cls.assertEqual(child.unit_amount, 4.0, "wrong qty in child")
+        self.assertEqual(len(plan_lines), 1, "bad plan lines")
+        self.assertEqual(child.state, "confirm", "child not confirm")
+        self.assertEqual(child.unit_amount, 4.0, "wrong qty in child")
         # test produce
         wiz_id = (
-            cls.env["analytic.resource.plan.line.produce"]
+            self.env["analytic.resource.plan.line.produce"]
             .with_context(
                 active_model="analytic.resource.plan.line",
-                active_ids=cls.resource_plan_line_wbom_id.id,
+                active_ids=self.resource_plan_line_wbom_id.id,
             )
             .create({})
         )
         move_action = wiz_id.do_produce()
         move_id = safe_eval(move_action["domain"])[0][2]
-        move = cls.env["stock.move"].browse(move_id)[0]
-        cls.assertEqual(
+        move = self.env["stock.move"].browse(move_id)[0]
+        self.assertEqual(
             move.product_qty, child.unit_amount, "Wrong consumed qty"
         )
         # test consume
         wiz_id = (
-            cls.env["analytic.resource.plan.line.consume"]
+            self.env["analytic.resource.plan.line.consume"]
             .with_context(
                 active_model="analytic.resource.plan.line",
-                active_ids=cls.resource_plan_line_wbom_id.id,
+                active_ids=self.resource_plan_line_wbom_id.id,
             )
             .create({})
         )
         wiz_id.do_consume()
-        cls.assertEqual(
+        self.assertEqual(
             move.product_qty, child.unit_amount, "Wrong consumed qty"
         )

analytic_resource_plan_purchase_request/tests/test_analytic_resource_plan_purchase.py

@@ -12,41 +12,43 @@
 class TestAnalyticResourcePlanPurchase(
     test_analytic_resource_plan_stock.TestAnalyticResourcePlanStock
 ):
-    def setUp(cls):
-        super(TestAnalyticResourcePlanPurchase, cls).setUp()
 
-    def test_res_purchase(cls):
-        cls.assertEqual(
-            cls.resource_plan_line.qty_available,
+    @classmethod
+    def setUpClass(cls):
+        super(TestAnalyticResourcePlanPurchase, cls).setUpClass()
+
+    def test_res_purchase(self):
+        self.assertEqual(
+            self.resource_plan_line.qty_available,
             0.0,
             "Showing qty where there is not",
         )
-        cls.assertEqual(
-            cls.resource_plan_line.request_state, "none", "should no request"
+        self.assertEqual(
+            self.resource_plan_line.request_state, "none", "should no request"
         )
-        cls.resource_plan_line.action_button_confirm()
-        cls.assertEqual(
-            cls.resource_plan_line.request_state,
+        self.resource_plan_line.action_button_confirm()
+        self.assertEqual(
+            self.resource_plan_line.request_state,
             "draft",
             "should draft request",
         )
-        purchase_request_line = cls.resource_plan_line.purchase_request_lines[
+        purchase_request_line = self.resource_plan_line.purchase_request_lines[
             0
         ]
         purchase_request = purchase_request_line.request_id
         purchase_request.button_to_approve()
         purchase_request.button_approved()
-        cls.assertEqual(
-            cls.resource_plan_line.request_state,
+        self.assertEqual(
+            self.resource_plan_line.request_state,
             "approved",
             "should approved request",
         )
-        cls.assertEqual(
-            cls.resource_plan_line.requested_qty, 1.0, "requested wrong qty"
+        self.assertEqual(
+            self.resource_plan_line.requested_qty, 1.0, "requested wrong qty"
         )
-        cls.resource_plan_line.action_button_draft()
-        cls.assertEqual(
-            cls.resource_plan_line.request_state,
+        self.resource_plan_line.action_button_draft()
+        self.assertEqual(
+            self.resource_plan_line.request_state,
             "rejected",
             "should rejected request",
         )