From de92a0c9b8394e6f73753fe6da3b032afd17ecc3 Mon Sep 17 00:00:00 2001 From: Cedric Collins Date: Tue, 11 Oct 2022 19:27:18 -0500 Subject: [PATCH] [IMP] delivery_purolator: leave `carrier_price` unchanged during shipping H11065 --- delivery_purolator/models/delivery_purolator.py | 2 +- delivery_purolator/tests/test_purolator.py | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/delivery_purolator/models/delivery_purolator.py b/delivery_purolator/models/delivery_purolator.py index 11cc6fa0..fe9de227 100644 --- a/delivery_purolator/models/delivery_purolator.py +++ b/delivery_purolator/models/delivery_purolator.py @@ -353,7 +353,7 @@ class ProviderPurolator(models.Model): picking.carrier_tracking_ref = shipment_pin shipping_data = { - 'exact_price': 0.0, # TODO How can we know?! + 'exact_price': picking.carrier_price, # price is set during planning 'tracking_number': shipment_pin, } res.append(shipping_data) diff --git a/delivery_purolator/tests/test_purolator.py b/delivery_purolator/tests/test_purolator.py index efa37c0c..f70295fa 100644 --- a/delivery_purolator/tests/test_purolator.py +++ b/delivery_purolator/tests/test_purolator.py @@ -136,9 +136,11 @@ class TestPurolator(TransactionCase): # Basic case: no qty done or packages or anything at all really # it makes sense to be able to do 'something' in this case + picking.carrier_price = 50.0 picking.send_to_shipper() self.assertTrue(picking.carrier_tracking_ref) self.assertEqual(picking.message_attachment_count, 1) # has tracking label now + self.assertEqual(picking.carrier_price, 50.0) # price is set during planning and should remain unchanged # Void picking.cancel_shipment()