Changeset c87d6e for tests/regression/Molecules/ForceAnnealing/testsuite-molecules-force-annealing.at

Timestamp:

Apr 10, 2018, 6:43:12 AM (7 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

AutomationFragmentation_failures, Candidate_v1.6.1, ChemicalSpaceEvaluator, Enhanced_StructuralOptimization_continued, Exclude_Hydrogens_annealWithBondGraph, ForceAnnealing_with_BondGraph, ForceAnnealing_with_BondGraph_contraction-expansion, Gui_displays_atomic_force_velocity, PythonUI_with_named_parameters, StoppableMakroAction, TremoloParser_IncreasedPrecision

Children:

9bb8c8

Parents:

6145577

git-author:

Frederik Heber <frederik.heber@…> (11/04/17 10:00:15)

git-committer:

Frederik Heber <frederik.heber@…> (04/10/18 06:43:12)

Message:

ForceAnnealingAction's Undo removes trajectory item if it was added.

• before Undo would not truly undo the change to the trajectory. Now, we additionally store the trajectory size and may recogniye thereby whether we added a step or not. Note that stepping the world time does not add to the trajectory, as it uses the last present map element closest to the current time step.
• TESTFIX: XFAIL removed from Undo as it is now working. Extended Undo test to cover both cases (ForceAnnealing added new step and did not add new step).

File:

• tests/regression/Molecules/ForceAnnealing/testsuite-molecules-force-annealing.at (modified) (2 diffs)

tests/regression/Molecules/ForceAnnealing/testsuite-molecules-force-annealing.at

@@ -31 +31 @@
 
 
-AT_SETUP([Molecules - Force Annealing with Undo])
+# here, we have only one step in the trajectories
+AT_SETUP([Molecules - Force Annealing with Undo I])
 AT_KEYWORDS([molecules force-annealing undo])
 
@@ -40 +41 @@
 AT_CHECK([../../molecuilder -i $file --select-all-atoms --step-world-time 1 --force-annealing --forces-file test.forces --steps 1 --undo], 0, [stdout], [stderr])
 AT_CHECK([diff $file ${abs_top_srcdir}/tests/regression/Molecules/ForceAnnealing/post/test-undo.conf], 0, [ignore], [ignore])
+
+AT_CLEANUP
+
+
+# here two steps are present already
+AT_SETUP([Molecules - Force Annealing with Undo II])
+AT_KEYWORDS([molecules force-annealing undo])
+
+file=test2.conf
+AT_CHECK([/bin/cp -f ${abs_top_srcdir}/tests/regression/Molecules/ForceAnnealing/pre/test2.conf $file], 0)
+AT_CHECK([/bin/cp -f ${abs_top_srcdir}/tests/regression/Molecules/ForceAnnealing/pre/test.forces .], 0)
+AT_CHECK([chmod u+w $file], 0)
+AT_CHECK([../../molecuilder -i $file --select-all-atoms --set-world-time 1 --force-annealing --forces-file test.forces --steps 1 --undo], 0, [stdout], [stderr])
+AT_CHECK([diff $file ${abs_top_srcdir}/tests/regression/Molecules/ForceAnnealing/post/test-undo2.conf], 0, [ignore], [ignore])
 
 AT_CLEANUP