Refactor

src/main/java/me/mourjo/jamboree/JamboreeApplication.java

@@ -8,17 +8,15 @@
 import org.springframework.boot.autoconfigure.SpringBootApplication;
 import org.springframework.context.annotation.Bean;
 
-import java.util.concurrent.Executors;
-
 @SpringBootApplication
 public class JamboreeApplication {
 
-    private static Logger log = LoggerFactory.getLogger(JamboreeApplication.class);
+    private static final Logger logger = LoggerFactory.getLogger(JamboreeApplication.class);
 
     public static void main(String[] args) {
-        log.info("Starting application");
+        logger.info("Starting application");
         SpringApplication.run(JamboreeApplication.class, args);
-        log.info("Started application");
+        logger.info("Started application");
     }
 
     @Bean

src/main/java/me/mourjo/jamboree/data/Response.java

@@ -11,7 +11,7 @@
 
 public class Response extends ResponseEntity<Map<String, String>> {
 
-    private static Logger logger = LoggerFactory.getLogger(Response.class);
+    private static final Logger logger = LoggerFactory.getLogger(Response.class);
 
     public Response(Map<String, String> body, HttpStatusCode status) {
         super(body, status);

src/main/java/me/mourjo/jamboree/rest/PartyController.java

@@ -18,7 +18,7 @@
 public class PartyController {
 
     private final PartyService service;
-    private Logger logger = LoggerFactory.getLogger(getClass().getName());
+    private final Logger logger = LoggerFactory.getLogger(getClass().getName());
 
     PartyController(PartyService service) {
         this.service = service;

src/main/java/me/mourjo/jamboree/service/PartyService.java

@@ -15,7 +15,7 @@
 public class PartyService {
     private final PartyRepository repository;
     private final IDGenerator idGenerator;
-    private Logger logger = LoggerFactory.getLogger(getClass().getName());
+    private final Logger logger = LoggerFactory.getLogger(getClass().getName());
 
     PartyService(PartyRepository repository, IDGenerator idGenerator) {
         this.repository = repository;
@@ -38,7 +38,7 @@ public Party add(String name, String location, ZonedDateTime ts) {
 
     public Optional<Party> find(long id) {
         var party = repository.findById(id);
-        if (party.isEmpty()){
+        if (party.isEmpty()) {
             logger.warn("Requested party does not exist: {}", id);
         }
 

src/test/java/me/mourjo/jamboree/JamboreeApplicationTests.java

@@ -13,7 +13,6 @@
 import java.util.Set;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertTrue;
 
 
 @SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)

src/test/java/me/mourjo/jamboree/datetime/DatetimeDatetimeFormatTest.java

@@ -5,7 +5,7 @@
 import java.time.ZoneId;
 import java.time.ZonedDateTime;
 
-import static org.junit.jupiter.api.Assertions.*;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 class DatetimeDatetimeFormatTest {
     @Test