🤖 Enhance Inventory Validation and Error Handling in Order Processing

[sentry-autofix]

👋 Hi there! This PR was automatically generated by Autofix 🤖

This fix was triggered by [email protected]

Fixes FLASK-1D1

This change introduces additional validation checks and error handling in the order processing functionality. Key modifications include:

1. Immediate raise of an exception if either the inventory or quantities is empty, followed by incrementing a Sentry metric for checkout failures.
2. Enhanced checks to ensure that each inventory item is in a valid structure (not a list/tuple with fewer than three elements) before proceeding, with corresponding error handling that raises a specific exception if the structure is invalid.
3. Refined the inventory count and requested quantity comparison, and introduced detailed error messages to indicate specific issues (e.g., insufficient inventory) alongside metric increments for failed checkout attempts. These improvements aim to enhance robustness and traceability during order processing.

If you have any questions or feedback for the Sentry team about this fix, please email [email protected] with the Run ID: 1431.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
empower	⬜️ Ignored (Inspect)	Visit Preview		Nov 13, 2024 7:44pm

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: flask/src/main.py

Function	Unhandled Issue
checkout	Exception: Not enough inventory for product checkout Event Count: 3.7k
checkout	KeyError: 'form' checkout Event Count: 3
checkout	DatabaseConnectionError: get_inventory checkout Event Count: 2
checkout	KeyError: 'validate_inventory' checkout Event Count: 2
checkout	TypeError: list indices must be integers or slices, not str ... Event Count: 2

---

_{Did you find this useful? React with a 👍 or 👎}

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.

Project coverage is 35.96%. Comparing base (75af4e6) to head (7754f78).

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
flask/src/main.py	0.00%	11 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #627      +/-   ##
==========================================
- Coverage   36.21%   35.96%   -0.25%     
==========================================
  Files          37       37              
  Lines         867      873       +6     
  Branches      111      111              
==========================================
  Hits          314      314              
- Misses        536      542       +6     
  Partials       17       17              

Flag	Coverage Δ
api	7.49% <0.00%> (-0.14%)	⬇️
frontend	54.75% <ø> (ø)

Components	Coverage Δ
checkout_module	7.49% <0.00%> (-0.14%)	⬇️
product_component	56.41% <ø> (ø)

Files with missing lines	Coverage Δ
flask/src/main.py	0.00% <0.00%> (ø)