Bugfix in "TargetChangerController": "useStaticTargetForAgent()" did not...

Bugfix in "TargetChangerController": "useStaticTargetForAgent()" did not update agent's "nextTargetListIndex" properly

VadereSimulator/src/org/vadere/simulator/control/TargetChangerController.java

@@ -160,9 +160,7 @@ public class TargetChangerController {
     }
 
     private void useStaticTargetForAgent(Agent agent) {
-        LinkedList<Integer> newTarget = new LinkedList<>();
-        newTarget.add(targetChanger.getAttributes().getNextTarget());
-        agent.setTargets(newTarget);
+        agent.setSingleTarget(targetChanger.getAttributes().getNextTarget(), false);
     }
 
     private void notifyListenersTargetChangerAreaReached(final Agent agent) {

VadereSimulator/tests/org/vadere/simulator/control/TargetChangerControllerTest.java

@@ -169,6 +169,32 @@ public class TargetChangerControllerTest {
         assertListContainsSingleTarget(pedestrians.get(1).getTargets(), nextTarget);
     }
 
+    @Test
+    public void updateSetsNextTargetListIndexToZeroOfAffectedPedestrianIfProbabilityIsOne() {
+        LinkedList<Integer> currentTargets = new LinkedList<Integer>(Arrays.asList(1, 2, 3));
+        int nextTargetIndex = 1;
+
+        pedestrians.get(1).setTargets(currentTargets);
+        pedestrians.get(1).setNextTargetListIndex(nextTargetIndex);
+
+        int nextTarget = 2;
+        double probability = 1.0;
+
+        AttributesTargetChanger attributesTargetChanger = createAttributesWithFixedRectangle();
+        attributesTargetChanger.setNextTarget(nextTarget);
+        attributesTargetChanger.setProbabilityToChangeTarget(probability);
+
+        TargetChanger targetChanger = new TargetChanger(attributesTargetChanger);
+        TargetChangerController controllerUnderTest = createTargetChangerController(targetChanger);
+
+        assertEquals(nextTargetIndex, pedestrians.get(1).getNextTargetListIndex());
+
+        controllerUnderTest.update(simTimeInSec);
+
+        assertEquals(0, pedestrians.get(1).getNextTargetListIndex());
+        assertListContainsSingleTarget(pedestrians.get(1).getTargets(), nextTarget);
+    }
+
     @Test
     public void updateDoesNotChangeTargetListOfAffectedPedestrianIfProbabilityIsZero() {
         int nextTarget = 2;