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

Added test_load_basic #2225

Closed
wants to merge 1 commit into from
Closed

Added test_load_basic #2225

wants to merge 1 commit into from

Conversation

dtomci
Copy link

@dtomci dtomci commented Oct 21, 2020

Hello, so here is the test for load_basic. Hopefully its somewhat correct and what you expected.

@MyreMylar
Copy link
Contributor

It should be OK to actually call load_basic() in this test for now, we are discouraging the use of load_basic() in the documentation but for the unit test it makes sense to call it directly. I know load() redirects to load_basic() but that's why we have tests for load() too :)

Other things that could be tried is loading an actual file from the examples data directory e.g.:

load_basic(example_path("data/asprite.bmp"))

and perhaps loading from a file-like object like io.BytesIO?

@illume illume closed this Oct 27, 2020
@illume illume deleted the branch pygame:master October 27, 2020 16:15
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.

None yet

3 participants