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 Cameras #25404

Merged
merged 7 commits into from
Feb 2, 2023
Merged

Tests: Unit Tests for Cameras #25404

merged 7 commits into from
Feb 2, 2023

Conversation

epreston
Copy link
Contributor

@epreston epreston commented Feb 2, 2023

Related issue: none.

Description

This cleans up the unit tests for Cameras.
Fills in some unit tests, populates the missing member tests with stubs. Explicit block scope for variables used in tests.

ArrayCamera extends from PerspectiveCamera
ArrayCamera.isArrayCamera should be true
Stub missing member test.
Camera extends from Object3D
Camera.isCamera should be true
Explicit block scope for variables used in tests.
Stub missing member tests.
CubeCamera extends from Object3D
CubeCamera.type should be CubeCamera
Stub missing member tests.
OrthographicCamera extends from Camera
OrthographicCamera.type should be OrthographicCamera
OrthographicCamera.isOrthographicCamera should be true
Explicit block scope for variables used in tests.
Stub missing member tests.
Tone down previous comments.
PerspectiveCamera extends from Camera
PerspectiveCamera.type should be PerspectiveCamera
PerspectiveCamera.isPerspectiveCamera should be true
Explicit block scope for variables used in tests.
Stub missing member tests.
StereoCamera.type should be StereoCamera
Stub missing member tests.
@mrdoob mrdoob added this to the r150 milestone Feb 2, 2023
@mrdoob mrdoob merged commit 9eaa3f0 into mrdoob:dev Feb 2, 2023
@epreston epreston deleted the tests-cameras-cleanup branch February 2, 2023 12:41
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