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

Unit Tests for ArrayMesh class for Issue #43440 #67800

Merged
merged 1 commit into from
Oct 31, 2022

Conversation

jbcolli2
Copy link
Contributor

A set of unit tests for the ArrayMesh class to contribute to issue #43440. Use the PrimitiveMesh subclasses to generate arrays to test this class.

@Zireael07
Copy link
Contributor

Hopefully this prevents another #64854 in the future...

@jbcolli2
Copy link
Contributor Author

I currently only use PrimitiveMesh arrays to test. Would it be more useful if I also tested with a mesh imported from outside of godot?

Copy link
Member

@clayjohn clayjohn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, it would be nice to have a review from someone more familiar with the testing framework

@akien-mga akien-mga modified the milestones: 4.x, 4.0 Oct 31, 2022
@akien-mga akien-mga merged commit 1211f9d into godotengine:master Oct 31, 2022
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants