Skip to content

Conversation

svillegas-cdd
Copy link
Contributor

  • Implemented parse_taxpayer_data to parse taxpayer data from CTE HTML.
  • Added TaxpayerData and LastFiledDocument data models.
  • Created tests to validate parser functionality with sample HTML input.

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

@svillegas-cdd svillegas-cdd requested a review from a team as a code owner September 12, 2025 15:50
@svillegas-cdd svillegas-cdd changed the title feat(cte): Add parser for "Datos del Contribuyente" cte: Add parser for "Datos del Contribuyente" Sep 12, 2025
@svillegas-cdd svillegas-cdd self-assigned this Sep 12, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

The PR implements a parser for the "Datos del Contribuyente" (Taxpayer Data) section from CTE HTML content. It adds functionality to extract taxpayer information including start dates, activities, tax categories, and document filings.

  • Implemented parse_taxpayer_data function to parse taxpayer data from CTE HTML
  • Added TaxpayerData and LastFiledDocument data models with proper field definitions
  • Created comprehensive tests to validate parser functionality with sample HTML input

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/tests/test_data/sii-cte/cte_empty_f29.html Sample HTML test data containing CTE structure with taxpayer information
src/tests/test_cte_parsers.py Test cases for the new taxpayer data parser functionality
src/cl_sii/cte/parsers.py Implementation of parse_taxpayer_data parser function
src/cl_sii/cte/data_models.py New data models TaxpayerData and LastFiledDocument

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

Copy link

codecov bot commented Sep 12, 2025

Codecov Report

❌ Patch coverage is 82.08955% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 88.84%. Comparing base (c5e0015) to head (892f19c).

Files with missing lines Patch % Lines
src/cl_sii/cte/parsers.py 73.33% 4 Missing and 8 partials ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #890      +/-   ##
===========================================
- Coverage    88.97%   88.84%   -0.14%     
===========================================
  Files           40       40              
  Lines         3565     3631      +66     
  Branches       355      366      +11     
===========================================
+ Hits          3172     3226      +54     
- Misses         240      244       +4     
- Partials       153      161       +8     

☔ 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.

@svillegas-cdd svillegas-cdd force-pushed the task/sc-16534--parse-datos-del-contribuyente branch from 5cc7d54 to a82e219 Compare September 12, 2025 15:53
- Implemented `parse_taxpayer_data` to parse taxpayer data from CTE HTML.
- Added `TaxpayerData` and `LastFiledDocument` data models.
- Created tests to validate parser functionality with sample HTML input.

Ref: https://app.shortcut.com/cordada/story/16534/
@svillegas-cdd svillegas-cdd force-pushed the task/sc-16534--parse-datos-del-contribuyente branch from a82e219 to 892f19c Compare September 12, 2025 16:07
Copy link

sonarqubecloud bot commented Sep 12, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
49.4% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant