Skip to content

Conversation

@moonlight83340
Copy link
Contributor

@moonlight83340 moonlight83340 commented Apr 4, 2025

This commit introduces a GitHub Actions workflow for building and testing the project on Windows using CMake presets and Ninja.

This ensures automated builds and tests for both Debug and Release configurations.

I hope introducing CI in anitomy can help contribution in the future.

This commit introduces a GitHub Actions workflow for building and testing the
project on Windows using CMake presets and Ninja.

This ensures automated builds and tests for both Debug and Release
configurations.

Signed-off-by: Gaetan Perrot <[email protected]>
@moonlight83340 moonlight83340 marked this pull request as ready for review April 4, 2025 16:00
@erengy
Copy link
Owner

erengy commented May 2, 2025

Hey, thanks for the PR.

One thing that'd make automated tests much more useful is to be able to compare the output of data tests. Would it be possible to do that in the workflow?

To elaborate, running test.exe with --test-data argument outputs a bunch of results along with expected values. What I usually do is to manually compare these with previous results to see the impact of changes, like so:

image

I imagine doing this in the workflow would require something like the following:

  • Build previous commit, run tests and save the output to a.txt.
  • Build last commit, run tests and save the output to b.txt.
  • Run sdiff -s a.txt b.txt to show the differences.
  • In case of a PR, posting the diff as a comment would be nice too.

@moonlight83340
Copy link
Contributor Author

Hi,
Yes, I can definitely do that.

I can compare the output between the PR branch and the main branch (develop), and verify if they match. If there are any differences, I can upload the results to the GitHub Actions runner for manual inspection.

One thing to keep in mind: if a user adds new tests, the output will naturally differ. In that case, someone will need to review the logs manually — that's why uploading them could be helpful.

@moonlight83340
Copy link
Contributor Author

I will try to finish this this week or next week.

@erengy
Copy link
Owner

erengy commented May 12, 2025

Thanks! I'll appreciate it if you can do that when you have the time.

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 this pull request may close these issues.

2 participants