From 02826096e9e40a6c63769189e68054ea2ede1be9 Mon Sep 17 00:00:00 2001
From: Victor Mosin <vicmosin@gmail.com>
Date: Mon, 9 Jan 2017 08:26:31 +0100
Subject: [PATCH] Applied latest changes

---
 .../org/baeldung/controller/GenericEntityController.java   | 7 ++++---
 .../baeldung/converter/StringToLocalDateTimeConverter.java | 7 +------
 spring-boot/src/main/java/org/baeldung/domain/Modes.java   | 2 +-
 .../org/baeldung/SpringBootApplicationIntegrationTest.java | 3 ++-
 4 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/spring-boot/src/main/java/org/baeldung/controller/GenericEntityController.java b/spring-boot/src/main/java/org/baeldung/controller/GenericEntityController.java
index c8649e0d3e..7d6293056a 100644
--- a/spring-boot/src/main/java/org/baeldung/controller/GenericEntityController.java
+++ b/spring-boot/src/main/java/org/baeldung/controller/GenericEntityController.java
@@ -5,6 +5,7 @@ import org.baeldung.domain.Modes;
 import org.baeldung.web.resolver.Version;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RequestMethod;
@@ -41,17 +42,17 @@ public class GenericEntityController {
         return entityList.stream().filter(entity -> entity.getId().equals(id)).findFirst().get();
     }
 
-    @RequestMapping("/entity/findbydate/{date}")
+    @GetMapping("/entity/findbydate/{date}")
     public GenericEntity findByDate(@PathVariable("date") LocalDateTime date) {
         return entityList.stream().findFirst().get();
     }
 
-    @RequestMapping("/entity/findbymode/{mode}")
+    @GetMapping("/entity/findbymode/{mode}")
     public GenericEntity findByEnum(@PathVariable("mode") Modes mode) {
         return entityList.stream().findFirst().get();
     }
 
-    @RequestMapping("/entity/findbyversion")
+    @GetMapping("/entity/findbyversion")
     public ResponseEntity findByVersion(@Version String version) {
         return version != null
             ? new ResponseEntity(entityList.stream().findFirst().get(), HttpStatus.OK)
diff --git a/spring-boot/src/main/java/org/baeldung/converter/StringToLocalDateTimeConverter.java b/spring-boot/src/main/java/org/baeldung/converter/StringToLocalDateTimeConverter.java
index 85c86faae1..cbb9e6ddb4 100644
--- a/spring-boot/src/main/java/org/baeldung/converter/StringToLocalDateTimeConverter.java
+++ b/spring-boot/src/main/java/org/baeldung/converter/StringToLocalDateTimeConverter.java
@@ -5,17 +5,12 @@ import org.springframework.stereotype.Component;
 
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
-import java.time.format.DateTimeParseException;
 
 @Component
 public class StringToLocalDateTimeConverter implements Converter<String, LocalDateTime> {
 
     @Override
     public LocalDateTime convert(final String source) {
-        try {
-            return LocalDateTime.parse(source, DateTimeFormatter.ISO_LOCAL_DATE_TIME);
-        } catch (DateTimeParseException e) {
-            throw new IllegalArgumentException("Couldn't parse value");
-        }
+        return LocalDateTime.parse(source, DateTimeFormatter.ISO_LOCAL_DATE_TIME);
     }
 }
diff --git a/spring-boot/src/main/java/org/baeldung/domain/Modes.java b/spring-boot/src/main/java/org/baeldung/domain/Modes.java
index 41b1fa0d4e..473406ef26 100644
--- a/spring-boot/src/main/java/org/baeldung/domain/Modes.java
+++ b/spring-boot/src/main/java/org/baeldung/domain/Modes.java
@@ -2,5 +2,5 @@ package org.baeldung.domain;
 
 public enum Modes {
 
-    Alpha, Beta;
+    ALPHA, BETA;
 }
diff --git a/spring-boot/src/test/java/org/baeldung/SpringBootApplicationIntegrationTest.java b/spring-boot/src/test/java/org/baeldung/SpringBootApplicationIntegrationTest.java
index 94623d3cdd..8cdcdb2216 100644
--- a/spring-boot/src/test/java/org/baeldung/SpringBootApplicationIntegrationTest.java
+++ b/spring-boot/src/test/java/org/baeldung/SpringBootApplicationIntegrationTest.java
@@ -4,6 +4,7 @@ import static org.hamcrest.Matchers.equalTo;
 import static org.hamcrest.Matchers.hasSize;
 import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
 
+import org.baeldung.domain.Modes;
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -54,7 +55,7 @@ public class SpringBootApplicationIntegrationTest {
     public void givenRequestHasBeenMade_whenMeetsFindByModeOfGivenConditions_thenCorrect() throws Exception {
         MediaType contentType = new MediaType(MediaType.APPLICATION_JSON.getType(), MediaType.APPLICATION_JSON.getSubtype(), Charset.forName("utf8"));
 
-        mockMvc.perform(MockMvcRequestBuilders.get("/entity/findbymode/{mode}", "Alpha"))
+        mockMvc.perform(MockMvcRequestBuilders.get("/entity/findbymode/{mode}", Modes.ALPHA.name()))
             .andExpect(MockMvcResultMatchers.status().isOk())
             .andExpect(MockMvcResultMatchers.content().contentType(contentType))
             .andExpect(jsonPath("$.id", equalTo(1)));