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

Add a test suite for Object #43583

Merged
merged 1 commit into from
Nov 20, 2020
Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Nov 15, 2020

It's quite sparse currently. Feel free to suggest additional tests to add.

@Xrayez
Copy link
Contributor

Xrayez commented Nov 19, 2020

Worth adding object.is_class("Object") (should return true) along with existing get_class. 🙂

@akien-mga
Copy link
Member

akien-mga commented Nov 20, 2020

I guess let's merge once @Xrayez's suggestion is amended in, so that we have the base test available for other to add more tests when they fix bugs in Object.

We should likely open an issue as a reminder that the test needs more content to cover the existing API (or leave the Object entry unchecked in #43440 with a call for more test cases).

@akien-mga akien-mga merged commit 34895e7 into godotengine:master Nov 20, 2020
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the test-add-object branch March 31, 2021 09:44
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.

3 participants