[FIX] no create payments with paid by company

OCA · Nov 16, 2023 · af82747 · af82747
1 parent 687c309
commit af82747
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 89 deletions.
diff --git a/hr_expense_cancel/tests/test_hr_expense_cancel.py b/hr_expense_cancel/tests/test_hr_expense_cancel.py
@@ -79,13 +79,8 @@ def test_action_cancel_no_update_posted(self):
     def test_action_cancel_company_account(self):
         self.expense.payment_mode = "company_account"
         self.expense_sheet.action_sheet_move_create()
-
-        self.assertEqual(len(self.expense_sheet.payment_ids), 1)
         self.assertTrue(self.expense_sheet.account_move_id)
-
         self.expense_sheet.action_cancel()
-        payment = self.expense_sheet.payment_ids.filtered(lambda l: l.state != "cancel")
-        self.assertFalse(payment)
         self.assertFalse(self.expense_sheet.account_move_id)
 
     def test_action_cancel_own_account(self):

diff --git a/hr_expense_payment/models/__init__.py b/hr_expense_payment/models/__init__.py
@@ -1,5 +1,4 @@
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
 from . import account_payment
-from . import hr_expense
 from . import hr_expense_sheet
diff --git a/hr_expense_payment/models/hr_expense.py b/hr_expense_payment/models/hr_expense.py
diff --git a/hr_expense_payment/tests/test_hr_expense_payment.py b/hr_expense_payment/tests/test_hr_expense_payment.py
@@ -82,18 +82,3 @@ def test_get_payment_vals(self):
         self.assertFalse(self.expense_sheet.payment_ids)
         payment_wizard.action_create_payments()
         self.assertEqual(len(self.expense_sheet.payment_ids), 1)
-
-    def test_action_sheet_move_create(self):
-        self.expense.payment_mode = "company_account"
-        self.assertFalse(self.expense_sheet.payment_ids)
-        self.expense_sheet.action_sheet_move_create()
-        self.assertEqual(len(self.expense_sheet.payment_ids), 1)
-
-    def test_action_sheet_move_create_skip_payment(self):
-        self.expense.payment_mode = "company_account"
-        self.assertFalse(self.expense_sheet.payment_ids)
-        self.expense_sheet.with_context(
-            skip_create_payment_company_account=True
-        ).action_sheet_move_create()
-        self.assertFalse(self.expense_sheet.payment_ids)
-        self.assertEqual(self.expense_sheet.state, "done")