test: CDI-Runner eingebaut und refactoring betrieben

Thomas Ziemer
1 parent d80fc452
Showing 5 changed files with 39 additions and 7 deletions
README.md
pom.xml
src/main/java/net/ziemers/swxercise/db/utils/JpaTestUtils.java → src/test/java/net/ziemers/swxercise/db/utils/JpaTestUtils.java
src/test/java/net/ziemers/swxercise/lg/user/service/UserServiceTest.java
src/test/java/net/ziemers/swxercise/ui/UserViewControllerTest.java
@@ -31,12 +31,16 @@ Dann kann/muss das Kennwort geändert werden, beispielsweise in &#39;root&#39;:
 > ALTER USER 'root'@'localhost' IDENTIFIED BY ‘root‘;
-Dann muss eine projektspezifische Datenbank erstellt und mit den nötigen Rechten versehen werden:
+Dann müssen die beiden projektspezifischen Datenbanken (für Test und "Produktion") erstellt und mit den nötigen Rechten versehen werden:
 > create database swxercise;
+> create database swxercise_test;
+
 > grant all privileges on swxercise.* to 'root'@'localhost' identified by 'root';
+> grant all privileges on swxercise_test.* to 'root'@'localhost' identified by 'root';
+
 ## MySQL-Datenbank im WildFly einbinden
 Zunächst muss der JBoss WildFly Application Server installiert werden. Dies ist betriebssystemspezifisch.
@@ -19,6 +19,7 @@
 		<mockito-version>2.7.22</mockito-version>
 		<dbunit-version>2.5.1</dbunit-version>
 		<slf4j-version>1.7.25</slf4j-version>
+        <jglue-version>3.1.4</jglue-version>
 		<jackson-version>2.8.8.1</jackson-version>
 		<!-- Substituiert die Platzhalter in der "persistence.xml" -->
@@ -67,8 +68,7 @@
 			<groupId>org.dbunit</groupId>
 			<artifactId>dbunit</artifactId>
 			<version>${dbunit-version}</version>
-			<!--scope>test</scope-->
-			<!-- TODO Wie koennen wir es erreichen, dass dies nur noch im Testkontext abhängig ist? -->
+            <scope>test</scope>
 		</dependency>
         <dependency>
             <groupId>org.hibernate</groupId>
@@ -80,6 +80,13 @@
 			<artifactId>mysql-connector-java</artifactId>
 			<version>${mysql-version}</version>
 		</dependency>
+        <dependency>
+            <groupId>org.jglue.cdi-unit</groupId>
+            <artifactId>cdi-unit</artifactId>
+            <version>${jglue-version}</version>
+            <scope>test</scope>
+        </dependency>
+        <!-- Sorgt für eine schnellere Verwaltung der JSON-Umwandlung im REST-Kontext -->
 		<!-- dependency>
 			<groupId>com.fasterxml.jackson.core</groupId>
 			<artifactId>jackson-databind</artifactId>
@@ -5,9 +5,21 @@ import javax.inject.Inject;
 import net.ziemers.swxercise.db.utils.JpaTestUtils;
 import net.ziemers.swxercise.lg.user.dto.UserDto;
+import org.jglue.cdiunit.CdiRunner;
 import org.junit.Before;
 import org.junit.Test;
-
+import org.junit.runner.RunWith;
+
+/*
+ * Testing your Java CDI application with CDI-Unit couldn't be easier.
+ * Just specify @RunWith(CdiRunner.class) on your JUnit test class
+ * to enable injection directly into the test class.
+ *
+ * To test classes in isolation we shouldn't be using their dependencies.
+ * Instead we should be using a mock. There are many mocking libraries out
+ * there, however CDI-Unit has extra support for Mockito @Mock annotations.
+ */
+@RunWith(CdiRunner.class)
 public class UserServiceTest extends JpaTestUtils {
     private static boolean dbInitialized;
@@ -15,9 +15,17 @@ import static org.mockito.Mockito.*;
 @RunWith(MockitoJUnitRunner.class)
 public class UserViewControllerTest {
+    /*
+     * You have the @InjectMocks annotation which tries to do constructor,
+     * method and field dependency injection based on the type.
+     */
     @InjectMocks
     private UserViewController underTest;
+    /*
+     * Mockito allows to configure the return values of its mocks via a fluent API.
+     * Unspecified method calls return "empty" values
+     */
     @Mock
     private UserService userService;
@@ -112,10 +120,11 @@ public class UserViewControllerTest {
     }
     private void loginUser() {
-        // wir simulieren den Erfolg des userService (der hier nicht getestet werden soll),
-        // sofern das korrekte userDto an ihn übergeben wurde
+        /*
+         * The when().thenReturn() method chain is used to specify
+         * a return value for a method call with pre-defined parameters.
+         */
         when(userService.loginUser(userDto)).thenReturn(true);
-        //when(userService.getSessionUser()).thenReturn(user);
         actual = underTest.loginUser(userDto);
     }