Skip to content

Conversation

@ioan-ghisoi-cko
Copy link
Collaborator

  • Enhanced CardNumber component for safer input handling and updated formatDataForTokenization to robustly manage potential null values in card number and expiry date.
  • Added unit tests for card utilities and context guards; improved HTTP and logger test coverage; updated type definitions.
  • Bumped package version to 1.2.4 in package.json.

… and clarity

- Updated CardNumber component to ensure safe handling of card number input, replacing null values and using a sanitized digit string for validation.
- Enhanced formatDataForTokenization function to handle potential null values for card number and expiry date, improving robustness in data formatting.
@sonarqubecloud
Copy link

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.52%. Comparing base (793ac34) to head (c02faf6).

Files with missing lines Patch % Lines
src/utils/http.tsx 28.57% 1 Missing and 4 partials ⚠️
src/components/CardNumber.tsx 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #50       +/-   ##
===========================================
+ Coverage   68.83%   85.52%   +16.68%     
===========================================
  Files          12       12               
  Lines         215      221        +6     
  Branches       62       68        +6     
===========================================
+ Hits          148      189       +41     
+ Misses         37        3       -34     
+ Partials       30       29        -1     

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

@ioan-ghisoi-cko ioan-ghisoi-cko merged commit 2c9690e into main Aug 13, 2025
6 checks passed
@ioan-ghisoi-cko ioan-ghisoi-cko deleted the fix branch August 13, 2025 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants