Documentation: added configuration in logger table

Code: Some small optimizations.

Author: swa-ds

README.md

@@ -69,8 +69,8 @@ Die in der folgenden Tabelle gelisteten Logger-Klassen sollen erstellt werden. U
 Implementierung nicht in eine tatsächliche Datei oder Datenbank loggen. Stattdessen wird jeweils die Nachricht
 inklusive eines entsprechenden Suffix nach `System.out` geschrieben.
 
-| Logger-Klasse | Funktion                | Bemerkung |
-|---------------|-------------------------|-----------|
-| FileLogger    | Loggt in eine Datei     | Schreibt `nachricht + " (in eine Datei geloggt)"` nach `System.out`. |
-| DbLogger      | Loggt in eine Datenbank | Schreibt `nachricht + " (in eine DB geloggt)"` nach `System.out`. |
-| SilentLogger  | Macht gar nichts        | Ignoriert die Nachricht und schreibt nichts nach `System.out`. |
+| Logger-Klasse | Konfiguration | Funktion                | Bemerkung |
+|---------------|---------------|-------------------------|-----------|
+| FileLogger    | file          | Loggt in eine Datei     | Schreibt `nachricht + " (in eine Datei geloggt)"` nach `System.out`. |
+| DbLogger      | db            | Loggt in eine Datenbank | Schreibt `nachricht + " (in eine DB geloggt)"` nach `System.out`. |
+| SilentLogger  | silent        | Macht gar nichts        | Ignoriert die Nachricht und schreibt nichts nach `System.out`. |

pom.xml

@@ -15,6 +15,8 @@
         <maven.compiler.target>1.8</maven.compiler.target>
     </properties>
 
+    <!-- Test dependencies -->
+
     <dependencies>
         <dependency>
             <groupId>junit</groupId>

src/main/java/de/doubleslash/kata/designpattern/factory/LoggerFactory.java

@@ -1,7 +1,5 @@
 package de.doubleslash.kata.designpattern.factory;
 
-import static java.lang.String.format;
-
 public class LoggerFactory {
 
     private final LoggerConfiguration loggerConfig;

src/test/java/de/doubleslash/kata/designpattern/factory/LoggerFactoryTest.java

@@ -1,7 +1,6 @@
 package de.doubleslash.kata.designpattern.factory;
 
 import org.apache.commons.io.output.TeeOutputStream;
-import org.hamcrest.CoreMatchers;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.Test;
@@ -10,11 +9,7 @@
 import java.io.PrintStream;
 import java.nio.charset.StandardCharsets;
 
-import static org.hamcrest.CoreMatchers.is;
-import static org.hamcrest.CoreMatchers.notNullValue;
-import static org.hamcrest.CoreMatchers.equalTo;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotNull;
+import static org.hamcrest.CoreMatchers.*;
 import static org.junit.Assert.assertThat;
 
 public class LoggerFactoryTest {
@@ -196,9 +191,7 @@ public void testWhenLoggingToSilentLoggerNothingIsLogged() throws Exception {
 
         // Testen
         Logger logger = factory.getLogger();
-        if (logger != null) {
-            logger.log(LOG_MESSAGE);
-        }
+        logger.log(LOG_MESSAGE);
 
         // Auswerten
         assertThatNothingWasLogged();