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

Set up XUnit #20

Closed
AdamNorberg opened this issue May 1, 2021 · 1 comment · Fixed by #31
Closed

Set up XUnit #20

AdamNorberg opened this issue May 1, 2021 · 1 comment · Fixed by #31
Assignees

Comments

@AdamNorberg
Copy link
Collaborator

AdamNorberg commented May 1, 2021

It looks like we're okay with XUnit for testing, so it's up to me to set it up. I'll consider it "set up" when I can run a "hello world" test from MSBuild, since that's what the Godot directions prefer. After that, I'll set up a GitHub Action to automatically build and run all unit tests on every pull request or commit to master. Then, well, I've got some unit tests to write!

@AdamNorberg AdamNorberg self-assigned this May 1, 2021
@AdamNorberg
Copy link
Collaborator Author

It turns out the setup required is negligible - simply declare the dependency and NuGet does the rest of the work, and my IDE sets up the dependency for me in the .csproj file. It just took me a while to learn that unit tests have to be in a separate "project" from the rest of the code.

That, however, poses a problem, because the current layout is not "multiple-projects"-flavored. #23 describes what needs to be done to make this work.

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 a pull request may close this issue.

1 participant