diff --git a/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRow.java b/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRow.java index af750cd..dfea755 100644 --- a/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRow.java +++ b/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRow.java @@ -205,11 +205,11 @@ public record GuiBatchRunResultRow( return RESET_PENDING_ICON; } return switch (status) { - case SUCCESS -> "\u2714"; // ✔ HEAVY CHECK MARK - case FAILED_RETRYABLE -> "\u26A0"; // ⚠ WARNING SIGN - case FAILED_PERMANENT -> "\u2718"; // ✘ HEAVY BALLOT X - case SKIPPED_ALREADY_PROCESSED, - SKIPPED_FINAL_FAILURE -> "\u25BA"; // ► BLACK RIGHT-POINTING POINTER + case SUCCESS -> "\u2714"; // ✔ HEAVY CHECK MARK + case FAILED_RETRYABLE -> "\u26A0"; // ⚠ WARNING SIGN + case FAILED_PERMANENT -> "\u2718"; // ✘ HEAVY BALLOT X + case SKIPPED_ALREADY_PROCESSED -> "\u23ED"; // ⏭ NEXT TRACK BUTTON + case SKIPPED_FINAL_FAILURE -> "\u2298"; // ⊘ CIRCLED DIVISION SLASH }; } diff --git a/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java b/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java index 9e56f12..04e1b14 100644 --- a/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java +++ b/pdf-umbenenner-adapter-in-gui/src/main/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTab.java @@ -1425,7 +1425,8 @@ public final class GuiBatchRunTab { case SUCCESS -> "#2e7d32"; case FAILED_RETRYABLE -> "#e65100"; case FAILED_PERMANENT -> "#c62828"; - case SKIPPED_ALREADY_PROCESSED, SKIPPED_FINAL_FAILURE -> "#757575"; + case SKIPPED_ALREADY_PROCESSED -> "#1565c0"; + case SKIPPED_FINAL_FAILURE -> "#757575"; }; } diff --git a/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunCoordinatorTest.java b/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunCoordinatorTest.java index 6e2b91b..73f749f 100644 --- a/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunCoordinatorTest.java +++ b/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunCoordinatorTest.java @@ -290,7 +290,7 @@ class GuiBatchRunCoordinatorTest { assertEquals("\u2714", row(DocumentCompletionStatus.SUCCESS).statusIcon()); assertEquals("\u26A0", row(DocumentCompletionStatus.FAILED_RETRYABLE).statusIcon()); assertEquals("\u2718", row(DocumentCompletionStatus.FAILED_PERMANENT).statusIcon()); - assertEquals("\u25BA", row(DocumentCompletionStatus.SKIPPED_ALREADY_PROCESSED).statusIcon()); + assertEquals("\u23ED", row(DocumentCompletionStatus.SKIPPED_ALREADY_PROCESSED).statusIcon()); } @Test diff --git a/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRowTest.java b/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRowTest.java index 270569c..b11cdc4 100644 --- a/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRowTest.java +++ b/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunResultRowTest.java @@ -109,13 +109,21 @@ class GuiBatchRunResultRowTest { } @Test - void statusIcon_skippedAlreadyProcessed_isPointer() { - assertEquals("\u25BA", row(DocumentCompletionStatus.SKIPPED_ALREADY_PROCESSED).statusIcon()); + void statusIcon_skippedAlreadyProcessed_isNextTrack() { + assertEquals("\u23ED", row(DocumentCompletionStatus.SKIPPED_ALREADY_PROCESSED).statusIcon()); } @Test - void statusIcon_skippedFinalFailure_isPointer() { - assertEquals("\u25BA", row(DocumentCompletionStatus.SKIPPED_FINAL_FAILURE).statusIcon()); + void statusIcon_skippedFinalFailure_isCircledDivisionSlash() { + assertEquals("\u2298", row(DocumentCompletionStatus.SKIPPED_FINAL_FAILURE).statusIcon()); + } + + @Test + void statusIcon_skippedValues_areDifferentFromEachOther() { + String alreadyProcessed = row(DocumentCompletionStatus.SKIPPED_ALREADY_PROCESSED).statusIcon(); + String finalFailure = row(DocumentCompletionStatus.SKIPPED_FINAL_FAILURE).statusIcon(); + assertFalse(alreadyProcessed.equals(finalFailure), + "SKIPPED_ALREADY_PROCESSED und SKIPPED_FINAL_FAILURE müssen unterschiedliche Icons haben"); } // ------------------------------------------------------------------------- diff --git a/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTabSmokeTest.java b/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTabSmokeTest.java index adc3513..fba727c 100644 --- a/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTabSmokeTest.java +++ b/pdf-umbenenner-adapter-in-gui/src/test/java/de/gecheckt/pdf/umbenenner/adapter/in/gui/batchrun/GuiBatchRunTabSmokeTest.java @@ -140,10 +140,10 @@ class GuiBatchRunTabSmokeTest { tab().resultTable().getSelectionModel().select(1); assertTrue(tab().detailArea().getText().contains(GuiBatchRunTab.NO_REASONING_TEXT)); - // SKIPPED row must carry the ► icon, not ✘. + // SKIPPED_ALREADY_PROCESSED muss das Weiterspulen-Icon ⏭ tragen, nicht ✘. GuiBatchRunResultRow skippedRow = tab().resultTable().getItems().get(2); assertEquals(DocumentCompletionStatus.SKIPPED_ALREADY_PROCESSED, skippedRow.status()); - assertEquals("\u25BA", skippedRow.statusIcon()); + assertEquals("\u23ED", skippedRow.statusIcon()); }); }