update regression test files for old intrinsic models
Merged
requested to merge 516-regression-tests-for-correctly-loading-old-intrinsic-models into master
All threads resolved!
I added regression tests a while ago testing if older pet files that only saved parameters of one model were still correctly loaded, but probably while updating the version of the pet file the test files also got the new syntax, effectively making the tests pointless. So I reversed the changes and remade the pet files in the old version and syntax.
Reviewer Checklist
General code quality
-
naming conventions are met (see .clang-tidy for detailed information) -
no static analyzer warnings in new code parts (e.g., use clang-tidy for checking)
General usability
-
old versions of pet-files are still loadable -
documentation updated (if applicable)
Only if changes in UI
-
new elements are also saved and loaded from pet-file -
check if tab order is still correct -
all new SpinBoxes are promoted -
new keybindings added to Petrack::keyBindings()
Closes #516 (closed)
Edited by Tobias Arens
Merge request reports
Activity
assigned to @l.dressen
added 12 commits
-
0bd871b4...0ffe809b - 10 commits from branch
master
- 9fc16cd5 - update test files
- c9e11831 - Merge remote-tracking branch...
-
0bd871b4...0ffe809b - 10 commits from branch
requested review from @t.arens
- Resolved by Tobias Arens
Could you fill out the description of the merge request to specify what and why the files were changed e.g. why the update was necessary?
added 7 commits
-
c9e11831...116cf61c - 6 commits from branch
master
- 402d6f04 - update test files
-
c9e11831...116cf61c - 6 commits from branch
mentioned in commit 11887358
Please register or sign in to reply