From 760692b8eb5e66ef20be5c183ab0e00e98c470df Mon Sep 17 00:00:00 2001
From: Grzegorz Piwowarek <gpiwowarek@gmail.com>
Date: Sat, 1 Apr 2017 09:06:59 +0200
Subject: [PATCH] Build time optimization (#1553)

* Add integration tests

* Optimize build
---
 apache-poi/.gitignore                         |  2 ++
 jee7/pom.xml                                  |  2 +-
 .../ScheduleTimerBeanIntegrationTest.java     | 36 +++++++++++--------
 jooq/src/test/java/com/baeldung/JOOLTest.java |  3 --
 pom.xml                                       | 13 +++++++
 querydsl/pom.xml                              |  1 -
 ...Test.java => QueryDSLIntegrationTest.java} | 20 +++++------
 rxjava/pom.xml                                | 13 +++++++
 ...xJavaBackpressureLongRunningUnitTest.java} |  2 +-
 ... => MessageControllerIntegrationTest.java} |  5 +--
 spring-jooq/pom.xml                           |  2 +-
 11 files changed, 66 insertions(+), 33 deletions(-)
 rename querydsl/src/test/java/org/baeldung/querydsl/intro/{QueryDSLTest.java => QueryDSLIntegrationTest.java} (99%)
 rename rxjava/src/test/java/com/baeldung/rxjava/{RxJavaBackpressureTest.java => RxJavaBackpressureLongRunningUnitTest.java} (98%)
 rename spring-amqp-simple/src/test/java/com/baeldung/springamqpsimple/{MessageControllerTest.java => MessageControllerIntegrationTest.java} (88%)

diff --git a/apache-poi/.gitignore b/apache-poi/.gitignore
index e05054868c..9552c1e63d 100644
--- a/apache-poi/.gitignore
+++ b/apache-poi/.gitignore
@@ -1 +1,3 @@
 *.docx
+temp.xls
+temp.xlsx
diff --git a/jee7/pom.xml b/jee7/pom.xml
index f275f56d58..26d433df78 100644
--- a/jee7/pom.xml
+++ b/jee7/pom.xml
@@ -4,7 +4,7 @@
     <modelVersion>4.0.0</modelVersion>
 
     <groupId>com.baeldung</groupId>
-    <artifactId>jee7schedule</artifactId>
+    <artifactId>jee7</artifactId>
     <version>1.0-SNAPSHOT</version>
     <description>JavaEE 7 Arquillian Archetype Sample</description>
     <packaging>war</packaging>
diff --git a/jee7/src/test/java/com/baeldung/timer/ScheduleTimerBeanIntegrationTest.java b/jee7/src/test/java/com/baeldung/timer/ScheduleTimerBeanIntegrationTest.java
index c3c5ad44de..580edade77 100644
--- a/jee7/src/test/java/com/baeldung/timer/ScheduleTimerBeanIntegrationTest.java
+++ b/jee7/src/test/java/com/baeldung/timer/ScheduleTimerBeanIntegrationTest.java
@@ -19,25 +19,27 @@ import static com.jayway.awaitility.Awaitility.to;
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.equalTo;
 
-
 @RunWith(Arquillian.class)
 public class ScheduleTimerBeanIntegrationTest {
 
-    final static long TIMEOUT = 5000l;
-    final static long TOLERANCE = 1000l;
+    private final static long TIMEOUT = 5000l;
+    private final static long TOLERANCE = 1000l;
 
-    @Inject
-    TimerEventListener timerEventListener;
+    @Inject TimerEventListener timerEventListener;
 
     @Deployment
     public static WebArchive deploy() {
-        File[] jars = Maven.resolver().loadPomFromFile("pom.xml")
-                .resolve("com.jayway.awaitility:awaitility")
-                .withTransitivity().asFile();
+        File[] jars = Maven
+          .resolver()
+          .loadPomFromFile("pom.xml")
+          .resolve("com.jayway.awaitility:awaitility")
+          .withTransitivity()
+          .asFile();
 
-        return ShrinkWrap.create(WebArchive.class)
-                .addAsLibraries(jars)
-                .addClasses(WithinWindowMatcher.class, TimerEvent.class, TimerEventListener.class, ScheduleTimerBean.class);
+        return ShrinkWrap
+          .create(WebArchive.class)
+          .addAsLibraries(jars)
+          .addClasses(WithinWindowMatcher.class, TimerEvent.class, TimerEventListener.class, ScheduleTimerBean.class);
     }
 
     @Test
@@ -46,9 +48,15 @@ public class ScheduleTimerBeanIntegrationTest {
         Awaitility.setDefaultTimeout(30, TimeUnit.SECONDS);
         await().untilCall(to(timerEventListener.getEvents()).size(), equalTo(3));
 
-        TimerEvent firstEvent = timerEventListener.getEvents().get(0);
-        TimerEvent secondEvent = timerEventListener.getEvents().get(1);
-        TimerEvent thirdEvent = timerEventListener.getEvents().get(2);
+        TimerEvent firstEvent = timerEventListener
+          .getEvents()
+          .get(0);
+        TimerEvent secondEvent = timerEventListener
+          .getEvents()
+          .get(1);
+        TimerEvent thirdEvent = timerEventListener
+          .getEvents()
+          .get(2);
 
         long delay = secondEvent.getTime() - firstEvent.getTime();
         assertThat(delay, Matchers.is(WithinWindowMatcher.withinWindow(TIMEOUT, TOLERANCE)));
diff --git a/jooq/src/test/java/com/baeldung/JOOLTest.java b/jooq/src/test/java/com/baeldung/JOOLTest.java
index 13bf1a3ec3..a0d5f4037f 100644
--- a/jooq/src/test/java/com/baeldung/JOOLTest.java
+++ b/jooq/src/test/java/com/baeldung/JOOLTest.java
@@ -1,12 +1,9 @@
 package com.baeldung;
 
-
-import junit.framework.Assert;
 import org.jooq.lambda.Seq;
 import org.jooq.lambda.Unchecked;
 import org.jooq.lambda.function.Function1;
 import org.jooq.lambda.function.Function2;
-import org.jooq.lambda.tuple.Tuple;
 import org.jooq.lambda.tuple.Tuple2;
 import org.jooq.lambda.tuple.Tuple3;
 import org.jooq.lambda.tuple.Tuple4;
diff --git a/pom.xml b/pom.xml
index b26ada6f24..76f7247f51 100644
--- a/pom.xml
+++ b/pom.xml
@@ -214,6 +214,7 @@
         <module>vertx</module>
         <module>spring-data-gemfire</module>
     </modules>
+
     <build>
         <plugins>
             <plugin>
@@ -226,6 +227,18 @@
 
                 </configuration>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <configuration>
+                    <excludes>
+                        <exclude>**/*IntegrationTest.java</exclude>
+                        <exclude>**/*LongRunningUnitTest.java</exclude>
+                        <exclude>**/*ManualTest.java</exclude>
+                    </excludes>
+                    <testFailureIgnore>true</testFailureIgnore>
+                </configuration>
+            </plugin>
         </plugins>
         <extensions>
             <!--<extension>
diff --git a/querydsl/pom.xml b/querydsl/pom.xml
index a9218e7751..b0a07fb3ff 100644
--- a/querydsl/pom.xml
+++ b/querydsl/pom.xml
@@ -180,7 +180,6 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-surefire-plugin</artifactId>
-                <version>${maven-surefire-plugin.version}</version>
                 <configuration>
                     <excludes>
                         <exclude>**/*IntegrationTest.java</exclude>
diff --git a/querydsl/src/test/java/org/baeldung/querydsl/intro/QueryDSLTest.java b/querydsl/src/test/java/org/baeldung/querydsl/intro/QueryDSLIntegrationTest.java
similarity index 99%
rename from querydsl/src/test/java/org/baeldung/querydsl/intro/QueryDSLTest.java
rename to querydsl/src/test/java/org/baeldung/querydsl/intro/QueryDSLIntegrationTest.java
index 682fa2c245..15a32b8df5 100644
--- a/querydsl/src/test/java/org/baeldung/querydsl/intro/QueryDSLTest.java
+++ b/querydsl/src/test/java/org/baeldung/querydsl/intro/QueryDSLIntegrationTest.java
@@ -3,25 +3,25 @@
  */
 package org.baeldung.querydsl.intro;
 
-import java.util.List;
-import javax.persistence.EntityManager;
-import javax.persistence.EntityManagerFactory;
-import javax.persistence.Persistence;
-
-import org.baeldung.querydsl.intro.entities.BlogPost;
-import org.baeldung.querydsl.intro.entities.QBlogPost;
-import org.baeldung.querydsl.intro.entities.QUser;
-import org.baeldung.querydsl.intro.entities.User;
 import com.querydsl.core.Tuple;
 import com.querydsl.core.types.dsl.Expressions;
 import com.querydsl.core.types.dsl.NumberPath;
 import com.querydsl.jpa.JPAExpressions;
 import com.querydsl.jpa.impl.JPAQueryFactory;
+import org.baeldung.querydsl.intro.entities.BlogPost;
+import org.baeldung.querydsl.intro.entities.QBlogPost;
+import org.baeldung.querydsl.intro.entities.QUser;
+import org.baeldung.querydsl.intro.entities.User;
 import org.junit.*;
 
+import javax.persistence.EntityManager;
+import javax.persistence.EntityManagerFactory;
+import javax.persistence.Persistence;
+import java.util.List;
+
 import static org.junit.Assert.*;
 
-public class QueryDSLTest {
+public class QueryDSLIntegrationTest {
 
     private static EntityManagerFactory emf;
 
diff --git a/rxjava/pom.xml b/rxjava/pom.xml
index b3936bf78d..4172187b03 100644
--- a/rxjava/pom.xml
+++ b/rxjava/pom.xml
@@ -17,6 +17,18 @@
                     <target>1.8</target>
                 </configuration>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-surefire-plugin</artifactId>
+                <configuration>
+                    <excludes>
+                        <exclude>**/*IntegrationTest.java</exclude>
+                        <exclude>**/*LongRunningUnitTest.java</exclude>
+                        <exclude>**/*ManualTest.java</exclude>
+                    </excludes>
+                    <testFailureIgnore>true</testFailureIgnore>
+                </configuration>
+            </plugin>
         </plugins>
     </build>
 
@@ -47,4 +59,5 @@
         <hamcrest.version>1.3</hamcrest.version>
     </properties>
 
+
 </project>
\ No newline at end of file
diff --git a/rxjava/src/test/java/com/baeldung/rxjava/RxJavaBackpressureTest.java b/rxjava/src/test/java/com/baeldung/rxjava/RxJavaBackpressureLongRunningUnitTest.java
similarity index 98%
rename from rxjava/src/test/java/com/baeldung/rxjava/RxJavaBackpressureTest.java
rename to rxjava/src/test/java/com/baeldung/rxjava/RxJavaBackpressureLongRunningUnitTest.java
index 8a495650b3..040936a67a 100644
--- a/rxjava/src/test/java/com/baeldung/rxjava/RxJavaBackpressureTest.java
+++ b/rxjava/src/test/java/com/baeldung/rxjava/RxJavaBackpressureLongRunningUnitTest.java
@@ -14,7 +14,7 @@ import java.util.stream.IntStream;
 
 import static org.junit.Assert.assertTrue;
 
-public class RxJavaBackpressureTest {
+public class RxJavaBackpressureLongRunningUnitTest {
 
     @Test
     public void givenColdObservable_shouldNotThrowException() {
diff --git a/spring-amqp-simple/src/test/java/com/baeldung/springamqpsimple/MessageControllerTest.java b/spring-amqp-simple/src/test/java/com/baeldung/springamqpsimple/MessageControllerIntegrationTest.java
similarity index 88%
rename from spring-amqp-simple/src/test/java/com/baeldung/springamqpsimple/MessageControllerTest.java
rename to spring-amqp-simple/src/test/java/com/baeldung/springamqpsimple/MessageControllerIntegrationTest.java
index c62c86290a..a053edaa0e 100644
--- a/spring-amqp-simple/src/test/java/com/baeldung/springamqpsimple/MessageControllerTest.java
+++ b/spring-amqp-simple/src/test/java/com/baeldung/springamqpsimple/MessageControllerIntegrationTest.java
@@ -15,11 +15,12 @@ import org.springframework.test.context.junit4.SpringRunner;
 
 import static org.junit.Assert.assertEquals;
 import static org.mockito.Mockito.verify;
+import static org.springframework.boot.test.context.SpringBootTest.WebEnvironment.RANDOM_PORT;
 
 @RunWith(SpringRunner.class)
 @ActiveProfiles("test")
-@SpringBootTest(webEnvironment=SpringBootTest.WebEnvironment.RANDOM_PORT)
-public class MessageControllerTest {
+@SpringBootTest(webEnvironment = RANDOM_PORT)
+public class MessageControllerIntegrationTest {
 
     @Autowired
     private TestRestTemplate restTemplate;
diff --git a/spring-jooq/pom.xml b/spring-jooq/pom.xml
index 195c0b8514..e067e9b47f 100644
--- a/spring-jooq/pom.xml
+++ b/spring-jooq/pom.xml
@@ -2,7 +2,7 @@
     xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
     <groupId>com.baeldung</groupId>
-    <artifactId>jooq-spring</artifactId>
+    <artifactId>sprin-jooq</artifactId>
     <version>0.0.1-SNAPSHOT</version>
 
     <dependencyManagement>