rcv: Make iva_no_retenido optional in data models and parsers

[svillegas-cdd]

• Updated iva_no_retenido to be optional in RcDetalleEntry and its variants.
• Adjusted CSV parser to handle cases where iva_no_retenido is missing or None.
• Updated tests to reflect changes in iva_no_retenido handling.

Ref: https://app.shortcut.com/cordada/story/16600/

[Copilot]

Pull Request Overview

This PR makes the iva_no_retenido field optional across RCV data models and CSV parsers to handle cases where this field may be missing or null in the data source.

• Made iva_no_retenido field optional in data models by changing type from int to Optional[int]
• Updated CSV parser functions to use .get() method instead of direct key access for iva_no_retenido
• Updated test data to use None instead of 0 for iva_no_retenido to reflect the optional nature

Reviewed Changes

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

File	Description
src/cl_sii/rcv/data_models.py	Changed iva_no_retenido field type from int to Optional[int] in base data model
src/cl_sii/rcv/parse_csv.py	Updated CSV parser functions to handle optional iva_no_retenido field using .get() method
src/tests/test_rcv_data_models.py	Updated test case to use None instead of 0 for iva_no_retenido field

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.97%. Comparing base (c5e0015) to head (b471dfc).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #891   +/-   ##
========================================
  Coverage    88.97%   88.97%           
========================================
  Files           40       40           
  Lines         3565     3565           
  Branches       355      355           
========================================
  Hits          3172     3172           
  Misses         240      240           
  Partials       153      153           

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
66.7% Duplication on New Code

See analysis details on SonarQube Cloud