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

Tests: Unit Tests for Loaders #25391

Merged
merged 16 commits into from
Feb 1, 2023
Merged

Tests: Unit Tests for Loaders #25391

merged 16 commits into from
Feb 1, 2023

Conversation

epreston
Copy link
Contributor

@epreston epreston commented Feb 1, 2023

Related issue: none.

Description

This cleans up the unit tests for Loaders.
Adds missing unit test file, fills in some unit tests, populates the missing member tests with stubs. Removes tests for base class methods which are not redfined in the child class.

Add ImageBitmapLoader unit tests.
Loader defines a default manager if not supplied in constructor.
Loader defines crossOrigin.
Loader defines withCredentials.
Loader defines path.
Loader defines resourcePath.
Loader defines requestHeader.
Stub missing member tests.
AnimationLoader extends from Loader.
AudioLoader extends from Loader.
BufferGeometryLoader extends from Loader.
Cache defines enabled.
Cache defines files.
CompressedTextureLoader extends from Loader.
Remove test for base class method which is not redfined in this class.
CubeTextureLoader extends from Loader.
Remove test for base class method which is not redfined in this class.
DataTextureLoader extends from Loader.
FileLoader extends from Loader.
Remove tests for base class methods which are not redfined in this class.
ImageLoader extends from Loader.
Remove tests for base class methods which are not redfined in this class.
Stub missing member test.
Stub missing member tests.
MaterialLoader extends from Loader.
MaterialLoader defines textures.
Stub missing member test.
ObjectLoader extends from Loader.
Stub missing member tests.
Remove tests for base class methods which are not redfined in this class.
TextureLoader extends from Loader.
Remove tests for base class methods which are not redfined in this class.
@mrdoob mrdoob added this to the r150 milestone Feb 1, 2023
@mrdoob mrdoob merged commit 34640a5 into mrdoob:dev Feb 1, 2023
@epreston
Copy link
Contributor Author

epreston commented Feb 1, 2023

Not getting any hearts lately. haha ;-)

@epreston epreston deleted the tests-loaders-cleanup branch February 1, 2023 04:41
@mrdoob
Copy link
Owner

mrdoob commented Feb 1, 2023

Ops! Fixed 😅

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.

2 participants