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

Bug in test_nhl_roster.py #794

Open
SDL101 opened this issue May 24, 2024 · 1 comment
Open

Bug in test_nhl_roster.py #794

SDL101 opened this issue May 24, 2024 · 1 comment

Comments

@SDL101
Copy link

SDL101 commented May 24, 2024

Describe the bug
I went ahead and ran the unit tests to verify that they were functioning properly and found that the file test_nhl_roster.py failed one of the tests it was supposed to pass due to improper error handling. The error seems to be that the test expects the method player._retrieve_html_page() to return None, but instead, it's returning a list containing an HTML paragraph element ([

]). This mismatch between expected and actual outputs leads to the assertion failure.

To Reproduce
run the test_nhl_roster.py file.

Expected behavior
I expected to see both tests pass.

Desktop (please complete the following information):

  • OS: Windows 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz 2.42 GHz
  • Sportsipy Version: 0.6.0
@davidjkrause
Copy link

As with most recent issues here, this should be resolved by the fork at https://github.com/davidjkrause/sportsipy

There was a fair amount of work done on that fork to get every base test to pass, since so many were failing when I started using the package.

See my similar comment on most of the recent issues, this particular repo seems no longer maintained.

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

No branches or pull requests

2 participants