Skip to content

Commit

Permalink
Remove redundant extra charge entries in UPS rate tests
Browse files Browse the repository at this point in the history
Deleted zero-amount extra charge entries from various test cases in the UPS rate tests. This streamlines the test cases and clarifies the relevant charges being tested.
  • Loading branch information
jacobshilitz committed Sep 5, 2024
1 parent 8858659 commit 38ecfcc
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions modules/connectors/ups/tests/ups/test_rate.py
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,6 @@ def test_parse_rate_response(self):
"currency": "CAD",
"extra_charges": [
{"amount": 116.35, "currency": "CAD", "name": "Base charge"},
{"amount": 0.0, "currency": "CAD", "name": "14"},
{"amount": 21.52, "currency": "CAD", "name": "Fuel surcharge"},
{"amount": 20.68, "currency": "CAD", "name": "HST"},
],
Expand All @@ -135,7 +134,6 @@ def test_parse_rate_response(self):
"currency": "CAD",
"extra_charges": [
{"amount": 71.95, "currency": "CAD", "name": "Base charge"},
{"amount": 0.0, "currency": "CAD", "name": "01"},
{"amount": 13.31, "currency": "CAD", "name": "Fuel surcharge"},
{"amount": 12.79, "currency": "CAD", "name": "HST"},
],
Expand All @@ -150,7 +148,6 @@ def test_parse_rate_response(self):
"currency": "CAD",
"extra_charges": [
{"amount": 70.90, "currency": "CAD", "name": "Base charge"},
{"amount": 0.0, "currency": "CAD", "name": "13"},
{"amount": 13.12, "currency": "CAD", "name": "Fuel surcharge"},
{"amount": 12.6, "currency": "CAD", "name": "HST"},
],
Expand All @@ -165,7 +162,6 @@ def test_parse_rate_response(self):
"currency": "CAD",
"extra_charges": [
{"amount": 67.10, "currency": "CAD", "name": "Base charge"},
{"amount": 0.0, "currency": "CAD", "name": "02"},
{"amount": 12.41, "currency": "CAD", "name": "Fuel surcharge"},
{"amount": 11.93, "currency": "CAD", "name": "HST"},
],
Expand Down

0 comments on commit 38ecfcc

Please sign in to comment.