-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add unit tests #21
Merged
Merged
Add unit tests #21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Setup pytest with pytest.ini and .coveragerc Add conftest.py for fixtures Add __init__.py files for correct python directories search Add tests directory and tests/resources Add unit tests for tracks_count.py and artist_aggregate.py
Some fixes in previous tests Add files systems path Add test for exception raising
Add unit tests for Album, Artist and Track classes Add unit test for get_artist_response_template function from stats_utils.py Add json file to resources and fixture to get it
Add unit tests for stats_utils.py functions Add some fixtures Add resource files Refactor file_path for stats files and stats update methods
Add unit tests for stats_update_main function in stats_update.py Add and change some resource files Fix directories imports Rename main function and parameters Fix tracks_count.py for MacOS
Add unit tests for stats_update_main function in stats_update.py for invalid and empty artist ids Refactor and extend artist id in tests Extend ResponseObject class to util_classes.py Add doc-lines for tests and functions
Add unit tests for stats_from_files_main function in stats_from_files.py Refactor stats_from_files main function and extend parameters Fix imports paths Add some resource files Fix test artists files directory and paths
Add yaml workflow for unit tests Update README.md with tests and coverage badges Add requirements.txt for projects libraries
Remove unnecessary fixture for headers taking
Remove imports for auth token and client headers from functions for testing setup Extend headers, tokens and extensions lines with parameters Change unit tests args for headers and extensions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Setup pytest for unit tests
Add unit tests for artist_aggregate, classes, stats utils, stats update, stats from files
Add unit tests automation