From 891b4eb2484d69f34b620509ca7dbbbad803f7fb Mon Sep 17 00:00:00 2001
From: Grzegorz Piwowarek <gpiwowarek@gmail.com>
Date: Sat, 5 Aug 2017 14:11:45 +0200
Subject: [PATCH] Remove jooq module (#2375)

* Remove jooq module

* Refactor
---
 jooq/README.md                                |  3 -
 jooq/pom.xml                                  | 25 ------
 .../test/java/com/baeldung/jool/JOOLTest.java | 90 +++++++++----------
 pom.xml                                       |  1 -
 4 files changed, 45 insertions(+), 74 deletions(-)
 delete mode 100644 jooq/README.md
 delete mode 100644 jooq/pom.xml
 rename jooq/src/test/java/com/baeldung/JOOLUnitTest.java => libraries/src/test/java/com/baeldung/jool/JOOLTest.java (64%)

diff --git a/jooq/README.md b/jooq/README.md
deleted file mode 100644
index 2f09cab46b..0000000000
--- a/jooq/README.md
+++ /dev/null
@@ -1,3 +0,0 @@
-### Relevant articles
-
-- [Introduction to jOOL](http://www.baeldung.com/jool)
diff --git a/jooq/pom.xml b/jooq/pom.xml
deleted file mode 100644
index 667640d085..0000000000
--- a/jooq/pom.xml
+++ /dev/null
@@ -1,25 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<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/xsd/maven-4.0.0.xsd">
-    <parent>
-        <artifactId>parent-modules</artifactId>
-        <groupId>com.baeldung</groupId>
-        <version>1.0.0-SNAPSHOT</version>
-    </parent>
-    <modelVersion>4.0.0</modelVersion>
-
-    <artifactId>jooq</artifactId>
-
-    <dependencies>
-        <dependency>
-            <groupId>org.jooq</groupId>
-            <artifactId>jool</artifactId>
-            <version>${jool.version}</version>
-        </dependency>
-    </dependencies>
-
-    <properties>
-        <jool.version>0.9.12</jool.version>
-    </properties>
-
-</project>
\ No newline at end of file
diff --git a/jooq/src/test/java/com/baeldung/JOOLUnitTest.java b/libraries/src/test/java/com/baeldung/jool/JOOLTest.java
similarity index 64%
rename from jooq/src/test/java/com/baeldung/JOOLUnitTest.java
rename to libraries/src/test/java/com/baeldung/jool/JOOLTest.java
index 17873db50a..ba20e153fd 100644
--- a/jooq/src/test/java/com/baeldung/JOOLUnitTest.java
+++ b/libraries/src/test/java/com/baeldung/jool/JOOLTest.java
@@ -1,4 +1,4 @@
-package com.baeldung;
+package com.baeldung.jool;
 
 import org.jooq.lambda.Seq;
 import org.jooq.lambda.Unchecked;
@@ -13,6 +13,7 @@ import java.util.Arrays;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Objects;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
@@ -20,7 +21,7 @@ import static junit.framework.Assert.assertTrue;
 import static junit.framework.TestCase.assertEquals;
 import static org.jooq.lambda.tuple.Tuple.tuple;
 
-public class JOOLUnitTest {
+public class JOOLTest {
     @Test
     public void givenSeq_whenCheckContains_shouldReturnTrue() {
         List<Integer> concat = Seq.of(1, 2, 3).concat(Seq.of(4, 5, 6)).toList();
@@ -54,58 +55,58 @@ public class JOOLUnitTest {
     @Test
     public void givenSeq_whenJoin_shouldHaveElementsFromBothSeq() {
         assertEquals(
-                Seq.of(1, 2, 4).innerJoin(Seq.of(1, 2, 3), (a, b) -> a == b).toList(),
-                Arrays.asList(tuple(1, 1), tuple(2, 2))
+          Seq.of(1, 2, 4).innerJoin(Seq.of(1, 2, 3), Objects::equals).toList(),
+          Arrays.asList(tuple(1, 1), tuple(2, 2))
         );
 
 
         assertEquals(
-                Seq.of(1, 2, 4).leftOuterJoin(Seq.of(1, 2, 3), (a, b) -> a == b).toList(),
-                Arrays.asList(tuple(1, 1), tuple(2, 2), tuple(4, null))
+          Seq.of(1, 2, 4).leftOuterJoin(Seq.of(1, 2, 3), Objects::equals).toList(),
+          Arrays.asList(tuple(1, 1), tuple(2, 2), tuple(4, null))
         );
 
         assertEquals(
-                Seq.of(1, 2, 4).rightOuterJoin(Seq.of(1, 2, 3), (a, b) -> a == b).toList(),
-                Arrays.asList(tuple(1, 1), tuple(2, 2), tuple(null, 3))
+          Seq.of(1, 2, 4).rightOuterJoin(Seq.of(1, 2, 3), Objects::equals).toList(),
+          Arrays.asList(tuple(1, 1), tuple(2, 2), tuple(null, 3))
         );
 
         assertEquals(
-                Seq.of(1, 2).crossJoin(Seq.of("A", "B")).toList(),
-                Arrays.asList(tuple(1, "A"), tuple(1, "B"), tuple(2, "A"), tuple(2, "B"))
+          Seq.of(1, 2).crossJoin(Seq.of("A", "B")).toList(),
+          Arrays.asList(tuple(1, "A"), tuple(1, "B"), tuple(2, "A"), tuple(2, "B"))
         );
     }
 
     @Test
     public void givenSeq_whenManipulateSeq_seqShouldHaveNewElementsInIt() {
         assertEquals(
-                Seq.of(1, 2, 3).cycle().limit(9).toList(),
-                Arrays.asList(1, 2, 3, 1, 2, 3, 1, 2, 3)
+          Seq.of(1, 2, 3).cycle().limit(9).toList(),
+          Arrays.asList(1, 2, 3, 1, 2, 3, 1, 2, 3)
         );
 
         assertEquals(
-                Seq.of(1, 2, 3).duplicate().map((first, second) -> tuple(first.toList(), second.toList())),
-                tuple(Arrays.asList(1, 2, 3), Arrays.asList(1, 2, 3))
+          Seq.of(1, 2, 3).duplicate().map((first, second) -> tuple(first.toList(), second.toList())),
+          tuple(Arrays.asList(1, 2, 3), Arrays.asList(1, 2, 3))
         );
 
         assertEquals(
-                Seq.of(1, 2, 3, 4).intersperse(0).toList(),
-                Arrays.asList(1, 0, 2, 0, 3, 0, 4)
+          Seq.of(1, 2, 3, 4).intersperse(0).toList(),
+          Arrays.asList(1, 0, 2, 0, 3, 0, 4)
         );
 
         assertEquals(
-                Seq.of(1, 2, 3, 4, 5).shuffle().toList().size(),
-                5
+          Seq.of(1, 2, 3, 4, 5).shuffle().toList().size(),
+          5
         );
 
         assertEquals(
-                Seq.of(1, 2, 3, 4).partition(i -> i > 2).map((first, second) -> tuple(first.toList(), second.toList())),
-                tuple(Arrays.asList(3, 4), Arrays.asList(1, 2))
+          Seq.of(1, 2, 3, 4).partition(i -> i > 2).map((first, second) -> tuple(first.toList(), second.toList())),
+          tuple(Arrays.asList(3, 4), Arrays.asList(1, 2))
 
         );
 
         assertEquals(
-                Seq.of(1, 2, 3, 4).reverse().toList(),
-                Arrays.asList(4, 3, 2, 1)
+          Seq.of(1, 2, 3, 4).reverse().toList(),
+          Arrays.asList(4, 3, 2, 1)
         );
     }
 
@@ -117,20 +118,20 @@ public class JOOLUnitTest {
         expectedAfterGroupBy.put(0, Arrays.asList(2, 4));
 
         assertEquals(
-                Seq.of(1, 2, 3, 4).groupBy(i -> i % 2),
-                expectedAfterGroupBy
+          Seq.of(1, 2, 3, 4).groupBy(i -> i % 2),
+          expectedAfterGroupBy
         );
 
 
         assertEquals(
-                Seq.of("a", "b", "c").foldLeft("!", (u, t) -> u + t),
-                "!abc"
+          Seq.of("a", "b", "c").foldLeft("!", (u, t) -> u + t),
+          "!abc"
         );
 
 
         assertEquals(
-                Seq.of("a", "b", "c").foldRight("!", (t, u) -> t + u),
-                "abc!"
+          Seq.of("a", "b", "c").foldRight("!", (t, u) -> t + u),
+          "abc!"
         );
     }
 
@@ -138,13 +139,13 @@ public class JOOLUnitTest {
     public void givenSeq_whenUsingSeqWhile_shouldBehaveAsWhileLoop() {
 
         assertEquals(
-                Seq.of(1, 2, 3, 4, 5).skipWhile(i -> i < 3).toList(),
-                Arrays.asList(3, 4, 5)
+          Seq.of(1, 2, 3, 4, 5).skipWhile(i -> i < 3).toList(),
+          Arrays.asList(3, 4, 5)
         );
 
         assertEquals(
-                Seq.of(1, 2, 3, 4, 5).skipUntil(i -> i == 3).toList(),
-                Arrays.asList(3, 4, 5)
+          Seq.of(1, 2, 3, 4, 5).skipUntil(i -> i == 3).toList(),
+          Arrays.asList(3, 4, 5)
         );
     }
 
@@ -152,19 +153,19 @@ public class JOOLUnitTest {
     public void givenSeq_whenZip_shouldHaveZippedSeq() {
 
         assertEquals(
-                Seq.of(1, 2, 3).zip(Seq.of("a", "b", "c")).toList(),
-                Arrays.asList(tuple(1, "a"), tuple(2, "b"), tuple(3, "c"))
+          Seq.of(1, 2, 3).zip(Seq.of("a", "b", "c")).toList(),
+          Arrays.asList(tuple(1, "a"), tuple(2, "b"), tuple(3, "c"))
         );
 
         assertEquals(
-                Seq.of(1, 2, 3).zip(Seq.of("a", "b", "c"), (x, y) -> x + ":" + y).toList(),
-                Arrays.asList("1:a", "2:b", "3:c")
+          Seq.of(1, 2, 3).zip(Seq.of("a", "b", "c"), (x, y) -> x + ":" + y).toList(),
+          Arrays.asList("1:a", "2:b", "3:c")
         );
 
 
         assertEquals(
-                Seq.of("a", "b", "c").zipWithIndex().toList(),
-                Arrays.asList(tuple("a", 0L), tuple("b", 1L), tuple("c", 2L))
+          Seq.of("a", "b", "c").zipWithIndex().toList(),
+          Arrays.asList(tuple("a", 0L), tuple("b", 1L), tuple("c", 2L))
         );
     }
 
@@ -187,8 +188,8 @@ public class JOOLUnitTest {
 
         //then
         assertEquals(
-                collect,
-                Arrays.asList(1, 1, 1)
+          collect,
+          Arrays.asList(1, 1, 1)
         );
     }
 
@@ -197,13 +198,13 @@ public class JOOLUnitTest {
     public void givenOperationThatThrowsCheckedException_whenExecuteUsingUncheckedFuction_shouldPass() {
         //when
         List<Integer> collect = Stream.of("a", "b", "c")
-                .map(Unchecked.function(elem -> methodThatThrowsChecked(elem)))
-                .collect(Collectors.toList());
+          .map(Unchecked.function(this::methodThatThrowsChecked))
+          .collect(Collectors.toList());
 
         //then
         assertEquals(
-                collect,
-                Arrays.asList(1, 1, 1)
+          collect,
+          Arrays.asList(1, 1, 1)
         );
     }
 
@@ -236,5 +237,4 @@ public class JOOLUnitTest {
           Arrays.asList(tuple("michael", "similar", "winter", "summer"), tuple("jodie", "variable", "winter", "summer"))
         );
     }
-
 }
diff --git a/pom.xml b/pom.xml
index 3feba96d5a..82dccf832d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -83,7 +83,6 @@
         <!--<module>jee7</module>-->
         <!-- <module>jhipster/jhipster-monolithic</module> -->
         <module>jjwt</module>
-        <module>jooq</module>
         <module>jpa-storedprocedure</module>
         <module>jsf</module>
         <module>json-path</module>