[IMP] hr_payroll: add multi contract support

Purpose
=======

We would like to be able to manage multi contracts payslips and contract transitions.

Specification
=============

General flow:
1/ Benefits are generated based on contracts's calendar.
2/ When validating additionnal benefits, calendar.attendances are created in the contract's calendar
3/ Create one payslip per contract in state [open|pending|close]
4/ For each payslip, generate worked day lines based on the contract's calendar

New Constraints:
1/ An employee cannot have more than one contract at the same time (excluding draft, closed and cancelled contracts).
2/ A leave cannot overlap multiple contracts.
3/ A contract cannot have starting and ending dates such that a leave become in the middle of
   two contacts.

New state for a contract: Incoming. It means it's ready and will be running soon.
An incoming contract automatically becomes 'Running' if the start date is passed (cron)

Number of hours and days are always correct (according to the contract's calendar) on payslips as the
flow starts from the contract's calendar.
However, it is not correct in the Time Off app because it's counted based on the employee's calendar.
The number of days/hours for hr.leaves should be computed according to
the contract at the time of the leave.
e.g. If I currently work full time and I request 2 weeks off next month where I'll be working only
3 days/week and 5hours/day (other contract), It should count 6 days and 30 hours.

- in hr.benefits form view show the contract (below employee field)

TaskID: 1903483

closes odoo/odoo#30181

3 files changed, 99 insertions, 1 deletions

diff --git a/addons/hr_contract/tests/__init__.py b/addons/hr_contract/tests/__init__.py
index 6f1832811c2..ad627a612a4 100644
--- a/addons/hr_contract/tests/__init__.py
+++ b/addons/hr_contract/tests/__init__.py
@@ -2,3 +2,4 @@
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
 from . import test_auto_status
+from . import test_contract
diff --git a/addons/hr_contract/tests/test_auto_status.py b/addons/hr_contract/tests/test_auto_status.py
index 580a6d17423..20b8e4007fc 100644
--- a/addons/hr_contract/tests/test_auto_status.py
+++ b/addons/hr_contract/tests/test_auto_status.py
@@ -10,7 +10,12 @@ class TestHrContracts(TransactionCase):
     def setUp(self):
         super(TestHrContracts, self).setUp()
         self.contracts = self.env['hr.contract'].with_context(tracking_disable=True)
-        self.employee = self.env.ref('hr.employee_admin')
+        self.employee = self.env['hr.employee'].create({
+            'name': 'Richard',
+            'gender': 'male',
+            'birthday': '1984-05-01',
+            'country_id': self.ref('base.be'),
+        })
         self.test_contract = dict(name='Test', wage=1, employee_id=self.employee.id, state='open')
 
     def apply_cron(self):
@@ -51,3 +56,9 @@ class TestHrContracts(TransactionCase):
         self.contract.write(self.test_contract)
         self.apply_cron()
         self.assertEquals(self.contract.state, 'close')
+
+    def test_contract_start_date(self):
+        self.test_contract.update(dict(date_start=datetime.now(), state='incoming'))
+        self.contract = self.contracts.create(self.test_contract)
+        self.apply_cron()
+        self.assertEquals(self.contract.state, 'open')
diff --git a/addons/hr_contract/tests/test_contract.py b/addons/hr_contract/tests/test_contract.py
new file mode 100644
index 00000000000..39d6743d275
--- /dev/null
+++ b/addons/hr_contract/tests/test_contract.py
@@ -0,0 +1,86 @@
+# -*- coding: utf-8 -*-
+# Part of Odoo. See LICENSE file for full copyright and licensing details.
+from datetime import datetime
+from odoo.tests.common import TransactionCase
+from odoo.exceptions import ValidationError
+from dateutil.relativedelta import relativedelta
+
+
+class TestHrContracts(TransactionCase):
+
+    def setUp(self):
+        super(TestHrContracts, self).setUp()
+        self.contracts = self.env['hr.contract'].with_context(tracking_disable=True)
+        self.employee = self.env['hr.employee'].create({
+            'name': 'Richard',
+            'gender': 'male',
+            'birthday': '1984-05-01',
+            'country_id': self.ref('base.be'),
+        })
+
+    def create_contract(self, state, start, end=None):
+        return self.env['hr.contract'].create({
+            'name': 'Contract',
+            'employee_id': self.employee.id,
+            'state': state,
+            'wage': 1,
+            'date_start': start,
+            'date_end': end,
+        })
+
+    def test_incoming_overlapping_contract(self):
+        start = datetime.strptime('2015-11-01', '%Y-%m-%d').date()
+        end = datetime.strptime('2015-11-30', '%Y-%m-%d').date()
+        self.create_contract('open', start, end)
+
+        # Incoming contract
+        with self.assertRaises(ValidationError, msg="It should not create two contract in state open or incoming"):
+            start = datetime.strptime('2015-11-15', '%Y-%m-%d').date()
+            end = datetime.strptime('2015-12-30', '%Y-%m-%d').date()
+            self.create_contract('incoming', start, end)
+
+    def test_pending_overlapping_contract(self):
+        start = datetime.strptime('2015-11-01', '%Y-%m-%d').date()
+        end = datetime.strptime('2015-11-30', '%Y-%m-%d').date()
+        self.create_contract('open', start, end)
+
+        # Pending contract
+        with self.assertRaises(ValidationError, msg="It should not create two contract in state open or pending"):
+            start = datetime.strptime('2015-11-15', '%Y-%m-%d').date()
+            end = datetime.strptime('2015-12-30', '%Y-%m-%d').date()
+            self.create_contract('pending', start, end)
+
+        # Draft contract -> should not raise
+        start = datetime.strptime('2015-11-15', '%Y-%m-%d').date()
+        end = datetime.strptime('2015-12-30', '%Y-%m-%d').date()
+        self.create_contract('draft', start, end)
+
+    def test_draft_overlapping_contract(self):
+        start = datetime.strptime('2015-11-01', '%Y-%m-%d').date()
+        end = datetime.strptime('2015-11-30', '%Y-%m-%d').date()
+        self.create_contract('open', start, end)
+
+        # Draft contract -> should not raise even if overlapping
+        start = datetime.strptime('2015-11-15', '%Y-%m-%d').date()
+        end = datetime.strptime('2015-12-30', '%Y-%m-%d').date()
+        self.create_contract('draft', start, end)
+
+    def test_overlapping_contract_no_end(self):
+
+        # No end date
+        self.create_contract('open', datetime.strptime('2015-11-01', '%Y-%m-%d').date())
+
+        with self.assertRaises(ValidationError):
+            start = datetime.strptime('2015-11-15', '%Y-%m-%d').date()
+            end = datetime.strptime('2015-12-30', '%Y-%m-%d').date()
+            self.create_contract('incoming', start, end)
+
+    def test_overlapping_contract_no_end2(self):
+
+        start = datetime.strptime('2015-11-01', '%Y-%m-%d').date()
+        end = datetime.strptime('2015-12-30', '%Y-%m-%d').date()
+        self.create_contract('open', start, end)
+
+        with self.assertRaises(ValidationError):
+            # No end
+            self.create_contract('incoming', datetime.strptime('2015-01-01', '%Y-%m-%d').date())