Better TravisCI output #287
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Travis CI should run a single script that runs individual tests - this also lets developers run a single script and get easy-to-parse results.
POP now restores ALK and ALK_ALT_CO2, and that is reflected in an example input file.
The --no_pause option for the build tests (bld_lib.py and bld_exe.py) means those scripts run without the "Press [return] to continue..." prompt between compilers. This is great for automated testing, and run_test_suite.sh uses this new flag. Also added a "make clean" test and "./bld_lib --no_pause" prior to the "./bld_exe --no_pause" test.
Expect test failures since sphinx is not being loaded into Travis environment
Need sphinx, sphinx_rtd_theme (specifically 0.2.5b1, since that's what we are currently using), and sphinxcontrib-bibtex
Also updated .travis.yml to pip install -r docs/py_requirements.txt and updated run_test_suite.sh to only run Fortran tests if the executable builds correctly.
If bld_lib.py or bld_exe.py fails, then marbl_testing_class.py returns a status of 1. (Similarly, if one of the stand-alone tests fail, the python returns 1).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I created a bash script
MARBL_tools/run_test_suite.shthat runs 15 different tests (stand-alone python likeyaml_to_json.pyas well as python scripts that build / launch the Fortran stand-alone driver likebld_exe.pyandinit.py). It provides a nice summary at the end of the test:The
.travis.ymlfile now has Travis CI run two distinct tests: first it builds the documentation, and then it runsrun_test_suite.sh.