MAIN B-22267

[danieljordan-caci]

INT PR
INT PR 2

NOTE:
There was an INT specific commit in the v2 PR that is almost complete, but @TevinAdams's 21378 had a test (TestUpdateRequiredDeliveryDateUpdate) that is not yet in main so it is missing that specific change only

Agility ticket

Summary

When we first did pricing for international shipments, we were told to use DTOD for all mileage calculations, which was noted in the feature. Looks like that was incorrect so this BL is addressing dialing that requirement back and using RandMcNally aka "MilMove Miles" instead of DTOD per our GHC contract.

If we run into errors where there aren't associated rows in zip3_distances, then we need to consider that a data issue and add those ZIP combos to the database with the associated mileage. I've tested with quite a few ZIP combos, so please ensure you do a few moves with some different ZIP combos.

I've also updated a couple times where ZipTransitDistance is called to check for isInternationalShipment and where that wasn't possible I added comments explaining why the value was true or false

How to test

1. Create some moves, make a couple OCONUS with AK ZIPs
2. Go through the end to end flow alllll the way until you review the fuel surcharge as TIO
3. Verify that the mileage shown in the calculations is what is found in zip3_distances

Screenshots

[cameroncaci]

Seeing missing commits from the int PRs, for example this section

[danieljordan-caci]

Seeing missing commits from the int PRs, for example this section

That's from the destination queue BL - not relevant

[cameroncaci]

POA'd. Commits are kinda not good, but are technically good. HOWEVER, all commits present in the main branch are in fact present in int, it's just that int has other backlog item commits present per @danieljordan-caci. The V2 PR is a good match, just the V1 that gets a little silly

[danieljordan-caci]

POA'd. Commits are kinda not good, but are technically good. HOWEVER, all commits present in the main branch are in fact present in int, it's just that int has other backlog item commits present per @danieljordan-caci. The V2 PR is a good match, just the V1 that gets a little silly

Sorry about the mess lol

[traskowskycaci]

Matches other than the commit you called out. @TevinAdams making sure you see this PR and know that you may want to merge in main to your 21378 main branch and update your tests impacted here