PTHMINT-78: Avoid duplicate taxes when checkout options is autogenerated

[zulquer]

This pull request refines the generate_from_shopping_cart method to handle unique tax table selectors more effectively and updates the corresponding test cases to ensure correctness. The key changes include introducing logic for deduplication of tax table selectors, updating tests to reflect this change, and adding a new test case for scenarios with duplicate tax table selectors.

Enhancements to generate_from_shopping_cart:

• Introduced logic to deduplicate tax table selectors by reducing the array of items into a set of unique tax table selectors. This ensures that the generated tax rules are based on unique selectors only (src/multisafepay/api/paths/orders/request/components/checkout_options.py, src/multisafepay/api/paths/orders/request/components/checkout_options.pyR113-R124).

Updates to test cases:

• Modified test_generate_from_shopping_cart to sort both the generated and expected tax rules before comparison, ensuring consistent test results regardless of the order of processing (tests/multisafepay/integration/api/path/orders/request/components/test_integration_orders_components_checkout_options.py, tests/multisafepay/integration/api/path/orders/request/components/test_integration_orders_components_checkout_options.pyR63-R79).
• Added a new test case, test_generate_from_shopping_cart_with_items_with_same_tax_table_selector, to validate that duplicate tax table selectors in the shopping cart are correctly handled and result in a single tax rule (tests/multisafepay/integration/api/path/orders/request/components/test_integration_orders_components_checkout_options.py, tests/multisafepay/integration/api/path/orders/request/components/test_integration_orders_components_checkout_options.pyR153-R199).

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.48%. Comparing base (14f1825) to head (9db8adf).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master      #29   +/-   ##
=======================================
  Coverage   90.48%   90.48%           
=======================================
  Files         107      107           
  Lines        2322     2323    +1     
=======================================
+ Hits         2101     2102    +1     
  Misses        221      221           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull Request Overview

This pull request fixes duplicate tax generation in the generate_from_shopping_cart method by deduplicating tax table selectors and updates tests to ensure proper validation of this behavior.

• Introduced deduplication logic using a set to ensure unique tax table selectors
• Modified existing test to sort tax rules for consistent comparison
• Added new test case to validate duplicate tax table selector handling

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File	Description
src/multisafepay/api/paths/orders/request/components/checkout_options.py	Implemented deduplication logic using set comprehension to avoid duplicate tax rules
tests/multisafepay/integration/api/path/orders/request/components/test_integration_orders_components_checkout_options.py	Updated existing test with sorting logic and added new test for duplicate tax table selector scenario

Comments suppressed due to low confidence (1)

tests/multisafepay/integration/api/path/orders/request/components/test_integration_orders_components_checkout_options.py:174

• The test uses identical CartItem objects which may not adequately test the deduplication logic. Consider using different items with the same tax_table_selector to better validate that deduplication works across different products.

                name="Geometric Candle Holders",