Fix mdp parameter handling in mdrun tests
authorejjordan <e.jjordan12@gmail.com>
Thu, 7 Nov 2019 15:10:34 +0000 (16:10 +0100)
committerPaul Bauer <paul.bauer.q@gmail.com>
Sat, 9 Nov 2019 09:55:06 +0000 (10:55 +0100)
commitd6aafbab0ec598e36b5c9e6020ab9d34b3291612
tree63643d13fa0f8cca802b2f775fccb58d00189cf2
parentb2c4a71ba09d759e9be7ff503a383f9904050942
Fix mdp parameter handling in mdrun tests

Some mdp options that we intended to be set were in fact not being set.
In the two cases where mdp options were not in fact being set the default
option was desired, so this patch does not change any behavior. A note is
added to the mdp options handling code. A better long term solution might
look like popping options off the parameter map and throw if there are
options left after all defaults have been printed. This however may not
be worth the effort as the mdp options handling could change significantly
in the short term.

Change-Id: Ied6580683c91938e577861be43841e7dc3dcd8cd
src/programs/mdrun/tests/normalmodes.cpp
src/programs/mdrun/tests/simple_mdrun.cpp
src/testutils/simulationdatabase.cpp