Skip to content

Conversation

@svillegas-cdd
Copy link
Contributor

  • Improved exception message at .DteXmlData.validate_referencias_numero_linea_ref_order
  • Included Expected an actual value
  • Added list of all actual values or DteXmlData.referencias in message

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

@svillegas-cdd svillegas-cdd added the task Task or chore label Oct 31, 2023
@svillegas-cdd svillegas-cdd requested review from a team and jtrobles-cdd October 31, 2023 15:32
@svillegas-cdd svillegas-cdd self-assigned this Oct 31, 2023
@shortcut-integration
Copy link

@codecov
Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #557 (a5ca833) into develop (1f04133) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #557   +/-   ##
========================================
  Coverage    86.98%   86.98%           
========================================
  Files           34       34           
  Lines         3013     3013           
  Branches       377      378    +1     
========================================
  Hits          2621     2621           
  Misses         250      250           
  Partials       142      142           
Files Coverage Δ
src/cl_sii/dte/data_models.py 96.92% <100.00%> (ø)

* Improved exception message at `.DteXmlData.validate_referencias_numero_linea_ref_order`
* Included Expected an actual value
* Added list of all actual values or `DteXmlData.referencias` in message

Ref: https://app.shortcut.com/cordada/story/1493/
@svillegas-cdd svillegas-cdd force-pushed the task/sc-1493-improve-error-message-of-cl-sii-dte-data branch from e50a6b5 to a5ca833 Compare November 8, 2023 18:54
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jtrobles-cdd jtrobles-cdd changed the title Add more info in validation message dte: Improve validation error message in DteXmlData.referencias Nov 8, 2023
@svillegas-cdd svillegas-cdd merged commit ba2ac3a into develop Nov 8, 2023
@svillegas-cdd svillegas-cdd deleted the task/sc-1493-improve-error-message-of-cl-sii-dte-data branch November 8, 2023 21:12
@svillegas-cdd svillegas-cdd mentioned this pull request Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants