diff options
Diffstat (limited to 'controller-server/src/test/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTester.java')
-rw-r--r-- | controller-server/src/test/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTester.java | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTester.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTester.java index 92bf22df535..093c2dc8d50 100644 --- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTester.java +++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTester.java @@ -275,7 +275,7 @@ public class DeploymentTester { assertEquals(job.jobName(), buildJob.jobName()); } if (expectOnlyTheseJobs) - assertEquals(jobs.length, countJobsOf(application)); + assertEquals("Unexpected job queue: " + jobsOf(application), jobs.length, jobsOf(application).size()); deploymentQueue().removeJobs(application.id()); } @@ -286,15 +286,17 @@ public class DeploymentTester { throw new IllegalArgumentException(jobType + " is not scheduled for " + application); } - private int countJobsOf(Application application) { - return (int) deploymentQueue().jobs().stream() - .filter(job -> job.projectId() == application.deploymentJobs().projectId().get()) - .count(); + private List<JobType> jobsOf(Application application) { + return deploymentQueue().jobs().stream() + .filter(job -> job.projectId() == application.deploymentJobs().projectId().get()) + .map(buildJob -> JobType.fromJobName(buildJob.jobName())) + .collect(Collectors.toList()); } private void notifyJobCompletion(DeploymentJobs.JobReport report) { clock().advance(Duration.ofMillis(1)); applications().notifyJobCompletion(report); + applications().deploymentTrigger().triggerReadyJobs(); } public static ApplicationPackage applicationPackage(String upgradePolicy) { |