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

Include boost-ext/ut micro unit test library #139

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

jt-traub
Copy link
Contributor

@jt-traub jt-traub commented Oct 9, 2023

This adds the Boost μt unit testing framework to the codebase.

This is a header-only freely redistributable framework which encapsulates most different unit testing styles (suites, rspec, gherkin). For the tests here, I am using the most simple 'suites' style, but we could use other styles as well.

The overall engine however, is not very amenable to unit testing in it's current state. I have provided some examples of testing various functions within the faction and unit classes, in order to provide some guidance on how tests might be written, but writing thorough tests will be hard without refactoring a lot of the current engine framework. That is something I believe should be undertaken over time however.

Makefile Show resolved Hide resolved
Makefile Show resolved Hide resolved
astring.cpp Show resolved Hide resolved
@jt-traub jt-traub marked this pull request as ready for review October 9, 2023 01:24
@jt-traub jt-traub force-pushed the jt-add-boost-ut branch 2 times, most recently from fcf3cc3 to a814bac Compare October 9, 2023 02:10
unit.cpp Show resolved Hide resolved
Begin process of wiring up creating and running unit test.
@valdisz
Copy link
Contributor

valdisz commented Oct 10, 2023

LGTM, I will push the merge button

@valdisz valdisz merged commit 1194a91 into Atlantis-PBEM:neworigins-v6 Oct 10, 2023
5 checks passed
@jt-traub jt-traub deleted the jt-add-boost-ut branch October 10, 2023 06:20
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.

3 participants