Use Java 17 language features

src/main/java/jenkins/advancedqueue/JobGroup.java

@@ -167,8 +167,8 @@ public int getPriority() {
     @Deprecated
     @CheckForNull
     public String getView() {
-        if (jobGroupStrategy instanceof ViewBasedJobInclusionStrategy) {
-            return ((ViewBasedJobInclusionStrategy) jobGroupStrategy).getViewName();
+        if (jobGroupStrategy instanceof ViewBasedJobInclusionStrategy strategy) {
+            return strategy.getViewName();
         }
         return null;
     }

src/main/java/jenkins/advancedqueue/PriorityConfiguration.java

@@ -271,8 +271,8 @@ public JobGroup getJobGroup(@NonNull PriorityConfigurationCallback priorityCallb
     }
 
     private boolean isJobInView(Job<?, ?> job, View view) {
-        if (view instanceof ViewGroup) {
-            return isJobInViewGroup(job, (ViewGroup) view);
+        if (view instanceof ViewGroup group) {
+            return isJobInViewGroup(job, group);
         } else {
             return view.contains((TopLevelItem) job);
         }

src/main/java/jenkins/advancedqueue/PriorityConfigurationPlaceholderTaskHelper.java

@@ -25,8 +25,7 @@ PriorityConfigurationCallback getPriority(
             @NonNull ExecutorStepExecution.PlaceholderTask task,
             @NonNull PriorityConfigurationCallback priorityCallback) {
         Queue.Task ownerTask = task.getOwnerTask();
-        if (ownerTask instanceof Job<?, ?>) {
-            Job<?, ?> job = (Job<?, ?>) ownerTask;
+        if (ownerTask instanceof Job<?, ?> job) {
             ItemInfo itemInfo = QueueItemCache.get().getItem(job.getName());
             if (itemInfo != null) {
                 priorityCallback.setPrioritySelection(itemInfo.getPriority());

src/main/java/jenkins/advancedqueue/jobinclusion/strategy/FolderPropertyLoader.java

@@ -44,14 +44,12 @@ public static String getJobGroupName(DecisionLogger decisionLogger, Job<?, ?> jo
         ItemGroup<?> parent = job.getParent();
         decisionLogger.addDecisionLog(2, "Checking for Cloudbees Folder inclusion ...");
         while (parent != null) {
-            if (parent instanceof AbstractFolder) {
-                AbstractFolder folder = (AbstractFolder) parent;
+            if (parent instanceof AbstractFolder<?> folder) {
                 decisionLogger.addDecisionLog(3, "Evaluating Folder [" + folder.getFullName() + "] ...");
                 DescribableList<AbstractFolderProperty<?>, AbstractFolderPropertyDescriptor> properties =
                         folder.getProperties();
                 for (AbstractFolderProperty<?> property : properties) {
-                    if (property instanceof JobInclusionFolderProperty) {
-                        JobInclusionFolderProperty incProperty = (JobInclusionFolderProperty) property;
+                    if (property instanceof JobInclusionFolderProperty incProperty) {
                         if (incProperty.isUseJobGroup()) {
                             String name = incProperty.getJobGroupName();
                             decisionLogger.addDecisionLog(4, "JobGroup is enabled, with JobGroup [" + name + "] ...");
@@ -60,8 +58,8 @@ public static String getJobGroupName(DecisionLogger decisionLogger, Job<?, ?> jo
                     }
                 }
             }
-            if (parent instanceof TopLevelItem) {
-                parent = ((TopLevelItem) parent).getParent();
+            if (parent instanceof TopLevelItem item) {
+                parent = item.getParent();
             } else {
                 parent = null;
             }

src/main/java/jenkins/advancedqueue/jobinclusion/strategy/PropertyBasedJobInclusionStrategy.java

@@ -113,8 +113,8 @@ public static ListBoxModel getPropertyBasesJobGroups() {
         ListBoxModel strategies = new ListBoxModel();
         for (JobGroup jobGroup : jobGroups) {
             JobInclusionStrategy inclusionStrategy = jobGroup.getJobGroupStrategy();
-            if (inclusionStrategy instanceof PropertyBasedJobInclusionStrategy) {
-                strategies.add(((PropertyBasedJobInclusionStrategy) inclusionStrategy).getName());
+            if (inclusionStrategy instanceof PropertyBasedJobInclusionStrategy strategy) {
+                strategies.add(strategy.getName());
             }
         }
         return strategies;

src/main/java/jenkins/advancedqueue/jobinclusion/strategy/ViewBasedJobInclusionStrategy.java

@@ -68,8 +68,8 @@ public ListBoxModel getListViewItems() {
         private void addViews(String parent, ListBoxModel items, Collection<View> views) {
             for (View view : views) {
                 items.add(parent + view.getDisplayName(), parent + view.getViewName());
-                if (view instanceof ViewGroup) {
-                    addViews(parent + view.getDisplayName() + "/", items, ((ViewGroup) view).getViews());
+                if (view instanceof ViewGroup group) {
+                    addViews(parent + view.getDisplayName() + "/", items, group.getViews());
                 }
             }
         }
@@ -203,8 +203,8 @@ private boolean isJobInView(Job<?, ?> job, View view) {
             return true;
         }
         // Then try to iterate over the ViewGroup (Nested View)
-        if (view instanceof ViewGroup) {
-            return isJobInViewGroup(job, (ViewGroup) view);
+        if (view instanceof ViewGroup group) {
+            return isJobInViewGroup(job, group);
         }
         return false;
     }

src/main/java/jenkins/advancedqueue/priority/strategy/JobPropertyStrategy.java

@@ -51,8 +51,7 @@ public JobPropertyStrategy() {}
 
     @CheckForNull
     private Integer getPriorityInternal(Queue.Item item) {
-        if (item.task instanceof Job<?, ?>) {
-            Job<?, ?> job = (Job<?, ?>) item.task;
+        if (item.task instanceof Job<?, ?> job) {
             PriorityJobProperty priorityProperty = job.getProperty(PriorityJobProperty.class);
             if (priorityProperty != null && priorityProperty.getUseJobPriority()) {
                 return priorityProperty.priority;

src/main/java/jenkins/advancedqueue/sorter/AdvancedQueueSorter.java

@@ -92,19 +92,21 @@ public void sortNotWaitingItems(List<? extends Queue.NotWaitingItem> items) {
         //
         if (items.size() > 0 && LOGGER.isLoggable(Level.FINER)) {
             StringBuilder queueStr = new StringBuilder(items.get(0).getClass().getName());
-            queueStr.append(" Queue:\n"
-                    + "+----------------------------------------------------------------------+\n"
-                    + "|   Item Id  |        Job Name       | Priority |        Weight        |\n"
-                    + "+----------------------------------------------------------------------+\n");
+            queueStr.append(
+                    """
+                     Queue:
+                    +----------------------------------------------------------------------+
+                    |   Item Id  |        Job Name       | Priority |        Weight        |
+                    +----------------------------------------------------------------------+
+                    """);
             for (Queue.NotWaitingItem item : items) {
                 ItemInfo itemInfo = QueueItemCache.get().getItem(item.getId());
                 String jobName = itemInfo.getJobName();
                 if (jobName.length() > 21) {
                     jobName = jobName.substring(0, 9) + "..." + jobName.substring(jobName.length() - 9);
                 }
-                queueStr.append(String.format(
-                        "| %10d | %20s | %8d | %20.5f |%n",
-                        item.getId(), jobName, itemInfo.getPriority(), itemInfo.getWeight()));
+                queueStr.append("| %10d | %20s | %8d | %20.5f |%n"
+                        .formatted(item.getId(), jobName, itemInfo.getPriority(), itemInfo.getWeight()));
             }
             queueStr.append("+----------------------------------------------------------------------+");
             LOGGER.log(Level.FINER, queueStr.toString());

src/main/java/jenkins/advancedqueue/sorter/ItemInfo.java

@@ -87,7 +87,7 @@ public PriorityConfigurationCallback setPrioritySelection(
     }
 
     public PriorityConfigurationCallback addDecisionLog(int indent, String log) {
-        this.decisionLog.add(String.format("%" + ((indent + 1) * 2) + "s%s", "", log));
+        this.decisionLog.add(("%" + ((indent + 1) * 2) + "s%s").formatted("", log));
         return this;
     }
 
@@ -162,8 +162,7 @@ public int compareTo(ItemInfo o) {
 
     @Override
     public boolean equals(Object obj) {
-        if (obj instanceof ItemInfo) {
-            ItemInfo itemInfo = (ItemInfo) obj;
+        if (obj instanceof ItemInfo itemInfo) {
             return compareTo(itemInfo) == 0;
         }
         return false;
@@ -190,9 +189,8 @@ public String toString() {
         if (priorityStrategy != null) {
             reason = priorityStrategy.getDescriptor().getDisplayName();
         }
-        return String.format(
-                "Id: %s, JobName: %s, jobGroupId: %s, reason: %s, priority: %s, weight: %s, status: %s",
-                itemId, jobName, jobGroupId, reason, priority, weight, itemStatus);
+        return "Id: %s, JobName: %s, jobGroupId: %s, reason: %s, priority: %s, weight: %s, status: %s"
+                .formatted(itemId, jobName, jobGroupId, reason, priority, weight, itemStatus);
     }
 
     public String getDescisionLog() {

src/main/java/jenkins/advancedqueue/sorter/StartedJobItemCache.java

@@ -127,8 +127,8 @@ private void maintainCache() {
             final PendingItem pi = it.next();
             final Executable e = pi.workUnit.getExecutable();
 
-            if (e instanceof Run) {
-                startedItems.put(new StartedItem(pi.itemInfo.getJobName(), ((Run<?, ?>) e).getNumber()), pi.itemInfo);
+            if (e instanceof Run<?, ?> run) {
+                startedItems.put(new StartedItem(pi.itemInfo.getJobName(), run.getNumber()), pi.itemInfo);
                 it.remove();
             }
         }

src/test/java/jenkins/advancedqueue/sorter/strategy/FQStrategyTest.java

@@ -27,7 +27,7 @@ private void assertIncreasingWeight(float initialWeight) {
         for (int i = 0; i < 10; ++i) {
             float newWeight = new FQStrategy().getWeightToUse(1, previousWeight);
             Assert.assertTrue(
-                    String.format("New weight %s should be greater than previous weight %s", newWeight, previousWeight),
+                    "New weight %s should be greater than previous weight %s".formatted(newWeight, previousWeight),
                     newWeight > previousWeight);
             previousWeight = newWeight;
         }