[11.0][FIX] stock_orderpoint_purchase_link:

* update PO line passes orderpoints
 * add tests

stock_orderpoint_purchase_link/__manifest__.py

@@ -4,7 +4,7 @@
 {
     "name": "Stock Orderpoint Purchase Link",
     "summary": "Link Reordering rules to purchase orders",
-    "version": "11.0.1.0.1",
+    "version": "11.0.1.0.2",
     "license": "LGPL-3",
     "website": "https://github.com/stock-logistics-warehouse",
     "author": "Eficent, "

stock_orderpoint_purchase_link/models/procurement_rule.py

@@ -28,4 +28,8 @@ class ProcurementRule(models.Model):
         if 'orderpoint_id' in values:
             vals['orderpoint_ids'] = [
                 (4, values['orderpoint_id'].id)]
+        # If the procurement was run by a stock move.
+        elif 'orderpoint_ids' in values:
+            vals['orderpoint_ids'] = [(4, o.id)
+                                      for o in values['orderpoint_ids']]
         return vals

stock_orderpoint_purchase_link/tests/__init__.py

@@ -0,0 +1 @@
+from . import test_stock_orderpoint_purchase_link

stock_orderpoint_purchase_link/tests/test_stock_orderpoint_purchase_link.py

@@ -0,0 +1,77 @@
+# Copyright 2018 Eficent Business and IT Consulting Services S.L.
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl-3.0).
+
+from odoo.tests import common
+
+
+class TestOrderpointPurchaseLink(common.TransactionCase):
+    def setUp(self):
+        super(TestOrderpointPurchaseLink, self).setUp()
+
+        self.product_obj = self.env['product.product']
+        self.partner_obj = self.env['res.partner']
+        self.pol_obj = self.env['purchase.order.line']
+        self.location_obj = self.env['stock.location']
+        self.orderpoint_obj = self.env['stock.warehouse.orderpoint']
+        self.route_obj = self.env['stock.location.route']
+        self.rule_obj = self.env['procurement.rule']
+        self.group_obj = self.env['procurement.group']
+
+        # WH and routes:
+        self.warehouse = self.env.ref('stock.warehouse0')
+        self.stock_location = self.warehouse.lot_stock_id
+        self.test_location = self.location_obj.create({
+            'name': 'Test',
+            'location_id': self.warehouse.view_location_id.id,
+        })
+        route_buy = self.env.ref('purchase.route_warehouse0_buy').id
+        route_test = self.route_obj.create({
+            'name': 'Stock to Test',
+        }).id
+        self.rule_obj.create({
+            'name': 'Stock to Test',
+            'action': 'move',
+            'procure_method': 'make_to_order',
+            'location_id': self.test_location.id,
+            'location_src_id': self.stock_location.id,
+            'route_id': route_test,
+            'picking_type_id': self.warehouse.int_type_id.id,
+            'warehouse_id': self.warehouse.id,
+            'group_propagation_option': 'none',
+        })
+        # Partners:
+        vendor1 = self.partner_obj.create({'name': 'Vendor 1'})
+
+        # Create products:
+        self.tp1 = self.product_obj.create({
+            'name': 'Test Product 1',
+            'type': 'product',
+            'list_price': 150.0,
+            'route_ids': [(6, 0, [route_buy, route_test])],
+            'seller_ids': [(0, 0, {'name': vendor1.id, 'price': 20.0})],
+        })
+
+        # Create Orderpoints:
+        self.op1 = self.orderpoint_obj.create({
+            'product_id': self.tp1.id,
+            'location_id': self.stock_location.id,
+            'product_min_qty': 5.0,
+            'product_max_qty': 20.0,
+        })
+        self.op2 = self.orderpoint_obj.create({
+            'product_id': self.tp1.id,
+            'location_id': self.test_location.id,
+            'product_min_qty': 5.0,
+            'product_max_qty': 20.0,
+        })
+
+    def test_01_po_line_from_orderpoints(self):
+        """Test that a PO line created/updated by two orderpoints keeps
+        the link with both of them."""
+        self.group_obj.run_scheduler()
+        po_line = self.env['purchase.order.line'].search(
+            [('product_id', '=', self.tp1.id)])
+        self.assertTrue(po_line)
+        # Each orderpoint must have required 20.0 units:
+        self.assertEqual(po_line.product_qty, 40.0)
+        self.assertEqual(len(po_line.orderpoint_ids), 2)