test(schedule10): section-order + combined-print ITs for Schedule 10 (Story 20.4 AC-8)

- ScheduleKeyTest (unit): pins the BR-08 fixed section order and that SCHEDULE_10
  sits between SCHEDULE_9 and SCHEDULE_11, so a combined print always emits it in
  that fixed position; plus its template path + bookmark title.
- PrintScheduleIT (Oracle IT), three cases on the existing Schedule 10 fixtures:
  - 710/2021 Schedule-10-only -> renders the section (heading, mill title block,
    division, region code->label "Northern Interior", road name) with EXACTLY one
    top-level bookmark (BR-08).
  - 514/2021 select 5+10 -> Schedule 5 prints, Schedule 10 skipped-with-a-log
    (BR-09), leaving only the Schedule 5 bookmark.
  - 715/2021 Schedule-10-only, zero pages -> 404 'Schedule not found.' (ERR-005).
  (The standalone Schedule 9 path is unchanged and stays covered by ReportSchedule9IT.)
Verified: PrintScheduleIT green under the Oracle Testcontainers profile.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: gpascucci

backend/src/test/java/ca/bc/gov/nrs/ilcr/reporting/PrintScheduleIT.java

@@ -460,6 +460,94 @@ void combinedFill_usesVirtualizer() throws Exception {
     verify(virtualizerFactory, atLeastOnce()).create();
   }
 
+  @Test
+  @DisplayName("Schedule 10 (Story 20.4): 710/2021 renders the section + exactly one bookmark")
+  void schedule10_rendersWithOneBookmark() throws Exception {
+    // Mill 710/2021 carries the rich Schedule 10 fixture (V20260817): 2 construction pages with
+    // road
+    // details, region RNI resolved to "Northern Interior". Selecting Schedule 10 alone must render
+    // its section (heading + real seeded values) and carry exactly its one top-level bookmark
+    // (BR-08).
+    String selection =
+        """
+        {"schedule10":true,"printScheduleInformation":true,"printComments":true}
+        """;
+    MvcResult result =
+        streamPdf(
+                post(ENDPOINT)
+                    .param("millId", "710")
+                    .param("year", "2021")
+                    .contentType(MediaType.APPLICATION_JSON)
+                    .content(body(selection))
+                    .accept(MediaType.APPLICATION_PDF))
+            .andExpect(status().isOk())
+            .andExpect(content().contentType(MediaType.APPLICATION_PDF))
+            .andReturn();
+
+    byte[] pdf = result.getResponse().getContentAsByteArray();
+    assertThat(new String(pdf, 0, 4)).isEqualTo("%PDF");
+
+    String text = extractText(pdf);
+    assertThat(text).contains("Schedule 10:  New Road Construction Costs");
+    assertThat(text).contains("Sch10 Rich Construction"); // mill title block (name-number)
+    assertThat(text).contains("North Division"); // page 8900 division
+    assertThat(text).contains("Northern Interior"); // region RNI resolved code -> label
+    assertThat(text).contains("Mainline A"); // road name (detail 8910)
+
+    // BR-08: a single-schedule print still carries exactly one top-level bookmark.
+    assertThat(topLevelBookmarks(pdf)).containsExactly(ScheduleKey.SCHEDULE_10.bookmarkTitle());
+  }
+
+  @Test
+  @DisplayName("skip-empty (BR-09): 514/2021 select 5+10, 10 has no data -> 5 prints, 10 omitted")
+  void schedule10_skipEmpty_keepsTheRest() throws Exception {
+    // Mill 514/2021 has Schedule 5 camps but NO Schedule 10 construction pages (the Schedule 10
+    // fixtures are mills 710-716), so selecting both must print Schedule 5 and silently omit
+    // Schedule 10 (BR-09) — leaving exactly the Schedule 5 bookmark.
+    String selection =
+        """
+        {"schedule5":true,"schedule10":true,"printScheduleInformation":true,"printComments":true}
+        """;
+    MvcResult result =
+        streamPdf(
+                post(ENDPOINT)
+                    .param("millId", "514")
+                    .param("year", "2021")
+                    .contentType(MediaType.APPLICATION_JSON)
+                    .content(body(selection)))
+            .andExpect(status().isOk())
+            .andExpect(content().contentType(MediaType.APPLICATION_PDF))
+            .andReturn();
+
+    byte[] pdf = result.getResponse().getContentAsByteArray();
+    String text = extractText(pdf);
+    assertThat(text).contains("Schedule 5:  Camp and Access Expense");
+    assertThat(text).contains("Cedar Flats Camp");
+    assertThat(text).doesNotContain("Schedule 10:  New Road Construction Costs");
+    assertThat(topLevelBookmarks(pdf)).containsExactly(ScheduleKey.SCHEDULE_5.bookmarkTitle());
+  }
+
+  @Test
+  @DisplayName("all-empty (ERR-005): 715/2021 select 10, no pages -> 404 'Schedule not found.'")
+  void schedule10Only_noData_returns404() throws Exception {
+    // Mill 715/2021 is a valid active context with ZERO Schedule 10 pages; a Schedule-10-only print
+    // is then all-empty, the legacy single-schedule outcome (ERR-005), not a blank PDF.
+    String selection =
+        """
+        {"schedule10":true,"printScheduleInformation":true,"printComments":true}
+        """;
+    mockMvc
+        .perform(
+            post(ENDPOINT)
+                .param("millId", "715")
+                .param("year", "2021")
+                .contentType(MediaType.APPLICATION_JSON)
+                .content(body(selection)))
+        .andExpect(status().isNotFound())
+        .andExpect(content().contentTypeCompatibleWith(PROBLEM_JSON))
+        .andExpect(jsonPath("$.detail").value(ERR_005));
+  }
+
   private static String extractText(byte[] pdf) throws Exception {
     try (PDDocument document = Loader.loadPDF(pdf)) {
       return new PDFTextStripper().getText(document);

backend/src/test/java/ca/bc/gov/nrs/ilcr/reporting/ScheduleKeyTest.java

@@ -0,0 +1,49 @@
+package ca.bc.gov.nrs.ilcr.reporting;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+import java.util.Arrays;
+import java.util.List;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+/**
+ * Unit test — {@link ScheduleKey} declaration order (BR-08). The enum's declaration order IS the
+ * combined-PDF section order (the orchestrator iterates {@link ScheduleKey#values()} with no
+ * separate ordering table), so this pins the FIXED legacy sequence and, in particular, that
+ * Schedule 10 (Story 20.4) sits between Schedule 9 and Schedule 11 — i.e. a combined print always
+ * emits it in that fixed position relative to the other selected sections. No Spring context or
+ * database.
+ */
+@DisplayName("ScheduleKey — fixed legacy section order (BR-08)")
+class ScheduleKeyTest {
+
+  @Test
+  @DisplayName("values() are in the fixed legacy print order, with SCHEDULE_10 between 9 and 11")
+  void values_areInFixedLegacyOrder() {
+    List<ScheduleKey> order = Arrays.asList(ScheduleKey.values());
+
+    assertThat(order)
+        .containsExactly(
+            ScheduleKey.SCHEDULE_5,
+            ScheduleKey.SCHEDULE_6,
+            ScheduleKey.SCHEDULE_7A,
+            ScheduleKey.SCHEDULE_7B,
+            ScheduleKey.SCHEDULE_9,
+            ScheduleKey.SCHEDULE_10,
+            ScheduleKey.SCHEDULE_11);
+
+    // The load-bearing 20.4 fact: Schedule 10 renders after 9 and before 11 in any combined print.
+    assertThat(order.indexOf(ScheduleKey.SCHEDULE_10))
+        .isGreaterThan(order.indexOf(ScheduleKey.SCHEDULE_9))
+        .isLessThan(order.indexOf(ScheduleKey.SCHEDULE_11));
+  }
+
+  @Test
+  @DisplayName("SCHEDULE_10 is bound to its template + bookmark title")
+  void schedule10_hasTemplateAndBookmark() {
+    assertThat(ScheduleKey.SCHEDULE_10.templatePath()).isEqualTo("reports/schedule10.jrxml");
+    assertThat(ScheduleKey.SCHEDULE_10.bookmarkTitle())
+        .isEqualTo("Schedule 10: New Road Construction Costs");
+  }
+}