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

Add system level tests for exe #166

Open
cmannett85 opened this issue Feb 3, 2021 · 0 comments
Open

Add system level tests for exe #166

cmannett85 opened this issue Feb 3, 2021 · 0 comments
Assignees
Labels
ci Affects continuous integration enhancement New feature or request

Comments

@cmannett85
Copy link
Owner

The unit tests do not cover any code in the main target, for obvious reasons. This means we need system level tests to be added to the code base, which are ran by the pr_checker action. This is how #165 has escaped our attention.

An initial obvious approach is to provide a python script that execute the program with a known set of inputs (test/programs/*) and test the output against the expected.

A better alternative would be to change the main() signature if doing a test build (should change the unit_test.cmake file in this case...), expose it publicly by a test-build-only header, and then call it from a normal test case whilst hijacking std::cin and std::cout (or providing alternatives to it?) .

@cmannett85 cmannett85 added enhancement New feature or request ci Affects continuous integration labels Feb 3, 2021
@cmannett85 cmannett85 self-assigned this Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Affects continuous integration enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant