Resolved PMD issues

1) Refactored LotteryNumbers to use Joiner from guava library to join lottery numbers. 2) Solved potential thread safety issue in LotteryTicketId class, where it was using raw primitive value and incrementing it which is not thread-safe. So used AtomicInteger for brevity 3) assertEquals arguments were in incorrect order at many places, so changed order of those 4) Replaced assertFalse and assertTrue at some places with assertEquals and assertNotEquals for reducing complexity of code 5) Removed public modifiers from test cases, as they are no more needed by JUnit 5
Remove CII best practices badge
2018-10-15 13:30:20 +05:30 · 2018-10-15 13:21:00 +05:30 · 2018-10-13 18:40:10 +03:00 · 2018-10-13 18:28:40 +03:00 · 2018-09-30 21:36:32 +03:00 · 2018-09-30 21:05:57 +03:00
140 changed files with 202 additions and 268 deletions
--- a/README.md
+++ b/README.md
@ -8,7 +8,6 @@
 [![License MIT](https://img.shields.io/badge/license-MIT-blue.svg)](https://raw.githubusercontent.com/iluwatar/java-design-patterns/master/LICENSE.md)
 [![Join the chat at https://gitter.im/iluwatar/java-design-patterns](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/iluwatar/java-design-patterns?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 [![Quality Gate](https://sonarcloud.io/api/project_badges/measure?project=com.iluwatar%3Ajava-design-patterns&metric=alert_status)](https://sonarcloud.io/dashboard/index/com.iluwatar%3Ajava-design-patterns)
-[![CII Best Practices](https://bestpractices.coreinfrastructure.org/projects/1503/badge)](https://bestpractices.coreinfrastructure.org/projects/1503)

 # Introduction

--- a/abstract-document/pom.xml
+++ b/abstract-document/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <artifactId>java-design-patterns</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <artifactId>abstract-document</artifactId>
    <dependencies>
--- a/abstract-factory/pom.xml
+++ b/abstract-factory/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>abstract-factory</artifactId>
  <dependencies>
--- a/acyclic-visitor/pom.xml
+++ b/acyclic-visitor/pom.xml
@ -30,7 +30,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	
 	<artifactId>acyclic-visitor</artifactId>
--- a/adapter/pom.xml
+++ b/adapter/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>adapter</artifactId>
  <dependencies>
--- a/aggregator-microservices/aggregator-service/pom.xml
+++ b/aggregator-microservices/aggregator-service/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <artifactId>aggregator-microservices</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>

--- a/aggregator-microservices/information-microservice/pom.xml
+++ b/aggregator-microservices/information-microservice/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <artifactId>aggregator-microservices</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>

--- a/aggregator-microservices/inventory-microservice/pom.xml
+++ b/aggregator-microservices/inventory-microservice/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <artifactId>aggregator-microservices</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>

--- a/aggregator-microservices/pom.xml
+++ b/aggregator-microservices/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <artifactId>java-design-patterns</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>
    <artifactId>aggregator-microservices</artifactId>
--- a/ambassador/pom.xml
+++ b/ambassador/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <artifactId>java-design-patterns</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <modelVersion>4.0.0</modelVersion>
  <artifactId>ambassador</artifactId>
--- a/api-gateway/api-gateway-service/pom.xml
+++ b/api-gateway/api-gateway-service/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <artifactId>api-gateway</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <modelVersion>4.0.0</modelVersion>
  <artifactId>api-gateway-service</artifactId>
--- a/api-gateway/image-microservice/pom.xml
+++ b/api-gateway/image-microservice/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <artifactId>api-gateway</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>

  <modelVersion>4.0.0</modelVersion>
--- a/api-gateway/pom.xml
+++ b/api-gateway/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <artifactId>java-design-patterns</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <modelVersion>4.0.0</modelVersion>
  <artifactId>api-gateway</artifactId>
--- a/api-gateway/price-microservice/pom.xml
+++ b/api-gateway/price-microservice/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <artifactId>api-gateway</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>

  <modelVersion>4.0.0</modelVersion>
--- a/async-method-invocation/pom.xml
+++ b/async-method-invocation/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>async-method-invocation</artifactId>
  <dependencies>
--- a/balking/pom.xml
+++ b/balking/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <artifactId>java-design-patterns</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <modelVersion>4.0.0</modelVersion>

--- a/bridge/pom.xml
+++ b/bridge/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>bridge</artifactId>
  <dependencies>
--- a/builder/pom.xml
+++ b/builder/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>builder</artifactId>
  <dependencies>
--- a/business-delegate/pom.xml
+++ b/business-delegate/pom.xml
@ -30,7 +30,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>business-delegate</artifactId>
 	<dependencies>
--- a/caching/pom.xml
+++ b/caching/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>caching</artifactId>
  <dependencies>
--- a/callback/pom.xml
+++ b/callback/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>callback</artifactId>
  <dependencies>
--- a/chain/pom.xml
+++ b/chain/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>chain</artifactId>
  <dependencies>
--- a/collection-pipeline/pom.xml
+++ b/collection-pipeline/pom.xml
@ -27,7 +27,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>collection-pipeline</artifactId>
  <dependencies>
--- a/command/pom.xml
+++ b/command/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>command</artifactId>
  <dependencies>
--- a/composite/pom.xml
+++ b/composite/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>composite</artifactId>
  <dependencies>
--- a/converter/pom.xml
+++ b/converter/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <artifactId>java-design-patterns</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>
    <dependencies>
--- a/cqrs/pom.xml
+++ b/cqrs/pom.xml
@ -21,7 +21,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>cqrs</artifactId>
 	<dependencies>
--- a/dao/pom.xml
+++ b/dao/pom.xml
@ -30,7 +30,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>dao</artifactId>

--- a/data-bus/pom.xml
+++ b/data-bus/pom.xml
@ -33,7 +33,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <artifactId>data-bus</artifactId>
    <dependencies>
--- a/data-mapper/pom.xml
+++ b/data-mapper/pom.xml
@ -28,7 +28,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>data-mapper</artifactId>
 	<dependencies>
--- a/data-transfer-object/pom.xml
+++ b/data-transfer-object/pom.xml
@ -28,7 +28,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <artifactId>data-transfer-object</artifactId>
    <dependencies>
--- a/decorator/pom.xml
+++ b/decorator/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>decorator</artifactId>
  <dependencies>
--- a/delegation/pom.xml
+++ b/delegation/pom.xml
@ -30,7 +30,7 @@
    <parent>
        <artifactId>java-design-patterns</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>

--- a/dependency-injection/pom.xml
+++ b/dependency-injection/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>dependency-injection</artifactId>
  <dependencies>
--- a/dirty-flag/pom.xml
+++ b/dirty-flag/pom.xml
@ -29,11 +29,11 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <groupId>com.iluwatar</groupId>
  <artifactId>dirty-flag</artifactId>
-  <version>1.20.0-SNAPSHOT</version>
+  <version>1.21.0-SNAPSHOT</version>
  <name>dirty-flag</name>
  <url>http://maven.apache.org</url>
  <properties>
--- a/double-checked-locking/pom.xml
+++ b/double-checked-locking/pom.xml
@ -27,7 +27,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>double-checked-locking</artifactId>
  <dependencies>
--- a/double-dispatch/pom.xml
+++ b/double-dispatch/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>double-dispatch</artifactId>
  <dependencies>
--- a/eip-aggregator/pom.xml
+++ b/eip-aggregator/pom.xml
@ -26,7 +26,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>

    <dependencies>
--- a/eip-message-channel/pom.xml
+++ b/eip-message-channel/pom.xml
@ -30,7 +30,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>eip-message-channel</artifactId>
 	<dependencies>
--- a/eip-publish-subscribe/pom.xml
+++ b/eip-publish-subscribe/pom.xml
@ -28,7 +28,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>eip-publish-subscribe</artifactId>
 	<dependencies>
--- a/eip-splitter/pom.xml
+++ b/eip-splitter/pom.xml
@ -26,7 +26,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>

    <dependencies>
--- a/eip-wire-tap/pom.xml
+++ b/eip-wire-tap/pom.xml
@ -26,7 +26,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>

    <dependencies>
--- a/event-aggregator/pom.xml
+++ b/event-aggregator/pom.xml
@ -28,7 +28,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>event-aggregator</artifactId>
 	<dependencies>
--- a/event-asynchronous/pom.xml
+++ b/event-asynchronous/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>event-asynchronous</artifactId>
  <dependencies>
--- a/event-driven-architecture/pom.xml
+++ b/event-driven-architecture/pom.xml
@ -31,7 +31,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>

    <artifactId>event-driven-architecture</artifactId>
--- a/event-queue/pom.xml
+++ b/event-queue/pom.xml
@ -30,7 +30,7 @@
  <parent>
    <artifactId>java-design-patterns</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>event-queue</artifactId>
  <dependencies>
--- a/event-sourcing/pom.xml
+++ b/event-sourcing/pom.xml
@ -30,7 +30,7 @@
  <parent>
    <artifactId>java-design-patterns</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>event-sourcing</artifactId>
  <dependencies>
--- a/execute-around/pom.xml
+++ b/execute-around/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>execute-around</artifactId>
  <dependencies>
--- a/extension-objects/pom.xml
+++ b/extension-objects/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <artifactId>java-design-patterns</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <modelVersion>4.0.0</modelVersion>

--- a/facade/pom.xml
+++ b/facade/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>facade</artifactId>
  <dependencies>
--- a/factory-kit/pom.xml
+++ b/factory-kit/pom.xml
@ -30,7 +30,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <artifactId>factory-kit</artifactId>
    <dependencies>
--- a/factory-method/pom.xml
+++ b/factory-method/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>factory-method</artifactId>
  <dependencies>
--- a/feature-toggle/pom.xml
+++ b/feature-toggle/pom.xml
@ -30,7 +30,7 @@
    <parent>
        <artifactId>java-design-patterns</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>

--- a/fluentinterface/pom.xml
+++ b/fluentinterface/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <artifactId>java-design-patterns</artifactId>
        <groupId>com.iluwatar</groupId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <modelVersion>4.0.0</modelVersion>

--- a/flux/pom.xml
+++ b/flux/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>flux</artifactId>
  <dependencies>
--- a/flyweight/pom.xml
+++ b/flyweight/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>flyweight</artifactId>
  <dependencies>
--- a/front-controller/pom.xml
+++ b/front-controller/pom.xml
@ -30,7 +30,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>front-controller</artifactId>
 	<dependencies>
--- a/guarded-suspension/pom.xml
+++ b/guarded-suspension/pom.xml
@ -30,7 +30,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <packaging>jar</packaging>
  <artifactId>guarded-suspension</artifactId>
--- a/half-sync-half-async/pom.xml
+++ b/half-sync-half-async/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>half-sync-half-async</artifactId>
  <dependencies>
--- a/hexagonal/pom.xml
+++ b/hexagonal/pom.xml
@ -30,7 +30,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>hexagonal</artifactId>
  <dependencies>
--- a/hexagonal/src/main/java/com/iluwatar/hexagonal/database/MongoTicketRepository.java
+++ b/hexagonal/src/main/java/com/iluwatar/hexagonal/database/MongoTicketRepository.java
@ -110,21 +110,6 @@ public class MongoTicketRepository implements LotteryTicketRepository {
    return result.getInteger("seq");
  }

-  /**
-   * @return mongo client
-   */
-  public MongoClient getMongoClient() {
-    return mongoClient;
-  }
-
-  /**
-   *
-   * @return mongo database
-   */
-  public MongoDatabase getMongoDatabase() {
-    return database;
-  }
-
  /**
   *
   * @return tickets collection
--- a/hexagonal/src/main/java/com/iluwatar/hexagonal/domain/LotteryNumbers.java
+++ b/hexagonal/src/main/java/com/iluwatar/hexagonal/domain/LotteryNumbers.java
@ -22,10 +22,10 @@
 */
 package com.iluwatar.hexagonal.domain;

-import java.util.ArrayList;
+import com.google.common.base.Joiner;
+
 import java.util.Collections;
 import java.util.HashSet;
-import java.util.List;
 import java.util.PrimitiveIterator;
 import java.util.Random;
 import java.util.Set;
@ -85,16 +85,7 @@ public class LotteryNumbers {
   * @return numbers as comma separated string
   */
  public String getNumbersAsString() {
-    List<Integer> list = new ArrayList<>();
-    list.addAll(numbers);
-    StringBuilder builder = new StringBuilder();
-    for (int i = 0; i < NUM_NUMBERS; i++) {
-      builder.append(list.get(i));
-      if (i < NUM_NUMBERS - 1) {
-        builder.append(",");
-      }
-    }
-    return builder.toString();
+    return Joiner.on(',').join(numbers);
  }
  
  /**
--- a/hexagonal/src/main/java/com/iluwatar/hexagonal/domain/LotteryTicketId.java
+++ b/hexagonal/src/main/java/com/iluwatar/hexagonal/domain/LotteryTicketId.java
@ -22,17 +22,18 @@
 */
 package com.iluwatar.hexagonal.domain;

+import java.util.concurrent.atomic.AtomicInteger;
+
 /**
 * Lottery ticked id
 */
 public class LotteryTicketId {

-  private static volatile int numAllocated;
+  private static AtomicInteger numAllocated = new AtomicInteger(0);
  private final int id;
  
  public LotteryTicketId() {
-    this.id = numAllocated + 1;
-    numAllocated++;
+    this.id = numAllocated.incrementAndGet();
  }

  public LotteryTicketId(int id) {
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/AppTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/AppTest.java
@ -27,10 +27,10 @@ import org.junit.jupiter.api.Test;
 /**
 * Unit test for simple App.
 */
-public class AppTest {
+class AppTest {

  @Test
-  public void testApp() {
+  void testApp() {
    String[] args = {};
    App.main(args);
  }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/InMemoryBankTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/InMemoryBankTest.java
@ -32,19 +32,19 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 * Tests for banking
 *
 */
-public class InMemoryBankTest {
+class InMemoryBankTest {

  private final WireTransfers bank = new InMemoryBank();
  
  @Test
-  public void testInit() {
-    assertEquals(bank.getFunds("foo"), 0);
+  void testInit() {
+    assertEquals(0, bank.getFunds("foo"));
    bank.setFunds("foo", 100);
-    assertEquals(bank.getFunds("foo"), 100);
+    assertEquals(100, bank.getFunds("foo"));
    bank.setFunds("bar", 150);
-    assertEquals(bank.getFunds("bar"), 150);
+    assertEquals(150, bank.getFunds("bar"));
    assertTrue(bank.transferFunds(50, "bar", "foo"));
-    assertEquals(bank.getFunds("foo"), 150);
-    assertEquals(bank.getFunds("bar"), 100);
+    assertEquals(150, bank.getFunds("foo"));
+    assertEquals(100, bank.getFunds("bar"));
  }
 }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/MongoBankTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/banking/MongoBankTest.java
@ -34,7 +34,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
 * Tests for Mongo banking adapter
 */
@Disabled
-public class MongoBankTest {
+class MongoBankTest {

  private static final String TEST_DB = "lotteryDBTest";
  private static final String TEST_ACCOUNTS_COLLECTION = "testAccounts";
@ -42,7 +42,7 @@ public class MongoBankTest {
  private MongoBank mongoBank;

  @BeforeEach
-  public void init() {
+  void init() {
    MongoConnectionPropertiesLoader.load();
    MongoClient mongoClient = new MongoClient(System.getProperty("mongo-host"),
        Integer.parseInt(System.getProperty("mongo-port")));
@ -52,12 +52,12 @@ public class MongoBankTest {
  }

  @Test
-  public void testSetup() {
+  void testSetup() {
    assertEquals(0, mongoBank.getAccountsCollection().count());
  }

  @Test
-  public void testFundTransfers() {
+  void testFundTransfers() {
    assertEquals(0, mongoBank.getFunds("000-000"));
    mongoBank.setFunds("000-000", 10);
    assertEquals(10, mongoBank.getFunds("000-000"));
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/database/InMemoryTicketRepositoryTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/database/InMemoryTicketRepositoryTest.java
@ -38,23 +38,23 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 * Tests for {@link LotteryTicketRepository}
 *
 */
-public class InMemoryTicketRepositoryTest {
+class InMemoryTicketRepositoryTest {

  private final LotteryTicketRepository repository = new InMemoryTicketRepository();
  
  @BeforeEach
-  public void clear() {
+  void clear() {
    repository.deleteAll();
  }
  
  @Test
-  public void testCrudOperations() {
+  void testCrudOperations() {
    LotteryTicketRepository repository = new InMemoryTicketRepository();
-    assertEquals(repository.findAll().size(), 0);
+    assertTrue(repository.findAll().isEmpty());
    LotteryTicket ticket = LotteryTestUtils.createLotteryTicket();
    Optional<LotteryTicketId> id = repository.save(ticket);
    assertTrue(id.isPresent());
-    assertEquals(repository.findAll().size(), 1);
+    assertEquals(1, repository.findAll().size());
    Optional<LotteryTicket> optionalTicket = repository.findById(id.get());
    assertTrue(optionalTicket.isPresent());
  }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/database/MongoTicketRepositoryTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/database/MongoTicketRepositoryTest.java
@ -41,7 +41,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 * Tests for Mongo based ticket repository
 */
@Disabled
-public class MongoTicketRepositoryTest {
+class MongoTicketRepositoryTest {

  private static final String TEST_DB = "lotteryTestDB";
  private static final String TEST_TICKETS_COLLECTION = "lotteryTestTickets";
@ -50,7 +50,7 @@ public class MongoTicketRepositoryTest {
  private MongoTicketRepository repository;

  @BeforeEach
-  public void init() {
+  void init() {
    MongoConnectionPropertiesLoader.load();
    MongoClient mongoClient = new MongoClient(System.getProperty("mongo-host"),
        Integer.parseInt(System.getProperty("mongo-port")));
@ -61,20 +61,20 @@ public class MongoTicketRepositoryTest {
  }

  @Test
-  public void testSetup() {
-    assertTrue(repository.getCountersCollection().count() == 1);
-    assertTrue(repository.getTicketsCollection().count() == 0);
+  void testSetup() {
+    assertEquals(1, repository.getCountersCollection().count());
+    assertEquals(0, repository.getTicketsCollection().count());
  }

  @Test
-  public void testNextId() {
+  void testNextId() {
    assertEquals(1, repository.getNextId());
    assertEquals(2, repository.getNextId());
    assertEquals(3, repository.getNextId());
  }

  @Test
-  public void testCrudOperations() {
+  void testCrudOperations() {
    // create new lottery ticket and save it
    PlayerDetails details = new PlayerDetails("foo@bar.com", "123-123", "07001234");
    LotteryNumbers random = LotteryNumbers.createRandom();
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryNumbersTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryNumbersTest.java
@ -28,7 +28,7 @@ import java.util.Arrays;
 import java.util.HashSet;

 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;

@ -37,10 +37,10 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 * Unit tests for {@link LotteryNumbers}
 *
 */
-public class LotteryNumbersTest {
+class LotteryNumbersTest {
  
  @Test
-  public void testGivenNumbers() {
+  void testGivenNumbers() {
    LotteryNumbers numbers = LotteryNumbers.create(
            new HashSet<>(Arrays.asList(1, 2, 3, 4)));
    assertEquals(numbers.getNumbers().size(), 4);
@ -51,7 +51,7 @@ public class LotteryNumbersTest {
  }
  
  @Test
-  public void testNumbersCantBeModified() {
+  void testNumbersCantBeModified() {
    LotteryNumbers numbers = LotteryNumbers.create(
            new HashSet<>(Arrays.asList(1, 2, 3, 4)));
    assertThrows(UnsupportedOperationException.class, () -> {
@ -60,20 +60,20 @@ public class LotteryNumbersTest {
  }
  
  @Test
-  public void testRandomNumbers() {
+  void testRandomNumbers() {
    LotteryNumbers numbers = LotteryNumbers.createRandom();
    assertEquals(numbers.getNumbers().size(), LotteryNumbers.NUM_NUMBERS);
  }
  
  @Test
-  public void testEquals() {
+  void testEquals() {
    LotteryNumbers numbers1 = LotteryNumbers.create(
            new HashSet<>(Arrays.asList(1, 2, 3, 4)));
    LotteryNumbers numbers2 = LotteryNumbers.create(
            new HashSet<>(Arrays.asList(1, 2, 3, 4)));
-    assertTrue(numbers1.equals(numbers2));
+    assertEquals(numbers1, numbers2);
    LotteryNumbers numbers3 = LotteryNumbers.create(
            new HashSet<>(Arrays.asList(11, 12, 13, 14)));
-    assertFalse(numbers1.equals(numbers3));
+    assertNotEquals(numbers1, numbers3);
  }
 }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTest.java
@ -38,6 +38,7 @@ import java.util.Map;
 import java.util.Optional;

 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;

 /**
@ -45,7 +46,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 * Test the lottery system
 *
 */
-public class LotteryTest {
+class LotteryTest {

  private Injector injector;
  @Inject
@ -55,22 +56,22 @@ public class LotteryTest {
  @Inject
  private WireTransfers wireTransfers;

-  public LotteryTest() {
+  LotteryTest() {
    this.injector = Guice.createInjector(new LotteryTestingModule());
  }

  @BeforeEach
-  public void setup() {
+  void setup() {
    injector.injectMembers(this);
    // add funds to the test player's bank account
    wireTransfers.setFunds("123-12312", 100);
  }
  
  @Test
-  public void testLottery() {
+  void testLottery() {
    // admin resets the lottery
    administration.resetLottery();
-    assertEquals(administration.getAllSubmittedTickets().size(), 0);
+    assertEquals(0, administration.getAllSubmittedTickets().size());
    
    // players submit the lottery tickets
    Optional<LotteryTicketId> ticket1 = service.submitTicket(LotteryTestUtils.createLotteryTicket("cvt@bbb.com",
@ -82,7 +83,7 @@ public class LotteryTest {
    Optional<LotteryTicketId> ticket3 = service.submitTicket(LotteryTestUtils.createLotteryTicket("arg@boo.com",
        "123-12312", "+32421255", new HashSet<>(Arrays.asList(6, 8, 13, 19))));
    assertTrue(ticket3.isPresent());
-    assertEquals(administration.getAllSubmittedTickets().size(), 3);
+    assertEquals(3, administration.getAllSubmittedTickets().size());
    
    // perform lottery
    LotteryNumbers winningNumbers = administration.performLottery();
@ -91,23 +92,23 @@ public class LotteryTest {
    Optional<LotteryTicketId> ticket4 = service.submitTicket(LotteryTestUtils.createLotteryTicket("lucky@orb.com",
        "123-12312", "+12421255", winningNumbers.getNumbers()));
    assertTrue(ticket4.isPresent());
-    assertEquals(administration.getAllSubmittedTickets().size(), 4);
+    assertEquals(4, administration.getAllSubmittedTickets().size());
    
    // check winners
    Map<LotteryTicketId, LotteryTicket> tickets = administration.getAllSubmittedTickets();
    for (LotteryTicketId id: tickets.keySet()) {
      LotteryTicketCheckResult checkResult = service.checkTicketForPrize(id, winningNumbers);
-      assertTrue(checkResult.getResult() != CheckResult.TICKET_NOT_SUBMITTED);
+      assertNotEquals(CheckResult.TICKET_NOT_SUBMITTED, checkResult.getResult());
      if (checkResult.getResult().equals(CheckResult.WIN_PRIZE)) {
        assertTrue(checkResult.getPrizeAmount() > 0);
      } else if (checkResult.getResult().equals(CheckResult.WIN_PRIZE)) {
-        assertEquals(checkResult.getPrizeAmount(), 0);
+        assertEquals(0, checkResult.getPrizeAmount());
      }
    }
    
    // check another ticket that has not been submitted
    LotteryTicketCheckResult checkResult = service.checkTicketForPrize(new LotteryTicketId(), winningNumbers);
-    assertTrue(checkResult.getResult() == CheckResult.TICKET_NOT_SUBMITTED);
-    assertEquals(checkResult.getPrizeAmount(), 0);
+    assertEquals(CheckResult.TICKET_NOT_SUBMITTED, checkResult.getResult());
+    assertEquals(0, checkResult.getPrizeAmount());
  }
 }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTicketCheckResultTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTicketCheckResultTest.java
@ -26,21 +26,21 @@ import com.iluwatar.hexagonal.domain.LotteryTicketCheckResult.CheckResult;
 import org.junit.jupiter.api.Test;

 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;

 /**
 * 
 * Unit tests for {@link LotteryTicketCheckResult}
 *
 */
-public class LotteryTicketCheckResultTest {
+class LotteryTicketCheckResultTest {

  @Test
-  public void testEquals() {
+  void testEquals() {
    LotteryTicketCheckResult result1 = new LotteryTicketCheckResult(CheckResult.NO_PRIZE);
    LotteryTicketCheckResult result2 = new LotteryTicketCheckResult(CheckResult.NO_PRIZE);
    assertEquals(result1, result2);
    LotteryTicketCheckResult result3 = new LotteryTicketCheckResult(CheckResult.WIN_PRIZE, 300000);
-    assertFalse(result1.equals(result3));
+    assertNotEquals(result1, result3);
  } 
 }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTicketIdTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTicketIdTest.java
@ -24,22 +24,22 @@ package com.iluwatar.hexagonal.domain;

 import org.junit.jupiter.api.Test;

-import static org.junit.jupiter.api.Assertions.assertFalse;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;

 /**
 * Tests for lottery ticket id
 */
-public class LotteryTicketIdTest {
+class LotteryTicketIdTest {

  @Test
-  public void testEquals() {
+  void testEquals() {
    LotteryTicketId ticketId1 = new LotteryTicketId();
    LotteryTicketId ticketId2 = new LotteryTicketId();
    LotteryTicketId ticketId3 = new LotteryTicketId();
-    assertFalse(ticketId1.equals(ticketId2));
-    assertFalse(ticketId2.equals(ticketId3));
+    assertNotEquals(ticketId1, ticketId2);
+    assertNotEquals(ticketId2, ticketId3);
    LotteryTicketId ticketId4 = new LotteryTicketId(ticketId1.getId());
-    assertTrue(ticketId1.equals(ticketId4));
+    assertEquals(ticketId1, ticketId4);
  }
 }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTicketTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/LotteryTicketTest.java
@ -29,14 +29,15 @@ import java.util.HashSet;

 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;

 /**
 * Test Lottery Tickets for equality
 */
-public class LotteryTicketTest {
+class LotteryTicketTest {

  @Test
-  public void testEquals() {
+  void testEquals() {
    PlayerDetails details1 = new PlayerDetails("bob@foo.bar", "1212-121212", "+34332322");
    LotteryNumbers numbers1 = LotteryNumbers.create(new HashSet<Integer>(Arrays.asList(1, 2, 3, 4)));
    LotteryTicket ticket1 = new LotteryTicket(new LotteryTicketId(), details1, numbers1);
@ -47,6 +48,6 @@ public class LotteryTicketTest {
    PlayerDetails details3 = new PlayerDetails("elsa@foo.bar", "1223-121212", "+49332322");
    LotteryNumbers numbers3 = LotteryNumbers.create(new HashSet<Integer>(Arrays.asList(1, 2, 3, 8)));
    LotteryTicket ticket3 = new LotteryTicket(new LotteryTicketId(), details3, numbers3);
-    assertFalse(ticket1.equals(ticket3));
+    assertNotEquals(ticket1, ticket3);
  }
 }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/PlayerDetailsTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/domain/PlayerDetailsTest.java
@ -25,21 +25,21 @@ package com.iluwatar.hexagonal.domain;
 import org.junit.jupiter.api.Test;

 import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;

 /**
 * 
 * Unit tests for {@link PlayerDetails}
 *
 */
-public class PlayerDetailsTest {
+class PlayerDetailsTest {

  @Test
-  public void testEquals() {
+  void testEquals() {
    PlayerDetails details1 = new PlayerDetails("tom@foo.bar", "11212-123434", "+12323425");
    PlayerDetails details2 = new PlayerDetails("tom@foo.bar", "11212-123434", "+12323425");
    assertEquals(details1, details2);
    PlayerDetails details3 = new PlayerDetails("john@foo.bar", "16412-123439", "+34323432");
-    assertFalse(details1.equals(details3));
+    assertNotEquals(details1, details3);
  }  
 }
--- a/hexagonal/src/test/java/com/iluwatar/hexagonal/eventlog/MongoEventLogTest.java
+++ b/hexagonal/src/test/java/com/iluwatar/hexagonal/eventlog/MongoEventLogTest.java
@ -35,7 +35,7 @@ import static org.junit.jupiter.api.Assertions.assertEquals;
 * Tests for Mongo event log
 */
@Disabled
-public class MongoEventLogTest {
+class MongoEventLogTest {

  private static final String TEST_DB = "lotteryDBTest";
  private static final String TEST_EVENTS_COLLECTION = "testEvents";
@ -43,7 +43,7 @@ public class MongoEventLogTest {
  private MongoEventLog mongoEventLog;

  @BeforeEach
-  public void init() {
+  void init() {
    MongoConnectionPropertiesLoader.load();
    MongoClient mongoClient = new MongoClient(System.getProperty("mongo-host"),
        Integer.parseInt(System.getProperty("mongo-port")));
@ -53,12 +53,12 @@ public class MongoEventLogTest {
  }

  @Test
-  public void testSetup() {
+  void testSetup() {
    assertEquals(0, mongoEventLog.getEventsCollection().count());
  }

  @Test
-  public void testFundTransfers() {
+  void testFundTransfers() {
    PlayerDetails playerDetails = new PlayerDetails("john@wayne.com", "000-000", "03432534543");
    mongoEventLog.prizeError(playerDetails, 1000);
    assertEquals(1, mongoEventLog.getEventsCollection().count());
--- a/intercepting-filter/pom.xml
+++ b/intercepting-filter/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>intercepting-filter</artifactId>
  <dependencies>
--- a/interpreter/pom.xml
+++ b/interpreter/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>interpreter</artifactId>
  <dependencies>
--- a/iterator/pom.xml
+++ b/iterator/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>iterator</artifactId>
  <dependencies>
--- a/layers/pom.xml
+++ b/layers/pom.xml
@ -30,7 +30,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<groupId>com.iluwatar.layers</groupId>
 	<artifactId>layers</artifactId>
--- a/lazy-loading/pom.xml
+++ b/lazy-loading/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>lazy-loading</artifactId>
  <dependencies>
--- a/marker/pom.xml
+++ b/marker/pom.xml
@ -24,7 +24,7 @@
  <parent>
    <artifactId>java-design-patterns</artifactId>
    <groupId>com.iluwatar</groupId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <modelVersion>4.0.0</modelVersion>

--- a/mediator/pom.xml
+++ b/mediator/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>mediator</artifactId>
  <dependencies>
--- a/memento/pom.xml
+++ b/memento/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>memento</artifactId>
  <dependencies>
--- a/model-view-controller/pom.xml
+++ b/model-view-controller/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>model-view-controller</artifactId>
  <dependencies>
--- a/model-view-presenter/pom.xml
+++ b/model-view-presenter/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>model-view-presenter</artifactId>
  <name>model-view-presenter</name>
--- a/module/.gitignore
+++ b/module/.gitignore
@ -0,0 +1,2 @@
+error.txt
+output.txt
--- a/module/error.txt
+++ b/module/error.txt
@ -1,23 +0,0 @@
-====
-    The MIT License
-    Copyright (c) 2014 Ilkka Seppälä
-
-    Permission is hereby granted, free of charge, to any person obtaining a copy
-    of this software and associated documentation files (the "Software"), to deal
-    in the Software without restriction, including without limitation the rights
-    to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-    copies of the Software, and to permit persons to whom the Software is
-    furnished to do so, subject to the following conditions:
-
-    The above copyright notice and this permission notice shall be included in
-    all copies or substantial portions of the Software.
-
-    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-    AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-    THE SOFTWARE.
-====
-
--- a/module/output.txt
+++ b/module/output.txt
@ -1,23 +0,0 @@
-====
-    The MIT License
-    Copyright (c) 2014 Ilkka Seppälä
-
-    Permission is hereby granted, free of charge, to any person obtaining a copy
-    of this software and associated documentation files (the "Software"), to deal
-    in the Software without restriction, including without limitation the rights
-    to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-    copies of the Software, and to permit persons to whom the Software is
-    furnished to do so, subject to the following conditions:
-
-    The above copyright notice and this permission notice shall be included in
-    all copies or substantial portions of the Software.
-
-    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-    IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-    AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-    LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-    THE SOFTWARE.
-====
-
--- a/module/pom.xml
+++ b/module/pom.xml
@ -28,7 +28,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>module</artifactId>
 	<dependencies>
--- a/monad/pom.xml
+++ b/monad/pom.xml
@ -29,7 +29,7 @@
    <parent>
        <groupId>com.iluwatar</groupId>
        <artifactId>java-design-patterns</artifactId>
-        <version>1.20.0-SNAPSHOT</version>
+        <version>1.21.0-SNAPSHOT</version>
    </parent>
    <artifactId>monad</artifactId>
    <dependencies>
--- a/monostate/pom.xml
+++ b/monostate/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>monostate</artifactId>
  <dependencies>
--- a/multiton/pom.xml
+++ b/multiton/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>multiton</artifactId>
  <dependencies>
--- a/mute-idiom/pom.xml
+++ b/mute-idiom/pom.xml
@ -21,7 +21,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>java-design-patterns</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>
 	<artifactId>mute-idiom</artifactId>
 	<dependencies>
--- a/mutex/pom.xml
+++ b/mutex/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>mutex</artifactId>
  <dependencies>
--- a/naked-objects/dom/pom.xml
+++ b/naked-objects/dom/pom.xml
@ -16,7 +16,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>naked-objects</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>

 	<artifactId>naked-objects-dom</artifactId>
--- a/naked-objects/fixture/pom.xml
+++ b/naked-objects/fixture/pom.xml
@ -16,7 +16,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>naked-objects</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>

 	<artifactId>naked-objects-fixture</artifactId>
--- a/naked-objects/integtests/pom.xml
+++ b/naked-objects/integtests/pom.xml
@ -16,7 +16,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>naked-objects</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>

 	<artifactId>naked-objects-integtests</artifactId>
--- a/naked-objects/pom.xml
+++ b/naked-objects/pom.xml
@ -15,7 +15,7 @@
 	<parent>
 		<artifactId>java-design-patterns</artifactId>
 		<groupId>com.iluwatar</groupId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>

 	<artifactId>naked-objects</artifactId>
@ -350,17 +350,17 @@
 			<dependency>
 				<groupId>${project.groupId}</groupId>
 				<artifactId>naked-objects-dom</artifactId>
-				<version>1.20.0-SNAPSHOT</version>
+				<version>1.21.0-SNAPSHOT</version>
 			</dependency>
 			<dependency>
 				<groupId>${project.groupId}</groupId>
 				<artifactId>naked-objects-fixture</artifactId>
-				<version>1.20.0-SNAPSHOT</version>
+				<version>1.21.0-SNAPSHOT</version>
 			</dependency>
 			<dependency>
 				<groupId>${project.groupId}</groupId>
 				<artifactId>naked-objects-webapp</artifactId>
-				<version>1.20.0-SNAPSHOT</version>
+				<version>1.21.0-SNAPSHOT</version>
 			</dependency>

 		</dependencies>
--- a/naked-objects/webapp/pom.xml
+++ b/naked-objects/webapp/pom.xml
@ -16,7 +16,7 @@
 	<parent>
 		<groupId>com.iluwatar</groupId>
 		<artifactId>naked-objects</artifactId>
-		<version>1.20.0-SNAPSHOT</version>
+		<version>1.21.0-SNAPSHOT</version>
 	</parent>

 	<artifactId>naked-objects-webapp</artifactId>
--- a/null-object/pom.xml
+++ b/null-object/pom.xml
@ -29,7 +29,7 @@
  <parent>
    <groupId>com.iluwatar</groupId>
    <artifactId>java-design-patterns</artifactId>
-    <version>1.20.0-SNAPSHOT</version>
+    <version>1.21.0-SNAPSHOT</version>
  </parent>
  <artifactId>null-object</artifactId>
  <dependencies>
--- a/Show More
+++ b/Show More
Author	SHA1	Message	Date
Narendra Pathai	37ae182630	Resolved PMD issues	2018-10-15 13:30:20 +05:30
Narendra Pathai	ab2c12e734	1) Refactored LotteryNumbers to use Joiner from guava library to join lottery numbers. 2) Solved potential thread safety issue in LotteryTicketId class, where it was using raw primitive value and incrementing it which is not thread-safe. So used AtomicInteger for brevity 3) assertEquals arguments were in incorrect order at many places, so changed order of those 4) Replaced assertFalse and assertTrue at some places with assertEquals and assertNotEquals for reducing complexity of code 5) Removed public modifiers from test cases, as they are no more needed by JUnit 5	2018-10-15 13:21:00 +05:30
Ilkka Seppälä	db33cc533b	Remove CII best practices badge	2018-10-13 18:40:10 +03:00
Ilkka Seppälä	8433c7b712	Merge pull request #793 from andrievsky/master Fix redundant list alloc in LotteryNumbers	2018-10-13 18:28:40 +03:00
Ilkka Seppälä	ee74fec53c	Fix pattern title	2018-09-30 21:36:32 +03:00
Ilkka Seppälä	7a7e891384	Set version for next development iteration	2018-09-30 21:05:57 +03:00
Ilkka Seppälä	26d6d96f78	Reach milestone 1.20.0	2018-09-30 21:03:48 +03:00
Ilkka Seppälä	3cec7a9ec1	Merge pull request #796 from LyndonArmitage/module-ignore-test-output Delete & Ignore output.txt & error.txt	2018-09-25 22:49:25 +03:00
Ilkka Seppälä	b079aec1fd	Merge pull request #794 from iluwatar/Issue781 #781 Resolved ClasscastException from Acyclic Visitor	2018-09-25 22:45:25 +03:00
Lyndon Armitage	8f53df91b9	Delete & Ignore output.txt & error.txt Both output.txt and error.txt are produced by tests. Each file contained a header that was overridden upon executing said tests causing tracked files to be changed. Added them to a local .gitignore file for this module.	2018-09-24 10:08:02 +01:00
Nick Andrievsky	1d12d94bac	Fix redundant list alloc in LotteryNumbers	2018-09-11 11:55:53 +02:00