Code review comment: Horrible keyboardss on the 2018 models :)

Code under review:
public void testMultipleInstancesWithDifferentChanges() {
        DeploymentContext i1 = tester.newDeploymentContext("t", "a", "i1");
        DeploymentContext i2 = tester.newDeploymentContext("t", "a", "i2");
        DeploymentContext i3 = tester.newDeploymentContext("t", "a", "i3");
        DeploymentContext i4 = tester.newDeploymentContext("t", "a", "i4");
        ApplicationPackage applicationPackage = ApplicationPackageBuilder
                .fromDeploymentXml("<deployment version='1'>\n" +
                                   "  <parallel>\n" +
                                   "    <instance id='i1'>\n" +
                                   "      <prod>\n" +
                                   "        <region>us-east-3</region>\n" +
                                   "        <delay hours='6' />\n" +
                                   "      </prod>\n" +
                                   "    </instance>\n" +
                                   "    <instance id='i2'>\n" +
                                   "      <prod>\n" +
                                   "        <region>us-east-3</region>\n" +
                                   "      </prod>\n" +
                                   "    </instance>\n" +
                                   "  </parallel>\n" +
                                   "  <instance id='i3'>\n" +
                                   "    <prod>\n" +
                                   "      <region>us-east-3</region>\n" +
                                   "        <delay hours='18' />\n" +
                                   "      <test>us-east-3</test>\n" +
                                   "    </prod>\n" +
                                   "  </instance>\n" +
                                   "  <instance id='i4'>\n" +
                                   "    <test />\n" +
                                   "    <staging />\n" +
                                   "    <prod>\n" +
                                   "      <region>us-east-3</region>\n" +
                                   "    </prod>\n" +
                                   "  </instance>\n" +
                                   "</deployment>\n");

        
        i1.submit(applicationPackage);
        Optional<ApplicationVersion> v0 = i1.lastSubmission();
        tester.outstandingChangeDeployer().run();
        assertEquals(v0, i1.instance().change().application());
        assertEquals(v0, i2.instance().change().application());
        assertEquals(Optional.empty(), i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        i4.runJob(systemTest).runJob(stagingTest);
        i1.runJob(productionUsEast3);
        i2.runJob(productionUsEast3);

        
        tester.outstandingChangeDeployer().run();
        assertEquals(v0, i1.instance().latestDeployed());
        assertEquals(v0, i2.instance().latestDeployed());
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(Optional.empty(), i2.instance().change().application());
        assertEquals(Optional.empty(), i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        tester.clock().advance(Duration.ofHours(6));
        tester.outstandingChangeDeployer().run();
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(Optional.empty(), i2.instance().change().application());
        assertEquals(v0, i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        i3.runJob(productionUsEast3); 
        tester.clock().advance(Duration.ofHours(12));
        i1.submit(applicationPackage);
        Optional<ApplicationVersion> v1 = i1.lastSubmission();
        i4.runJob(systemTest).runJob(stagingTest);
        i1.runJob(productionUsEast3); 
        i2.runJob(productionUsEast3); 
        assertEquals(v1, i1.instance().latestDeployed());
        assertEquals(v1, i2.instance().latestDeployed());
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(Optional.empty(), i2.instance().change().application());
        assertEquals(v0, i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        tester.clock().advance(Duration.ofHours(3));
        i1.submit(applicationPackage);
        Optional<ApplicationVersion> v2 = i1.lastSubmission();
        i4.runJob(systemTest).runJob(stagingTest);
        i1.runJob(productionUsEast3); 
        tester.clock().advance(Duration.ofHours(3));

        
        
        
        assertEquals(v0, i3.instance().change().application());

        
        i3.runJob(testUsEast3);
        assertEquals(Optional.empty(), i3.instance().change().application());
        tester.outstandingChangeDeployer().run();
        assertEquals(v2, i1.instance().latestDeployed());
        assertEquals(v1, i2.instance().latestDeployed());
        assertEquals(v0, i3.instance().latestDeployed());
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(v2, i2.instance().change().application());
        assertEquals(v1, i3.instance().change().application());
        assertEquals(v0, i4.instance().change().application());
    }


Revised code:
public void testMultipleInstancesWithDifferentChanges() {
        DeploymentContext i1 = tester.newDeploymentContext("t", "a", "i1");
        DeploymentContext i2 = tester.newDeploymentContext("t", "a", "i2");
        DeploymentContext i3 = tester.newDeploymentContext("t", "a", "i3");
        DeploymentContext i4 = tester.newDeploymentContext("t", "a", "i4");
        ApplicationPackage applicationPackage = ApplicationPackageBuilder
                .fromDeploymentXml("<deployment version='1'>\n" +
                                   "  <parallel>\n" +
                                   "    <instance id='i1'>\n" +
                                   "      <prod>\n" +
                                   "        <region>us-east-3</region>\n" +
                                   "        <delay hours='6' />\n" +
                                   "      </prod>\n" +
                                   "    </instance>\n" +
                                   "    <instance id='i2'>\n" +
                                   "      <prod>\n" +
                                   "        <region>us-east-3</region>\n" +
                                   "      </prod>\n" +
                                   "    </instance>\n" +
                                   "  </parallel>\n" +
                                   "  <instance id='i3'>\n" +
                                   "    <prod>\n" +
                                   "      <region>us-east-3</region>\n" +
                                   "        <delay hours='18' />\n" +
                                   "      <test>us-east-3</test>\n" +
                                   "    </prod>\n" +
                                   "  </instance>\n" +
                                   "  <instance id='i4'>\n" +
                                   "    <test />\n" +
                                   "    <staging />\n" +
                                   "    <prod>\n" +
                                   "      <region>us-east-3</region>\n" +
                                   "    </prod>\n" +
                                   "  </instance>\n" +
                                   "</deployment>\n");

        
        i1.submit(applicationPackage);
        Optional<ApplicationVersion> v0 = i1.lastSubmission();
        tester.outstandingChangeDeployer().run();
        assertEquals(v0, i1.instance().change().application());
        assertEquals(v0, i2.instance().change().application());
        assertEquals(Optional.empty(), i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        i4.runJob(systemTest).runJob(stagingTest);
        i1.runJob(productionUsEast3);
        i2.runJob(productionUsEast3);

        
        tester.outstandingChangeDeployer().run();
        assertEquals(v0, i1.instance().latestDeployed());
        assertEquals(v0, i2.instance().latestDeployed());
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(Optional.empty(), i2.instance().change().application());
        assertEquals(Optional.empty(), i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        tester.clock().advance(Duration.ofHours(6));
        tester.outstandingChangeDeployer().run();
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(Optional.empty(), i2.instance().change().application());
        assertEquals(v0, i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        i3.runJob(productionUsEast3); 
        tester.clock().advance(Duration.ofHours(12));
        i1.submit(applicationPackage);
        Optional<ApplicationVersion> v1 = i1.lastSubmission();
        i4.runJob(systemTest).runJob(stagingTest);
        i1.runJob(productionUsEast3); 
        i2.runJob(productionUsEast3); 
        assertEquals(v1, i1.instance().latestDeployed());
        assertEquals(v1, i2.instance().latestDeployed());
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(Optional.empty(), i2.instance().change().application());
        assertEquals(v0, i3.instance().change().application());
        assertEquals(Optional.empty(), i4.instance().change().application());

        
        tester.clock().advance(Duration.ofHours(3));
        i1.submit(applicationPackage);
        Optional<ApplicationVersion> v2 = i1.lastSubmission();
        i4.runJob(systemTest).runJob(stagingTest);
        i1.runJob(productionUsEast3); 
        tester.clock().advance(Duration.ofHours(3));

        
        
        
        assertEquals(v0, i3.instance().change().application());

        
        i3.runJob(testUsEast3);
        assertEquals(Optional.empty(), i3.instance().change().application());
        tester.outstandingChangeDeployer().run();
        assertEquals(v2, i1.instance().latestDeployed());
        assertEquals(v1, i2.instance().latestDeployed());
        assertEquals(v0, i3.instance().latestDeployed());
        assertEquals(Optional.empty(), i1.instance().change().application());
        assertEquals(v2, i2.instance().change().application());
        assertEquals(v1, i3.instance().change().application());
        assertEquals(v0, i4.instance().change().application());
    }