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

PR: Add tests to validate results from the HELP3O extension against a genuine test case #46

Merged
merged 7 commits into from Nov 27, 2018

Conversation

jnsebgosselin
Copy link
Member

@jnsebgosselin jnsebgosselin commented Nov 27, 2018

After my failed attempt at fixing Issue #24 in PR #45, I realized how important it was that I implemented tests to validate the results from the HELP3O extension against a genuine test case.

This PR does that by adding tests that runs the HELP3O extension against a test case that was originally created in 1997 and distributed alongside the official FORTRAN source code of HELP version 3.07.

The files from this original test case are also added to this repo in this PR.

Test that the HELP3O extension run and create an output file as
expected.
Run HELP for a single cell using the RCA test case and assert that the
result are as expected.
@jnsebgosselin jnsebgosselin added tag: HELP modifs When a PR or Issue inply changes to the FORTRAN base code of HELP type: maintenance Release, CI, tests, etc. labels Nov 27, 2018
@jnsebgosselin jnsebgosselin added this to the 0.1.1 milestone Nov 27, 2018
@jnsebgosselin jnsebgosselin self-assigned this Nov 27, 2018
@pep8speaks
Copy link

pep8speaks commented Nov 27, 2018

Hello @jnsebgosselin! Thanks for updating the PR.

Comment last updated on November 27, 2018 at 06:13 Hours UTC

@jnsebgosselin jnsebgosselin changed the title PR: Add tests to validate results from the HELP3O extension PR: Add tests to validate results from the HELP3O extension against a genuine test case Nov 27, 2018
@jnsebgosselin jnsebgosselin merged commit 97c5ba4 into master Nov 27, 2018
@jnsebgosselin jnsebgosselin deleted the add_test_HELP3O branch November 27, 2018 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag: HELP modifs When a PR or Issue inply changes to the FORTRAN base code of HELP type: maintenance Release, CI, tests, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants