diff --git a/src/main/java/jenkins/advancedqueue/priority/PriorityStrategy.java b/src/main/java/jenkins/advancedqueue/priority/PriorityStrategy.java index 882d850b..a4a15934 100644 --- a/src/main/java/jenkins/advancedqueue/priority/PriorityStrategy.java +++ b/src/main/java/jenkins/advancedqueue/priority/PriorityStrategy.java @@ -41,7 +41,7 @@ public abstract class PriorityStrategy implements ExtensionPoint, Describabletrue if the {@link PriorityStrategy} is applicable else false @@ -52,7 +52,7 @@ public abstract class PriorityStrategy implements ExtensionPoint, Describable> list = PriorityStrategy.all(); assertNotNull("DescriptorExtensionList should not be null", list); + // The list.size() method returns 7 because the DescriptorExtensionList for PriorityStrategy contains 7 + // descriptors. This means there are 7 different implementations of the PriorityStrategy class registered in the + // Jenkins instance. + assertEquals(7, list.size()); + } + + @Test + public void testItemTaskIsInstanceOfJob() { + item = new Queue.WaitingItem(Calendar.getInstance(), project, new ArrayList<>()); + assertTrue(item.task instanceof Job); } private static class TestPriorityStrategy extends PriorityStrategy { @@ -114,9 +124,6 @@ public int getNewNumberOfPriorities() { return newNumberOfPriorities; } - /** - * - */ @Override public Descriptor getDescriptor() { return null;