adopt content

Author: rieckpil

labs/lab-7/src/test/java/pragmatech/digital/workshops/lab7/solutions/SharedPostgresContainerExtension.java

@@ -0,0 +1,61 @@
+# Claude Instructions
+
+You are to follow a strict test-driven development (TDD) workflow for all code changes. The process is as follows:
+
+1. Write Tests First:
+
+- Begin by writing comprehensive tests (unit, integration, or end-to-end) that define the expected behavior of the code.
+- Use explicit input/output pairs and edge cases.
+- Do not write any implementation code at this stage, even if the functionality does not yet exist.
+- Confirm that you are only writing tests, not mock implementations.
+
+2. Run and Confirm Failing Tests:
+
+- Run the tests and confirm that they fail, as the implementation does not exist yet.
+- Do not write or suggest any implementation code at this stage.
+
+3. Commit the Tests:
+
+Once the tests are complete and you are satisfied with their coverage, commit only the test code.
+
+4. Write Implementation Code:
+
+- Write the minimal implementation code required to make the tests pass.
+- Do not modify the tests during this step.
+- Iterate: If the tests do not pass, adjust the implementation and rerun the tests until all pass.
+- Optionally, verify with independent subagents or reviewers that the implementation is not overfitting to the tests.
+
+5. Commit the Implementation:
+
+- Once all tests pass and you are satisfied with the implementation, commit the code changes.
+
+
+**Guidelines:**
+
+- Always keep tests and implementation changes in separate commits.
+- Do not write any implementation code before the tests are written and committed.
+- Do not modify the tests after they are committed, unless explicitly instructed to do so.
+- Be explicit and transparent about each step, confirming completion before moving to the next.
+
+**Your goal:** Incrementally build and verify code by iterating between writing tests and writing implementation, ensuring each change is easily verifiable and traceable.
+
+## Test Code
+
+When writing test code, please follow these guidelines:
+
+- Use JUnit 5 for all test classes
+- Name test methods using the pattern: `should<ExpectedBehavior>When<Condition>`
+- Structure each test with the Arrange-Act-Assert pattern
+- Use AssertJ for all assertions
+- Avoid for loops and if statements in tests
+- Avoid comments in the test code
+- Name the class under test variable as cut
+- Create a separate test class for each production class
+- Follow a consistent setup pattern for all tests
+- Use `@DisplayName` for more descriptive test names in reports
+- Group related tests with `@Nested` classes
+- Use parameterized tests for testing multiple scenarios
+- Mock external dependencies with Mockito
+- Use Java records for test data classes
+- When constructing test data for existing domain classes, use the Builder and Object Mother pattern
+- Use Java text blocks for larger JSON data

labs/lab-8/CLAUDE.md

@@ -1,5 +1,13 @@
 # Lab 8: General FAQ and Customer Specific Issues
 
+## AI
+
+```
+For my BookController, please implement a new endpoint to export all avialable books as CSV. Include all available fields of the Book and sort it by default by the creation date.
+
+Make sure this can only be done by authenticated users with the "ADMIN" role.
+```
+
 This lab covers practical topics that frequently come up in real-world Spring Boot testing projects: GitHub Actions pipeline best practices, Maven tips for faster builds, and test organization using JUnit 5 `@Tag` with Maven profiles.
 
 ## Learning Objectives

labs/lab-8/README.md

@@ -1,5 +1,6 @@
 package pragmatech.digital.workshops.lab8.service;
 
+import java.time.LocalDate;
 import java.util.List;
 import java.util.Optional;
 

labs/lab-8/src/main/java/pragmatech/digital/workshops/lab8/service/BookService.java

@@ -1,5 +1,6 @@
 package pragmatech.digital.workshops.lab8.service;
 
+import java.math.BigDecimal;
 import java.time.Clock;
 import java.time.LocalDate;
 import java.time.temporal.ChronoUnit;
@@ -11,28 +12,32 @@
 @Service
 public class LateReturnFeeCalculator {
 
+  private static final BigDecimal RATE_TIER_ONE = new BigDecimal("1.00");
+  private static final BigDecimal RATE_TIER_TWO = new BigDecimal("1.50");
+  private static final BigDecimal RATE_TIER_THREE = new BigDecimal("2.00");
+
   private final Clock clock;
 
   public LateReturnFeeCalculator(Clock clock) {
     this.clock = clock;
   }
 
-  public double calculateFee(Book book, LocalDate borrowedDate) {
+  public BigDecimal calculateFee(Book book, LocalDate borrowedDate) {
     if (book.getStatus() != BookStatus.BORROWED) {
-      return 0.0;
+      return BigDecimal.ZERO;
     }
 
     LocalDate today = LocalDate.now(clock);
     long daysOverdue = ChronoUnit.DAYS.between(borrowedDate, today);
 
     if (daysOverdue <= 0) {
-      return 0.0;
+      return BigDecimal.ZERO;
     } else if (daysOverdue <= 7) {
-      return daysOverdue * 1.0;
+      return RATE_TIER_ONE.multiply(BigDecimal.valueOf(daysOverdue));
     } else if (daysOverdue <= 14) {
-      return daysOverdue * 1.5;
+      return RATE_TIER_TWO.multiply(BigDecimal.valueOf(daysOverdue));
     } else {
-      return daysOverdue * 2.0;
+      return RATE_TIER_THREE.multiply(BigDecimal.valueOf(daysOverdue));
     }
   }
 }

labs/lab-8/src/main/java/pragmatech/digital/workshops/lab8/service/LateReturnFeeCalculator.java

@@ -1,5 +1,6 @@
 package pragmatech.digital.workshops.lab8.experiment;
 
+import java.math.BigDecimal;
 import java.time.Clock;
 import java.time.Instant;
 import java.time.LocalDate;
@@ -40,19 +41,19 @@ void shouldReturnZeroFeeWhenBookIsAvailable() {
       Book book = new Book("978-0-13-468599-1", "Clean Code", "Martin", LocalDate.of(2008, 8, 1));
       book.setStatus(BookStatus.AVAILABLE);
 
-      double fee = cut.calculateFee(book, LocalDate.of(2025, 5, 1));
+      BigDecimal fee = cut.calculateFee(book, LocalDate.of(2025, 5, 1));
 
-      assertThat(fee).isEqualTo(0.0);
+      assertThat(fee).isEqualByComparingTo(BigDecimal.ZERO);
     }
 
     @Test
     void shouldReturnZeroFeeWhenBookIsReserved() {
       Book book = new Book("978-0-13-468599-1", "Clean Code", "Martin", LocalDate.of(2008, 8, 1));
       book.setStatus(BookStatus.RESERVED);
 
-      double fee = cut.calculateFee(book, LocalDate.of(2025, 5, 1));
+      BigDecimal fee = cut.calculateFee(book, LocalDate.of(2025, 5, 1));
 
-      assertThat(fee).isEqualTo(0.0);
+      assertThat(fee).isEqualByComparingTo(BigDecimal.ZERO);
     }
   }
 
@@ -69,58 +70,58 @@ void setUp() {
 
     @Test
     void shouldReturnZeroFeeWhenReturnedOnTime() {
-      double fee = cut.calculateFee(borrowedBook, LocalDate.of(2025, 6, 1));
+      BigDecimal fee = cut.calculateFee(borrowedBook, LocalDate.of(2025, 6, 1));
 
-      assertThat(fee).isEqualTo(0.0);
+      assertThat(fee).isEqualByComparingTo(BigDecimal.ZERO);
     }
 
     @Test
     void shouldReturnZeroFeeWhenBorrowedInFuture() {
-      double fee = cut.calculateFee(borrowedBook, LocalDate.of(2025, 6, 15));
+      BigDecimal fee = cut.calculateFee(borrowedBook, LocalDate.of(2025, 6, 15));
 
-      assertThat(fee).isEqualTo(0.0);
+      assertThat(fee).isEqualByComparingTo(BigDecimal.ZERO);
     }
 
     @ParameterizedTest(name = "{0} days overdue -> fee ${1}")
     @CsvSource({
-      "1,  1.0",
-      "3,  3.0",
-      "7,  7.0"
+      "1,  1.00",
+      "3,  3.00",
+      "7,  7.00"
     })
-    void shouldChargeOneDollarPerDayWhenOneToSevenDaysOverdue(long daysOverdue, double expectedFee) {
+    void shouldChargeOneDollarPerDayWhenOneToSevenDaysOverdue(long daysOverdue, BigDecimal expectedFee) {
       LocalDate borrowedDate = LocalDate.of(2025, 6, 1).minusDays(daysOverdue);
 
-      double fee = cut.calculateFee(borrowedBook, borrowedDate);
+      BigDecimal fee = cut.calculateFee(borrowedBook, borrowedDate);
 
-      assertThat(fee).isEqualTo(expectedFee);
+      assertThat(fee).isEqualByComparingTo(expectedFee);
     }
 
     @ParameterizedTest(name = "{0} days overdue -> fee ${1}")
     @CsvSource({
-      "8,  12.0",
-      "10, 15.0",
-      "14, 21.0"
+      "8,  12.00",
+      "10, 15.00",
+      "14, 21.00"
     })
-    void shouldChargeDollarFiftyPerDayWhenEightToFourteenDaysOverdue(long daysOverdue, double expectedFee) {
+    void shouldChargeDollarFiftyPerDayWhenEightToFourteenDaysOverdue(long daysOverdue, BigDecimal expectedFee) {
       LocalDate borrowedDate = LocalDate.of(2025, 6, 1).minusDays(daysOverdue);
 
-      double fee = cut.calculateFee(borrowedBook, borrowedDate);
+      BigDecimal fee = cut.calculateFee(borrowedBook, borrowedDate);
 
-      assertThat(fee).isEqualTo(expectedFee);
+      assertThat(fee).isEqualByComparingTo(expectedFee);
     }
 
     @ParameterizedTest(name = "{0} days overdue -> fee ${1}")
     @CsvSource({
-      "15, 30.0",
-      "20, 40.0",
-      "30, 60.0"
+      "15, 30.00",
+      "20, 40.00",
+      "30, 60.00"
     })
-    void shouldChargeTwoDollarsPerDayWhenFifteenOrMoreDaysOverdue(long daysOverdue, double expectedFee) {
+    void shouldChargeTwoDollarsPerDayWhenFifteenOrMoreDaysOverdue(long daysOverdue, BigDecimal expectedFee) {
       LocalDate borrowedDate = LocalDate.of(2025, 6, 1).minusDays(daysOverdue);
 
-      double fee = cut.calculateFee(borrowedBook, borrowedDate);
+      BigDecimal fee = cut.calculateFee(borrowedBook, borrowedDate);
 
-      assertThat(fee).isEqualTo(expectedFee);
+      assertThat(fee).isEqualByComparingTo(expectedFee);
     }
 
     @Nested
@@ -130,36 +131,36 @@ class BoundaryValues {
       void shouldApplyTierOneBoundaryAtSevenDays() {
         LocalDate borrowedDate = LocalDate.of(2025, 5, 25);
 
-        double fee = cut.calculateFee(borrowedBook, borrowedDate);
+        BigDecimal fee = cut.calculateFee(borrowedBook, borrowedDate);
 
-        assertThat(fee).isEqualTo(7.0);
+        assertThat(fee).isEqualByComparingTo(new BigDecimal("7.00"));
       }
 
       @Test
       void shouldApplyTierTwoBoundaryAtEightDays() {
         LocalDate borrowedDate = LocalDate.of(2025, 5, 24);
 
-        double fee = cut.calculateFee(borrowedBook, borrowedDate);
+        BigDecimal fee = cut.calculateFee(borrowedBook, borrowedDate);
 
-        assertThat(fee).isEqualTo(12.0);
+        assertThat(fee).isEqualByComparingTo(new BigDecimal("12.00"));
       }
 
       @Test
       void shouldApplyTierTwoBoundaryAtFourteenDays() {
         LocalDate borrowedDate = LocalDate.of(2025, 5, 18);
 
-        double fee = cut.calculateFee(borrowedBook, borrowedDate);
+        BigDecimal fee = cut.calculateFee(borrowedBook, borrowedDate);
 
-        assertThat(fee).isEqualTo(21.0);
+        assertThat(fee).isEqualByComparingTo(new BigDecimal("21.00"));
       }
 
       @Test
       void shouldApplyTierThreeBoundaryAtFifteenDays() {
         LocalDate borrowedDate = LocalDate.of(2025, 5, 17);
 
-        double fee = cut.calculateFee(borrowedBook, borrowedDate);
+        BigDecimal fee = cut.calculateFee(borrowedBook, borrowedDate);
 
-        assertThat(fee).isEqualTo(30.0);
+        assertThat(fee).isEqualByComparingTo(new BigDecimal("30.00"));
       }
     }
   }