Move NPE check to factory method

pom.xml

@@ -89,29 +89,13 @@
 			<artifactId>powermock-api-easymock</artifactId>
 			<version>2.0.2</version>
 			<scope>test</scope>
-			<exclusions>
-				<exclusion>
-					<groupId>org.easymock</groupId>
-					<artifactId>easymock</artifactId>
-				</exclusion>
-				<exclusion>
-					<groupId>cglib</groupId>
-					<artifactId>cglib-nodep</artifactId>
-				</exclusion>
-			</exclusions>
 		</dependency>
 		<dependency>
 			<groupId>org.easymock</groupId>
 			<artifactId>easymock</artifactId>
 			<version>4.0.2</version>
 			<scope>test</scope>
 		</dependency>
-		<dependency>
-			<groupId>cglib</groupId>
-			<artifactId>cglib-nodep</artifactId>
-			<version>3.3.0</version>
-			<scope>test</scope>
-		</dependency>
 	</dependencies>
 
 	<build>

src/main/java/org/fabri1983/refactorexample/joblogger/enhanced/factory/JobLoggerFactory.java

@@ -25,6 +25,9 @@ public static IEnhancedJobLogger newFileJobLogger(File file) {
 	}
 
 	public static IEnhancedJobLogger newDatabaseJobLogger(Connection connection) {
+		if (connection == null) {
+			throw new JobLoggerException(JobLoggerExceptionMessage.CONNECTION_IS_MISSING);
+		}
 		return new DatabaseJobLogger(connection);
 	}
 

src/main/java/org/fabri1983/refactorexample/joblogger/enhanced/implementation/DatabaseJobLogger.java

@@ -20,9 +20,6 @@ public DatabaseJobLogger(Connection connection) {
 	}
 
 	protected void validateConnection(Connection connection) {
-		if (connection == null) {
-			throw new JobLoggerException(JobLoggerExceptionMessage.CONNECTION_IS_MISSING);
-		}
 		try {
 			if (connection.isClosed()) {
 				throw new JobLoggerException(JobLoggerExceptionMessage.CONNECTION_IS_CLOSED);