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 Animation #25409

Merged
merged 17 commits into from
Feb 3, 2023
Merged

Tests: Unit Tests for Animation #25409

merged 17 commits into from
Feb 3, 2023

Conversation

epreston
Copy link
Contributor

@epreston epreston commented Feb 2, 2023

Related issue: none.

Description

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

Explicit block scope for variables used in tests.
Stub missing member tests.
AnimationClip can be instanciated
AnimationClip can be named
Stub missing member tests.
AnimationMixer extends from EventDispatcher
Stub missing member tests.
AnimationObjectGroup can be instanciated
AnimationObjectGroup.isAnimationObjectGroup should be true
Explicit block scope for variables used in tests.
Stub missing member tests.
Stub missing member tests.
Stub missing member tests.
Explicit block scope for variables used in tests.
Stub missing member tests.
BooleanKeyframeTrack extends from KeyframeTrack
ColorKeyframeTrack extends from KeyframeTrack
NumberKeyframeTrack extends from KeyframeTrack
QuaternionKeyframeTrack extends from KeyframeTrack
StringKeyframeTrack extends from KeyframeTrack
VectorKeyframeTrack extends from KeyframeTrack
Separate the first import to clearly mark the test subject.
Explicit block scope for variables used in tests.
Split setEffectiveWeight into 3 functional tests and resolve the variable block scope issue at the same time.  At the moment, trying to minimize changes to working tests.
@mrdoob mrdoob added this to the r150 milestone Feb 3, 2023
@mrdoob mrdoob merged commit 672b8f2 into mrdoob:dev Feb 3, 2023
@epreston epreston deleted the tests-animation-cleanup branch February 3, 2023 02:48
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