Add tests for cosmetic objects #29

Luc1412 · 2024-10-14T13:43:57Z

Summary

This pull request add tests for all cosmetic objects.

If code changes were made then they have been tested.
- I have updated the documentation to reflect the changes.
This PR fixes an issue.
This PR adds something new (e.g. new method or parameters).
This PR is a breaking change (e.g. methods or parameters removed/renamed)
This PR is not a code change (e.g. documentation, README, ...)

trevorflahardy

Looks good

tests/test_async_methods.py

Format code with black

trevorflahardy

Much cleaner! Good work, this looks good to go 👍 Merge whenever you're ready

Luc1412 added 4 commits October 14, 2024 15:26

Add tests for cosmetic objects

cd3327b

Fix test for track image/album art

ad1ae3f

Remove wrong check for bean test

3618959

Fix tests for track difficulty

a0633bc

Luc1412 requested a review from trevorflahardy October 14, 2024 13:43

Luc1412 self-assigned this Oct 14, 2024

trevorflahardy requested changes Oct 14, 2024

View reviewed changes

tests/test_async_methods.py Outdated Show resolved Hide resolved

tests/test_async_methods.py Outdated Show resolved Hide resolved

Luc1412 added 3 commits October 15, 2024 12:45

Move tests for common objects into their own functions

91ddf54

Move type check into the common test method

42348cf

Reorder tests

856a76b

Format code with black

Luc1412 requested a review from trevorflahardy October 15, 2024 11:04

trevorflahardy mentioned this pull request Oct 15, 2024

Async/Sync Tests Rewrite #30

Draft

trevorflahardy approved these changes Oct 15, 2024

View reviewed changes

Luc1412 merged commit b5ebfa1 into master Oct 16, 2024
8 checks passed