B 21474 int calculate mileage using dtod for ak oconus shipments

[joeydoyecaci]

B-21474

Summary

Adds the ability to use DTOD to calculate mileage if either destination or pickup addresses are OCONUS to support calculation of RDD

###Setup
A breakpoint is required for the following lines
pkg/services/mto_shipment/mto_shipment_updater.go:1206
pkg/services/mto_shipment/mto_shipment_updater.go:1208

How to test

[important] Debugger must be enabled as RDD is not currently displayed

1. Create a new customer with a new HHG shipment. One of the addresses in the shipment should be Alaska
2. Submit shipment as SC like normal, approve the shipment as TOO
3. Update the shipment as a Prime user with all required fields, including a weight.
4. In calculate CalculateRequiredDeliveryDate use a breakpoint to verify thatplanner.ZipTransitDistance is run with internationalShipment set to true.
5. step through planner.ZipTransitDistance and verify that DTOD is called to calculate the mileage
6. verify that a nonzero mileage is being returned for planner.ZipTransitDistance

[loganwc]

Got 3708 mi between my house and Elmendorf, looks good to me

Google maps doesn't use the same data so there's not really a good way to confirm

[pambecker]

If this doesn't show on the UI, how will Mai test this and POA go for this story?

[joeydoyecaci]

If this doesn't show on the UI, how will Mai test this and POA go for this story?

In this case I'll need to demo the change at POA to show that DTOD is being called on a OCONUS shipment instead of RandMcNally. While the RDD can be seen on an HHG shipment after it is updated by Prime, I don't think it would provide the best visibility on this change.

[pambecker]

If this doesn't show on the UI, how will Mai test this and POA go for this story?

In this case I'll need to demo the change at POA to show that DTOD is being called on a OCONUS shipment instead of RandMcNally. While the RDD can be seen on an HHG shipment after it is updated by Prime, I don't think it would provide the best visibility on this change.

Ok, let's bring this up in team sync, just to get eyes on this early and to bring awareness.

[danieljordan-caci]

most excellent! I agree that the useDTOD became redundant with the addition of the isInternationalShipment bool. LGTM!