Skip to content
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

Testing Revamp #64

Open
DMcP89 opened this issue May 22, 2024 · 0 comments
Open

Testing Revamp #64

DMcP89 opened this issue May 22, 2024 · 0 comments

Comments

@DMcP89
Copy link
Collaborator

DMcP89 commented May 22, 2024

The current unit tests are outdated and need a revamp.

  • Test should mock calls to Yahoo API instead of making requests, end-to-end testing with real API requests should be handled by integration tests.
  • Tests should not rely on credential files such as OAuth1.json or OAuth2.yaml
  • Ensure code is fully covered

This should wait for #63 to be complete before implementing

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

No branches or pull requests

1 participant