fix(lesson-23): provide Librarian bean via @configuration; ensure controller accepts Librarian

Author: tobyev

lesson_23/api/java/api_app/src/main/java/com/codedifferently/lesson23/config/AppConfig.java

@@ -0,0 +1,14 @@
+package com.codedifferently.lesson23.config;
+
+import com.codedifferently.lesson23.library.Librarian;
+import org.springframework.context.annotation.Bean;
+import org.springframework.context.annotation.Configuration;
+
+@Configuration
+public class AppConfig {
+
+  @Bean
+  public Librarian librarian() {
+    return new Librarian("System", "[email protected]");
+  }
+}

lesson_23/api/java/api_app/src/main/java/com/codedifferently/lesson23/web/AddMediaItemRequest.java

@@ -1,18 +1,17 @@
 package com.codedifferently.lesson23.web;
-import java.util.UUID;
 
-import com.codedifferently.lesson23.library.MediaItem;
 import com.codedifferently.lesson23.library.Book;
+import com.codedifferently.lesson23.library.MediaItem;
+import jakarta.validation.constraints.NotNull;
 import java.util.List;
+import java.util.UUID;
 import lombok.Builder;
 import lombok.Value;
-import jakarta.validation.constraints.NotNull;
 
 @Value
 @Builder
 public class AddMediaItemRequest {
-  @NotNull
-  Item item;
+  @NotNull Item item;
 
   @Value
   @Builder
@@ -30,5 +29,7 @@ public MediaItem toDomain() {
     }
   }
 
-  public Item getItem() { return item; }
+  public Item getItem() {
+    return item;
+  }
 }

lesson_23/api/java/api_app/src/main/java/com/codedifferently/lesson23/web/MediaItemsController.java

@@ -1,16 +1,16 @@
 package com.codedifferently.lesson23.web;
-import java.util.Collections;
-import java.util.Map;
-import com.codedifferently.lesson23.library.Librarian;
-import java.util.UUID;
 
+import com.codedifferently.lesson23.library.Librarian;
 import com.codedifferently.lesson23.library.Library;
 import com.codedifferently.lesson23.library.MediaItem;
 import com.codedifferently.lesson23.library.search.SearchCriteria;
 import jakarta.validation.Valid;
 import java.io.IOException;
+import java.util.Collections;
 import java.util.List;
+import java.util.Map;
 import java.util.Set;
+import java.util.UUID;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.CrossOrigin;
 import org.springframework.web.bind.annotation.DeleteMapping;
@@ -23,10 +23,12 @@
 @RestController
 @CrossOrigin
 public class MediaItemsController {
+  private final Librarian librarian;
 
   private final Library library;
 
-  public MediaItemsController(Library library) throws IOException {
+  public MediaItemsController(Library library, Librarian librarian) throws IOException {
+    this.librarian = librarian;
     this.library = library;
   }
 
@@ -50,7 +52,8 @@ public ResponseEntity<Map<String, MediaItemResponse>> getItem(@PathVariable Stri
   }
 
   @PostMapping("/items")
-  public ResponseEntity<Map<String, MediaItemResponse>> addItem(@Valid @RequestBody AddMediaItemRequest request) {
+  public ResponseEntity<Map<String, MediaItemResponse>> addItem(
+      @Valid @RequestBody AddMediaItemRequest request) {
     MediaItem item = request.getItem().toDomain();
     library.addMediaItem(item, librarian);
     var body = Collections.singletonMap("item", MediaItemResponse.from(item));