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

test_fishery_dynamics.py creating test file using pytest for fishery_dynamics.py file #107

Merged
merged 23 commits into from
Dec 10, 2024

Conversation

Jaydon2005
Copy link
Contributor

@Jaydon2005 Jaydon2005 commented Aug 16, 2024

This Branch is branching off of the dev branch testing the fishery dynamics file using pytest , changed from fishery_dynamics to from msdbook.fishery_dynamics and has pyproject.toml with mock & pytest. Added from matplotlib.colorbar import colorbar

@Jaydon2005 Jaydon2005 requested a review from erexer August 16, 2024 17:57
@Jaydon2005 Jaydon2005 marked this pull request as ready for review August 16, 2024 17:57
Fixing corrected imports, import from msdbook getting the fishery_dynamics module, to ensure that the fishery_dynamics.py file is indeed in the msdbook package directory. Fixing the color bars, checking color bars in matpoltib used plt.colorbar replaced fig.colorbar.
Removed the comment next to the import message. Should fix the error module 'matplotlib.pyplot' has no attribute 'Colorbar'.
@erexer erexer changed the title test_fishery_dynamics.py creating test file using pytest for fishery_dynamcis.py file test_fishery_dynamics.py creating test file using pytest for fishery_dynamics.py file Aug 27, 2024
msdbook/tests/test_fishery_dynamics.py Outdated Show resolved Hide resolved
msdbook/tests/test_fishery_dynamics.py Outdated Show resolved Hide resolved
msdbook/tests/test_fishery_dynamics.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@erexer erexer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@erexer erexer marked this pull request as draft November 6, 2024 20:46
@erexer
Copy link
Collaborator

erexer commented Nov 6, 2024

converted to draft since checks are failing

Copy link

codecov bot commented Dec 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.49%. Comparing base (50763eb) to head (7890c0a).
Report is 84 commits behind head on dev.

Additional details and impacted files
@@            Coverage Diff             @@
##             dev     #107       +/-   ##
==========================================
+ Coverage   2.07%   45.49%   +43.41%     
==========================================
  Files          9       11        +2     
  Lines        529      633      +104     
==========================================
+ Hits          11      288      +277     
+ Misses       518      345      -173     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Jaydon2005 Jaydon2005 marked this pull request as ready for review December 9, 2024 19:43
@Jaydon2005 Jaydon2005 requested a review from thurber December 10, 2024 16:11
Copy link
Contributor

@thurber thurber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks!

@Jaydon2005
Copy link
Contributor Author

looks good, thanks!

Thank you, Travis! I will merge this PR.

@Jaydon2005 Jaydon2005 merged commit 65eda50 into dev Dec 10, 2024
6 checks passed
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.

3 participants