Skip to content

added few unit tests #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

pallasite99
Copy link

@pallasite99 pallasite99 commented Feb 9, 2025

Add Unit Tests in DukascopyFxService

Description

This PR addresses some new tests in DukascopyFxService and improves handling.

Changes Made:

  • Improved Debugging: Added log messages to track API requests and debugging failed tests.
  • Updated Assertion Method: Replaced assert_matches! with matches! to resolve unstable feature errors.

Additional Fixes:

  • Refactored Import: Used std::matches instead of assert_matches!.
  • Ensured API Calls Are Logged: Added debug statements to track API URLs.

How to Test

Run the updated test suite using:

cargo test -- --nocapture

This will allow you to see detailed output and debug any further failures.

Screenshots / Logs (if applicable)

(Attach relevant screenshots or logs for debugging)

Checklist

  • Code compiles successfully
  • All tests pass
  • Debug messages added for API calls
  • Weekend handling logic verified

Related Issues

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

Successfully merging this pull request may close these issues.

Add More Unit Tests
1 participant