Bael 1601 redirects (#4006)

* tidy pom
move classes to javax-servlets

* move into spring-mvc-simple

* updated POM to handle no default tests in module

spring-mvc-simple/pom.xml

@@ -1,17 +1,17 @@
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
     <modelVersion>4.0.0</modelVersion>
-    <groupId>com.baeldung</groupId>
     <artifactId>spring-mvc-simple</artifactId>
     <packaging>war</packaging>
     <version>0.0.1-SNAPSHOT</version>
     <name>Spring MVC simple Maven Webapp</name>
     <url>http://maven.apache.org</url>
-	<parent>
-	    <groupId>com.baeldung</groupId>
-	    <artifactId>parent-modules</artifactId>
-	    <version>1.0.0-SNAPSHOT</version>
-	 </parent>
+    <parent>
+        <groupId>com.baeldung</groupId>
+        <artifactId>parent-modules</artifactId>
+        <version>1.0.0-SNAPSHOT</version>
+    </parent>
     <properties>
         <maven.compiler.source>1.8</maven.compiler.source>
         <maven.compiler.target>1.8</maven.compiler.target>
@@ -19,7 +19,7 @@
         <springframework.version>5.0.2.RELEASE</springframework.version>
         <maven-war-plugin.version>3.2.0</maven-war-plugin.version>
         <maven.compiler.version>3.7.0</maven.compiler.version>
-        <maven-surefire-plugin.version>2.20</maven-surefire-plugin.version>
+        <maven-surefire-plugin.version>2.21.0</maven-surefire-plugin.version>
         <jstl.version>1.2</jstl.version>
         <javax.servlet.jsp-api.version>2.3.2-b02</javax.servlet.jsp-api.version>
         <javax.servlet-api.version>4.0.0</javax.servlet-api.version>
@@ -172,6 +172,14 @@
                 <plugin>
                     <artifactId>maven-surefire-plugin</artifactId>
                     <version>${maven-surefire-plugin.version}</version>
+                    <executions>
+                        <execution>
+                            <id>default-test</id>
+                            <configuration>
+                                <skip>true</skip>
+                            </configuration>
+                        </execution>
+                    </executions>
                     <dependencies>
                         <dependency>
                             <groupId>org.junit.platform</groupId>

spring-mvc-simple/src/main/java/com/baeldung/spring/servlets/ForwardedServlet.java

@@ -0,0 +1,20 @@
+package com.baeldung.spring.servlets;
+
+import javax.servlet.ServletException;
+import javax.servlet.annotation.WebServlet;
+import javax.servlet.http.HttpServlet;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import java.io.IOException;
+import java.io.PrintWriter;
+
+@WebServlet("/forwarded")
+public class ForwardedServlet extends HttpServlet {
+
+    protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
+        PrintWriter out = resp.getWriter();
+        out.write("In forwarded servlet page.");
+        out.write("\nWelcome:" + req.getParameter("name"));
+    }
+
+}

spring-mvc-simple/src/main/java/com/baeldung/spring/servlets/HelloServlet.java

@@ -0,0 +1,19 @@
+package com.baeldung.spring.servlets;
+
+import javax.servlet.RequestDispatcher;
+import javax.servlet.ServletException;
+import javax.servlet.annotation.WebServlet;
+import javax.servlet.http.HttpServlet;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import java.io.IOException;
+
+@WebServlet("/hello")
+public class HelloServlet extends HttpServlet {
+
+    protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
+        RequestDispatcher dispatcher = req.getRequestDispatcher("/forwarded");
+        dispatcher.forward(req, resp);
+    }
+
+}

spring-mvc-simple/src/main/java/com/baeldung/spring/servlets/RedirectedServlet.java

@@ -0,0 +1,20 @@
+package com.baeldung.spring.servlets;
+
+import javax.servlet.ServletException;
+import javax.servlet.annotation.WebServlet;
+import javax.servlet.http.HttpServlet;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import java.io.IOException;
+import java.io.PrintWriter;
+
+@WebServlet("/redirected")
+public class RedirectedServlet extends HttpServlet {
+
+    protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
+        PrintWriter out = resp.getWriter();
+        out.write("In redirected servlet page.");
+        out.write("\nWelcome:" + req.getParameter("name"));
+    }
+
+}

spring-mvc-simple/src/main/java/com/baeldung/spring/servlets/WelcomeServlet.java

@@ -0,0 +1,17 @@
+package com.baeldung.spring.servlets;
+
+import javax.servlet.ServletException;
+import javax.servlet.annotation.WebServlet;
+import javax.servlet.http.HttpServlet;
+import javax.servlet.http.HttpServletRequest;
+import javax.servlet.http.HttpServletResponse;
+import java.io.IOException;
+
+@WebServlet("/welcome")
+public class WelcomeServlet extends HttpServlet {
+
+    protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
+        resp.sendRedirect(req.getContextPath() + "/redirected");
+    }
+
+}

spring-mvc-simple/src/test/java/com/baeldung/spring/servlets/HelloServletTest.java

@@ -0,0 +1,24 @@
+package com.baeldung.spring.servlets;
+
+import org.junit.jupiter.api.Test;
+import org.springframework.mock.web.MockHttpServletRequest;
+import org.springframework.mock.web.MockHttpServletResponse;
+import javax.servlet.ServletException;
+import java.io.IOException;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+public class HelloServletTest {
+    @Test
+    public void whenRequested_thenForwardToCorrectUrl() throws ServletException, IOException {
+        MockHttpServletRequest request = new MockHttpServletRequest("GET", "/hello");
+        request.addParameter("name", "Dennis");
+        MockHttpServletResponse response = new MockHttpServletResponse();
+        HelloServlet servlet = new HelloServlet();
+
+        servlet.doGet(request, response);
+
+        assertEquals("/forwarded", response.getForwardedUrl());
+        assertEquals(200, response.getStatus());
+    }
+}

spring-mvc-simple/src/test/java/com/baeldung/spring/servlets/WelcomeServletTest.java

@@ -0,0 +1,24 @@
+package com.baeldung.spring.servlets;
+
+import org.junit.jupiter.api.Test;
+import org.springframework.mock.web.MockHttpServletRequest;
+import org.springframework.mock.web.MockHttpServletResponse;
+import javax.servlet.ServletException;
+import java.io.IOException;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+public class WelcomeServletTest {
+    @Test
+    public void whenRequested_thenRedirectedToCorrectUrl() throws ServletException, IOException {
+        MockHttpServletRequest request = new MockHttpServletRequest("GET", "/welcome");
+        request.addParameter("name", "Dennis");
+        WelcomeServlet servlet = new WelcomeServlet();
+        MockHttpServletResponse response = new MockHttpServletResponse();
+        
+        servlet.doGet(request, response);
+        
+        assertEquals("/redirected", response.getRedirectedUrl());
+        assertEquals(302, response.getStatus());
+    }
+}