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

Enable running unit-tests in GitHub workflow #1871

Merged
merged 8 commits into from
May 1, 2024

Conversation

smilediver
Copy link
Contributor

Trying to enable running unit-tests in GitHub workflow.

@smilediver
Copy link
Contributor Author

Windows is hitting #1872 issue.

@halx99 Can you help me in setting up running on macOS? I can't figure out how to insert arch for the command line here: https://github.com/smilediver/axmol/blob/ade1eba39c25f3d1a3909019db4fb9cdaf3067ba/.github/workflows/build.yml#L124

And I also have no idea why Linux fails. I don't know if the path to exe is wrong or something else...

@halx99
Copy link
Collaborator

halx99 commented May 1, 2024

sure

@halx99
Copy link
Collaborator

halx99 commented May 1, 2024

The linux will crash due to initialize FileUtilsLinux too early before entering main entrypoint

@halx99
Copy link
Collaborator

halx99 commented May 1, 2024

does the doctest support run tests in main function?

@halx99
Copy link
Collaborator

halx99 commented May 1, 2024

image

@halx99 halx99 added this to the 2.1.3 milestone May 1, 2024
@halx99 halx99 added the enhancement New feature or request label May 1, 2024
@halx99 halx99 marked this pull request as ready for review May 1, 2024 13:36
@halx99 halx99 merged commit 561b8c2 into axmolengine:dev May 1, 2024
14 of 15 checks passed
@smilediver
Copy link
Contributor Author

Thanks for getting this to work!

@smilediver smilediver deleted the unit-tests/enable branch May 1, 2024 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants