Skip to content

Add Godot-requiring unit test support #4963

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.

Sign up for GitHub

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

Jump to bottom
Closed
hhyyrylainen opened this issue Mar 25, 2024 · 4 comments
Closed

Add Godot-requiring unit test support #4963

hhyyrylainen opened this issue Mar 25, 2024 · 4 comments
Assignees
@hhyyrylainen
Labels
C# impeded intermediate programming
Milestone
Release 0.8.2

Comments

@hhyyrylainen
Copy link
Member

hhyyrylainen commented Mar 25, 2024 •
edited
Loading

to fully compliment how Thrive can be automatically tested,

For potential approaches we can check what others have done: godotengine/godot-proposals#432

Before closing this issue the CI should be made to run the tests.
@hhyyrylainen hhyyrylainen added difficult programming engine possibly engine related issue C# labels Mar 25, 2024
@hhyyrylainen hhyyrylainen changed the title Add Godot-requiring unit tests Add Godot-requiring unit test support Mar 25, 2024
@hhyyrylainen hhyyrylainen mentioned this issue Mar 25, 2024
Closed
2 tasks
@hhyyrylainen
Copy link
Member Author

Some unit tests could be made just by making the settings available when testing.

@hhyyrylainen hhyyrylainen moved this to Backburner in Thrive Planning Aug 7, 2024
@hhyyrylainen hhyyrylainen moved this from Backburner to In progress in Thrive Planning Nov 27, 2024
@hhyyrylainen hhyyrylainen self-assigned this Nov 27, 2024
@hhyyrylainen hhyyrylainen added this to the Release 0.8.0 milestone Nov 27, 2024
@hhyyrylainen hhyyrylainen added impeded and removed engine possibly engine related issue labels Nov 29, 2024
@hhyyrylainen
Copy link
Member Author

This is done but the tests are disabled for now due to a bug in the testing library: MikeSchulze/gdUnit4Net#166

@hhyyrylainen hhyyrylainen mentioned this issue Nov 29, 2024
Open
@hhyyrylainen hhyyrylainen mentioned this issue Dec 10, 2024
Closed
@hhyyrylainen
Copy link
Member Author

This should now just be waiting for a new release of gdUnit.

@hhyyrylainen hhyyrylainen moved this from In progress to Features for next release in Thrive Planning Feb 18, 2025
@hhyyrylainen
Copy link
Member Author

There's actually a separate issue about enabling this: #5737

@github-project-automation github-project-automation bot moved this from Features for next release to Done in Thrive Planning Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hhyyrylainen hhyyrylainen

Labels
C# impeded intermediate programming
Projects
Thrive Planning
Status: Done
Milestone
Release 0.8.2
Development

No branches or pull requests
1 participant
@hhyyrylainen