odoo-addon-shopfloor 16.0.1.0.0.24__py3-none-any.whl

odoo/addons/shopfloor/tests/test_cluster_picking_scan_destination_no_prefill_qty.py

@@ -0,0 +1,115 @@
+# Copyright 2020 Camptocamp SA (http://www.camptocamp.com)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+from .test_cluster_picking_base import ClusterPickingCommonCase
+
+# pylint: disable=missing-return
+
+
+class ClusterPickingScanDestinationPackPrefillQtyCase(ClusterPickingCommonCase):
+    """Tests covering the /scan_destination_pack endpoint
+
+    With the no prefill quantity option set
+
+    """
+
+    @classmethod
+    def setUpClassBaseData(cls, *args, **kwargs):
+        super().setUpClassBaseData(*args, **kwargs)
+        cls.menu.sudo().no_prefill_qty = True
+        cls.batch = cls._create_picking_batch(
+            [
+                [
+                    cls.BatchProduct(product=cls.product_a, quantity=10),
+                    cls.BatchProduct(product=cls.product_b, quantity=10),
+                ],
+                [cls.BatchProduct(product=cls.product_a, quantity=10)],
+            ]
+        )
+        cls.one_line_picking = cls.batch.picking_ids.filtered(
+            lambda picking: len(picking.move_ids) == 1
+        )
+        cls.two_lines_picking = cls.batch.picking_ids.filtered(
+            lambda picking: len(picking.move_ids) == 2
+        )
+
+        cls.bin1 = cls.env["stock.quant.package"].create({})
+        cls.bin2 = cls.env["stock.quant.package"].create({})
+
+        cls._simulate_batch_selected(cls.batch)
+
+    def test_scan_destination_pack_increment_with_product(self):
+        """Check quantity increment by scanning the product."""
+        line = self.batch.picking_ids.move_line_ids[0]
+        previous_qty_done = line.qty_done
+        for qty_done in range(1, 2):
+            response = self.service.dispatch(
+                "scan_destination_pack",
+                params={
+                    "picking_batch_id": self.batch.id,
+                    "move_line_id": line.id,
+                    "barcode": line.product_id.barcode,
+                    "quantity": qty_done,
+                },
+            )
+            # Ensure the qty has not changed.
+            self.assertEqual(line.qty_done, previous_qty_done)
+
+            expected_qty_done = qty_done + 1
+            self.assert_response(
+                response,
+                next_state="scan_destination",
+                data=self._line_data(line, qty_done=expected_qty_done),
+            )
+
+    def test_scan_destination_pack_increment_with_wrong_product(self):
+        """Check quantity is not incremented by scanning the wrong product."""
+        line = self.batch.picking_ids.move_line_ids[0]
+        previous_qty_done = line.qty_done
+        qty_done = 2
+        response = self.service.dispatch(
+            "scan_destination_pack",
+            params={
+                "picking_batch_id": self.batch.id,
+                "move_line_id": line.id,
+                "barcode": self.product_b.barcode,
+                "quantity": qty_done,
+            },
+        )
+        # Ensure the qty has not changed.
+        self.assertEqual(line.qty_done, previous_qty_done)
+
+        expected_qty_done = qty_done
+        self.assert_response(
+            response,
+            next_state="scan_destination",
+            data=self._line_data(line, qty_done=expected_qty_done),
+            message=self.service.msg_store.bin_not_found_for_barcode(
+                self.product_b.barcode
+            ),
+        )
+
+    def test_scan_destination_pack_increment_with_packaging(self):
+        """Check quantity incremented by scanning the packaging."""
+        line = self.batch.picking_ids.move_line_ids[0]
+        previous_qty_done = line.qty_done
+        packaging = self.product_a_packaging
+        qty_done = 2
+        response = self.service.dispatch(
+            "scan_destination_pack",
+            params={
+                "picking_batch_id": self.batch.id,
+                "move_line_id": line.id,
+                "barcode": packaging.barcode,
+                "quantity": qty_done,
+            },
+        )
+        # Ensure the qty has not changed in the record.
+        self.assertEqual(line.qty_done, previous_qty_done)
+
+        expected_qty_done = qty_done + packaging.qty
+        self.assert_response(
+            response,
+            next_state="scan_destination",
+            data=self._line_data(line, qty_done=expected_qty_done),
+        )

odoo/addons/shopfloor/tests/test_cluster_picking_scan_line.py

@@ -0,0 +1,402 @@
+# Copyright 2020 Camptocamp SA (http://www.camptocamp.com)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+from .test_cluster_picking_base import ClusterPickingLineCommonCase
+
+
+class ClusterPickingScanLineCase(ClusterPickingLineCommonCase):
+    """Tests covering the /scan_line endpoint
+
+    After a batch has been selected and the user confirmed they are
+    working on it.
+
+    User scans something and the scan_line endpoints validates they
+    scanned the proper thing to pick.
+    """
+
+    def _scan_line_ok(self, line, scanned, expected_qty_done=1):
+        batch = line.picking_id.batch_id
+        previous_qty_done = line.qty_done
+        response = self.service.dispatch(
+            "scan_line",
+            params={
+                "picking_batch_id": batch.id,
+                "move_line_id": line.id,
+                "barcode": scanned,
+            },
+        )
+        # For any barcode scanned, the quantity done is set in
+        # the response data to fully done but the record is not updated.
+        # We ensure the qty has not changed in the record.
+        self.assertEqual(line.qty_done, previous_qty_done)
+
+        self.assert_response(
+            response,
+            next_state="scan_destination",
+            data=self._line_data(line, qty_done=expected_qty_done),
+        )
+
+    def _scan_line_error(self, line, scanned, message, sublocation=None):
+        batch = line.picking_id.batch_id
+        response = self.service.dispatch(
+            "scan_line",
+            params={
+                "picking_batch_id": batch.id,
+                "move_line_id": line.id,
+                "barcode": scanned,
+            },
+        )
+        kw = {"sublocation": self.data.location(sublocation)} if sublocation else {}
+        self.assert_response(
+            response,
+            next_state="start_line",
+            data=self._line_data(line, **kw),
+            message=message,
+        )
+
+    def test_scan_line_pack_ok(self):
+        """Scan to check if user picks the correct pack for current line"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.package_id.name)
+
+    def test_scan_line_product_ok(self):
+        """Scan to check if user picks the correct product for current line"""
+        self._simulate_batch_selected(self.batch)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.product_id.barcode)
+
+    def test_scan_line_lot_ok(self):
+        """Scan to check if user picks the correct lot for current line"""
+        self.product_a.tracking = "lot"
+        self._simulate_batch_selected(self.batch, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.lot_id.name)
+
+    def test_scan_line_serial_ok(self):
+        """Scan to check if user picks the correct serial for current line"""
+        self.product_a.tracking = "serial"
+        self._simulate_batch_selected(self.batch, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.lot_id.name)
+
+    def test_scan_line_error_product_tracked(self):
+        """Scan a product tracked by lot, must scan the lot.
+
+        If for the same product there is multiple lots in the location,
+        the user must scan the lot.
+        """
+        self.product_a.tracking = "lot"
+        self._simulate_batch_selected(self.batch, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        # Add another lot for the same product in the location
+        location = self.batch.picking_ids.location_id
+        new_lot = self.env["stock.lot"].create(
+            {"product_id": self.product_a.id, "company_id": self.env.company.id}
+        )
+        self._update_qty_in_location(location, line.product_id, 2, lot=new_lot)
+        self._scan_line_error(
+            line,
+            line.product_id.barcode,
+            {
+                "message_type": "warning",
+                "body": "Product tracked by lot, please scan one.",
+            },
+        )
+
+    def test_scan_line_lot_ok_only_one_in_location(self):
+        """Scan a product tracked by lot but no error.
+
+        If only one lot for that product is in the location, it can
+        be safely picked up.
+        """
+        self.product_a.tracking = "lot"
+        self._simulate_batch_selected(self.batch, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.lot_id.name)
+
+    def test_scan_line_product_error_several_packages(self):
+        """When we scan a product which is in more than one package, error"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        line = self.batch.picking_ids.move_line_ids
+        # create a second move line for the same product in a different
+        # package
+        move = line.move_id.copy()
+        self._fill_stock_for_moves(move, in_package=True)
+        move._action_confirm(merge=False)
+        move._action_assign()
+
+        self._scan_line_error(
+            line,
+            move.product_id.barcode,
+            {
+                "message_type": "warning",
+                "body": "This product is part of multiple"
+                " packages, please scan a package.",
+            },
+        )
+
+    def test_scan_line_product_error_in_one_package_and_raw_same_location(self):
+        """Scan product which is both in a package and as raw in same location"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        line = self.batch.picking_ids.move_line_ids
+        # create a second move line for the same product in a different
+        # package
+        move = line.move_id.copy()
+        self._fill_stock_for_moves(move)
+        move._action_confirm(merge=False)
+        move._action_assign()
+        move.move_line_ids[0].package_id = None
+
+        self._scan_line_error(
+            line,
+            move.product_id.barcode,
+            {
+                "message_type": "warning",
+                "body": "This product is part of multiple"
+                " packages, please scan a package.",
+            },
+        )
+
+    def test_scan_line_product_error_in_one_package_and_raw_different_location(self):
+        """Scan product which is both in a package and as raw in another location"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        line = self.batch.picking_ids.move_line_ids
+        # create a second move line for the same product in a different
+        # package
+        move = line.move_id.copy()
+        move.location_id = line.location_id.copy()
+        self._fill_stock_for_moves(move)
+        move._action_confirm(merge=False)
+        move._action_assign()
+        move.move_line_ids[0].package_id = None
+        move.move_line_ids[0].location_id = line.location_id.copy()
+        self._scan_line_ok(line, move.product_id.barcode)
+
+    def test_scan_line_lot_error_several_packages(self):
+        """When we scan a lot which is in more than one package, error"""
+        self._simulate_batch_selected(self.batch, in_package=True, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        # create a second move line for the same product in a different
+        # package
+        move = line.move_id.copy()
+        self._fill_stock_for_moves(move, in_lot=line.lot_id)
+        move._action_confirm(merge=False)
+        move._action_assign()
+
+        self._scan_line_error(
+            line,
+            line.lot_id.name,
+            {
+                "message_type": "warning",
+                "body": "This lot is part of multiple"
+                " packages, please scan a package.",
+            },
+        )
+
+    def test_scan_line_lot_error_in_one_package_and_unit(self):
+        """When we scan a lot which is in a package and as raw, error"""
+        self._simulate_batch_selected(self.batch, in_package=True, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        # create a second move line for the same product in a different
+        # package
+        move = line.move_id.copy()
+        self._fill_stock_for_moves(move, in_lot=line.lot_id)
+        move._action_confirm(merge=False)
+        move._action_assign()
+        self._scan_line_error(
+            line,
+            line.lot_id.name,
+            {
+                "message_type": "warning",
+                "body": "This lot is part of multiple"
+                " packages, please scan a package.",
+            },
+        )
+
+    def test_scan_line_location_ok_single_package(self):
+        """Scan to check if user scans a correct location for current line
+
+        If there is only one single package in the location, there is no
+        ambiguity so we can use it.
+        """
+        self._simulate_batch_selected(self.batch, in_package=True)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.location_id.barcode)
+
+    def test_scan_line_location_ok_single_product(self):
+        """Scan to check if user scans a correct location for current line
+
+        If there is only one single product in the location, there is no
+        ambiguity so we can use it.
+        """
+        self._simulate_batch_selected(self.batch)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.location_id.barcode)
+
+    def test_scan_line_location_ok_single_lot(self):
+        """Scan to check if user scans a correct location for current line
+
+        If there is only one single lot in the location, there is no
+        ambiguity so we can use it.
+        """
+        self._simulate_batch_selected(self.batch, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        self._scan_line_ok(line, line.location_id.barcode)
+
+    def test_scan_line_location_error_several_package(self):
+        """Scan to check if user scans a correct location for current line
+
+        If there are several packages in the location, user has to scan one.
+        """
+        self._simulate_batch_selected(self.batch, in_package=True)
+        line = self.batch.picking_ids.move_line_ids
+        location = line.location_id
+        pack_1 = line.package_id
+        # add a second package for another product in the location
+        self._create_package_in_location(
+            location, [self.PackageContent(self.product_b, 10, lot=None)]
+        )
+        # it leads to an error, but now the location is kept as working location
+        self._scan_line_error(
+            line,
+            location.barcode,
+            {
+                "message_type": "warning",
+                "body": "Several packages found in Stock, please scan a package.",
+            },
+            sublocation=location,
+        )
+        # scanning the package works
+        self._scan_line_ok(line, pack_1.name)
+
+    def test_scan_line_location_error_several_products(self):
+        """Scan to check if user scans a correct location for current line
+
+        If there are several products in the location, user has to scan one.
+        """
+        self._simulate_batch_selected(self.batch)
+        line = self.batch.picking_ids.move_line_ids
+        location = line.location_id
+        # add a second product in the location, leads to an error
+        self._update_qty_in_location(location, self.product_b, 10)
+        self._scan_line_error(
+            line,
+            location.barcode,
+            {
+                "message_type": "warning",
+                "body": "Several products found in Stock, please scan a product.",
+            },
+            sublocation=location,
+        )
+        self._scan_line_ok(line, self.product_a.barcode)
+
+    def test_scan_line_location_error_several_lots(self):
+        """Scan to check if user scans a correct location for current line
+
+        If there are several lots in the location, user has to scan one.
+        """
+        self._simulate_batch_selected(self.batch, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        location = line.location_id
+        # add a 2nd lot in the same location
+        lot_2 = (
+            self.env["stock.lot"]
+            .sudo()
+            .create(
+                {
+                    "name": "LOT_2nd",
+                    "product_id": line.product_id.id,
+                    "company_id": self.env.company.id,
+                }
+            )
+        )
+        self._update_qty_in_location(location, line.product_id, 10, lot=lot_2)
+        # leads to an error
+        self._scan_line_error(
+            line,
+            location.barcode,
+            {
+                "message_type": "warning",
+                "body": "Several lots found in Stock, please scan a lot.",
+            },
+            sublocation=location,
+        )
+        self._scan_line_ok(line, line.lot_id.name)
+
+    def test_scan_line_error_wrong_package(self):
+        """Wrong package scanned"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        pack = self.env["stock.quant.package"].sudo().create({})
+        self._scan_line_error(
+            self.batch.picking_ids.move_line_ids,
+            pack.name,
+            {"message_type": "error", "body": "Wrong pack."},
+        )
+
+    def test_scan_line_error_wrong_product(self):
+        """Wrong product scanned"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        product = (
+            self.env["product.product"]
+            .sudo()
+            .create(
+                {
+                    "name": "Wrong",
+                    "barcode": "WRONGPRODUCT",
+                }
+            )
+        )
+        self._scan_line_error(
+            self.batch.picking_ids.move_line_ids,
+            product.barcode,
+            {"message_type": "error", "body": "Wrong product."},
+        )
+
+    def test_scan_line_error_wrong_lot(self):
+        """Wrong product scanned"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        lot = (
+            self.env["stock.lot"]
+            .sudo()
+            .create(
+                {
+                    "name": "WRONGLOT",
+                    "product_id": self.batch.picking_ids.move_line_ids[0].product_id.id,
+                    "company_id": self.env.company.id,
+                }
+            )
+        )
+        self._scan_line_error(
+            self.batch.picking_ids.move_line_ids,
+            lot.name,
+            {"message_type": "error", "body": "Wrong lot."},
+        )
+
+    def test_scan_line_error_wrong_location(self):
+        """Wrong product scanned"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        location = (
+            self.env["stock.location"]
+            .sudo()
+            .create(
+                {
+                    "name": "Wrong",
+                    "barcode": "WRONGLOCATION",
+                }
+            )
+        )
+        self._scan_line_error(
+            self.batch.picking_ids.move_line_ids,
+            location.barcode,
+            {"message_type": "error", "body": "Wrong location."},
+        )
+
+    def test_scan_line_error_not_found(self):
+        """Nothing found for the barcode"""
+        self._simulate_batch_selected(self.batch, in_package=True)
+        self._scan_line_error(
+            self.batch.picking_ids.move_line_ids,
+            "NO_EXISTING_BARCODE",
+            {"message_type": "error", "body": "Barcode not found"},
+        )

odoo/addons/shopfloor/tests/test_cluster_picking_scan_line_location_or_pack_first.py

@@ -0,0 +1,114 @@
+# Copyright 2023 Camptocamp SA (http://www.camptocamp.com)
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
+from .test_cluster_picking_base import ClusterPickingLineCommonCase
+
+# pylint: disable=missing-return
+
+
+class ClusterPickingScanLineLocationOrPackFirstCase(ClusterPickingLineCommonCase):
+    """Tests covering the /scan_line endpoint
+
+    When the scan location or pack frist option enabled.
+
+    """
+
+    def setUp(self):
+        super().setUp()
+        self.menu.sudo().scan_location_or_pack_first = True
+
+    def _scan_line_error(self, line, scanned, message, sublocation=None):
+        batch = line.picking_id.batch_id
+        response = self.service.dispatch(
+            "scan_line",
+            params={
+                "picking_batch_id": batch.id,
+                "move_line_id": line.id,
+                "barcode": scanned,
+                "sublocation_id": sublocation.id if sublocation else None,
+            },
+        )
+        kw = {"sublocation": self.data.location(sublocation)} if sublocation else {}
+        self.assert_response(
+            response,
+            next_state="start_line",
+            data=self._line_data(line, scan_location_or_pack_first=True, **kw),
+            message=message,
+        )
+        return response
+
+    def _scan_line_ok(self, line, scanned, expected_qty_done=1, sublocation_id=None):
+        batch = line.picking_id.batch_id
+        response = self.service.dispatch(
+            "scan_line",
+            params={
+                "picking_batch_id": batch.id,
+                "move_line_id": line.id,
+                "barcode": scanned,
+                "sublocation_id": sublocation_id,
+            },
+        )
+        self.assert_response(
+            response,
+            next_state="scan_destination",
+            data=self._line_data(
+                line, qty_done=expected_qty_done, scan_location_or_pack_first=True
+            ),
+        )
+
+    def test_scan_line_product_ask_for_package(self):
+        """Check scanning the product first will request to scan the package.
+
+        This is if the line being worked on as a package.
+
+        """
+        self._simulate_batch_selected(self.batch, in_package=True, in_lot=False)
+        line = self.batch.picking_ids.move_line_ids
+        location = self.batch.picking_ids.location_id
+        self._update_qty_in_location(location, self.product_a, 2)
+        self._scan_line_error(
+            line,
+            line.product_id.barcode,
+            self.msg_store.line_has_package_scan_package(),
+        )
+
+    def test_scan_line_product_ask_for_location(self):
+        """Check scanning the product first will request to scan the location.
+
+        That is if the product on the move line is tracked by lot.
+
+        """
+        self._simulate_batch_selected(self.batch, in_package=False, in_lot=True)
+        line = self.batch.picking_ids.move_line_ids
+        location = self.batch.picking_ids.location_id
+        self._update_qty_in_location(location, self.product_a, 2)
+        self._scan_line_error(
+            line, line.product_id.barcode, self.msg_store.scan_the_location_first()
+        )
+
+    def test_scan_line_location_ask_for_package(self):
+        """Check scanning location for a line with pack will ask to scan the pack."""
+        self._simulate_batch_selected(self.batch, in_package=True, in_lot=False)
+        line = self.batch.picking_ids.move_line_ids
+        location = self.batch.picking_ids.location_id
+        self._update_qty_in_location(location, self.product_a, 2)
+        self._scan_line_error(
+            line,
+            line.location_id.barcode,
+            self.msg_store.line_has_package_scan_package(),
+            sublocation=location,
+        )
+
+    def test_scan_line_location_with_multiple_product(self):
+        """Check scanning a location then a product without package.
+
+        When there is multiple product in the location and the location is scanned,
+        The user needs to scan the product but the system does not remember the location ?
+
+        """
+        self._simulate_batch_selected(self.batch, in_package=False, in_lot=False)
+        line = self.batch.picking_ids.move_line_ids
+        location = self.batch.picking_ids.location_id
+        self._update_qty_in_location(location, self.product_a, 2)
+        self._update_qty_in_location(location, self.product_b, 2)
+        self._scan_line_ok(line, line.product_id.barcode, 1.0, location.id)