Merge into master

Thomas Ziemer
2 parents 27791b11 8cae1cc5
Showing 10 changed files with 191 additions and 74 deletions
src/main/java/net/ziemers/swxercise/lg/user/dto/UserDto.java
src/main/java/net/ziemers/swxercise/lg/user/service/EntityToUserDtoMapper.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityContext.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityContextService.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserDtoToEntityMapper.java
src/main/java/net/ziemers/swxercise/lg/user/service/UserService.java
src/main/java/net/ziemers/swxercise/ui/UserViewController.java
src/test/java/net/ziemers/swxercise/lg/user/service/UserServiceTest.java
src/test/java/net/ziemers/swxercise/ui/UserViewControllerTest.java
src/test/resources/net/ziemers/swxercise/testdata/UserServiceTestData.xml
 package net.ziemers.swxercise.lg.user.dto;
+import net.ziemers.swxercise.lg.model.user.User;
+
 import javax.validation.constraints.NotNull;
 /**
@@ -8,7 +10,7 @@ import javax.validation.constraints.NotNull;
  */
 public class UserDto {
-    private Long entityId;
+    private User user = null;
     @NotNull
     private String username;    // aus dem Profile
@@ -22,12 +24,12 @@ public class UserDto {
     private String mailaddress; // aus dem Profile
-    public Long getEntityId() {
-        return entityId;
+    public User getUser() {
+        return user;
     }
-    public void setEntityId(Long entityId) {
-        this.entityId = entityId;
+    public void setUser(User user) {
+        this.user = user;
     }
     public String getUsername() {
@@ -9,7 +9,7 @@ import net.ziemers.swxercise.lg.user.dto.UserDto;
 public class EntityToUserDtoMapper {
     /**
-     * Erzeugt ein DTO aus dem übergebenem Kontext.
+     * Erzeugt ein Data Transfer Object aus dem übergebenem Kontext.
      *
      * @param ctx der Kontext
      * @return das erzeugte DTO.
@@ -18,15 +18,15 @@ public class EntityToUserDtoMapper {
         final UserDto dto = new UserDto();
         if (ctx.user != null) {
-          mapEntityId(ctx, dto);
+          mapUser(ctx, dto);
           mapFirstname(ctx, dto);
           mapLastname(ctx, dto);
         }
         return dto;
     }
-    private void mapEntityId(EntityToUserDtoContext ctx, UserDto dto) {
-        dto.setEntityId(ctx.user.getId());
+    private void mapUser(EntityToUserDtoContext ctx, UserDto dto) {
+        dto.setUser(ctx.user);
     }
     private void mapFirstname(EntityToUserDtoContext ctx, UserDto dto) {
@@ -13,6 +13,6 @@ public class UserDtoToEntityContext {
     public Profile profile;
-    public Address address;
+    public Address address = null;
 }
@@ -22,20 +22,27 @@ public class UserDtoToEntityContextService {
      */
     public UserDtoToEntityContext createContext(final UserDto dto) {
         final UserDtoToEntityContext ctx = new UserDtoToEntityContext();
+        final User existingUser = dto.getUser();
         // das übergebene UserDto in den Kontext füllen
         ctx.dto = dto;
-        // einen neuen oder einen bereits existierenden Benutzer in den Kontext füllen
-        ctx.user = dao.findByUsername(dto.getUsername());
-        if (ctx.user == null) {
-            // es soll niemals ein Profil ohne Benutzername und Kennwort geben
-            ctx.profile = new Profile(dto.getUsername(), dto.getPassword());
+        // einen neuen Benutzer in den Kontext füllen
+        if (existingUser == null) {
+            // es darf nicht mehrere Benutzer mit dem selben Benutzernamen geben!
+            if (dao.findByUsername(dto.getUsername()) == null) {
+                // es soll niemals ein Profil ohne Benutzername und Kennwort geben
+                ctx.profile = new Profile(dto.getUsername(), dto.getPassword());
+
+                // wir füllen das User-Objekt mit Method Chaining
+                ctx.user = new User(dto.getFirstname(), dto.getLastname())
+                        .withProfile(ctx.profile);
+            }
+        }
-            // wir füllen das User-Objekt mit Method Chaining
-            ctx.user = new User(dto.getFirstname(), dto.getLastname())
-                    .withProfile(ctx.profile);
-        } else {
+        // einen bereits existierenden Benutzer in den Kontext füllen
+        else {
+            ctx.user = existingUser;
             ctx.profile = ctx.user.getProfile();
             ctx.address = ctx.user.getAddress();
         }
@@ -11,26 +11,30 @@ import javax.ejb.Stateless;
 public class UserDtoToEntityMapper {
     /**
-     * Überträgt die Eigenschaften aus dem UserDto sowie dem zusätzlichen Kontext in die Zielentität.
+     * Überträgt die Eigenschaften aus dem UserDto sowie dem zusätzlichen Kontext in die Zielentitäten.
+     * Anzumerken ist, dass der Benutzername einer existierenden Profile-Entität niemals aktualisiert
+     * wird.
      *
      * @param ctx der Kontext mit den Eigenschaften und der Zielentität
      * @return den Kontext.
      */
     public UserDtoToEntityContext map(UserDtoToEntityContext ctx) {
-        // User-Objekt mappen
+        // ins User-Objekt mappen
         ctx.user.setFirstname(ctx.dto.getFirstname());
         ctx.user.setLastname(ctx.dto.getLastname());
-        // Profile-Objekt mappen, falls gegeben
+        // ins Profile-Objekt mappen, falls gegeben
         if (ctx.profile != null) {
             ctx.user.setProfile(ctx.profile);
+
+            // eventuell ein neues Kenntwort ins Profile-Objekt mappen
             if(ctx.dto.getPassword().length() > 0) {
                 ctx.profile.setPassword(ctx.dto.getPassword());
             }
             ctx.profile.setMailaddress(ctx.dto.getMailaddress());
         }
-        // Address-Objekt mappen, falls gegeben
+        // ins Address-Objekt mappen, falls gegeben
         if (ctx.address != null) {
             ctx.user.setAddress(ctx.address);
         }
@@ -81,20 +81,17 @@ public class UserService {
      * Zwischen der Groß- und Kleinschreibung wird nicht unterschieden.
      *
      * @param dto das {@link UserDto} enthält die Eigenschaften des zu erstellenden Benutzers
-     * @return die Id des neuen Benutzers, wenn die Erstellung erfolgreich war.
+     * @return <code>true</code>, wenn das Aktualisieren des Benutzers erfolgreich war.
      */
-    public Long createUser(final UserDto dto) {
+    public boolean createUser(final UserDto dto) {
+        // Kontext für den zu erstellenden Benutzer erzeugen; falls bereits ein Benutzer
+        // mit dem selben Benutzernamen existiert, wird kein Benutzerobjekt zurückgeliefert
         final UserDtoToEntityContext ctx = ctxService.createContext(dto);
-        mapper.map(ctx);
-        return persistUserIfNew(ctx);
-    }
-
-    private Long persistUserIfNew(final UserDtoToEntityContext ctx) {
-        // nur ein neuer Benutzer hat noch keine Id
-        if (ctx.user.getId() == null) {
-            return dao.save(ctx.user);
+        if (ctx.user != null) {
+            mapper.map(ctx);
+            return dao.save(ctx.user) != null;
         }
-        return null;
+        return false;
     }
     /**
@@ -107,8 +104,12 @@ public class UserService {
     public boolean updateUser(final Long id, final UserDto dto) {
         final User user = dao.findById(id);
         if (user != null) {
-            // TODO noch zu implementieren
-            return false;
+            // auf dieses Benutzerobjekt soll sich die Aktualisierung beziehen
+            dto.setUser(user);
+
+            final UserDtoToEntityContext ctx = ctxService.createContext(dto);
+            mapper.map(ctx);
+            return dao.saveOrUpdate(ctx.user) != null;
         }
         return false;
     }
@@ -120,16 +121,8 @@ public class UserService {
      * @return <code>true</code>, wenn das Aktualisieren des Benutzers erfolgreich war.
      */
     public boolean updateUser(final UserDto dto) {
-        // ist zurzeit ein Benutzer angemeldet, können wir ihn aktualisieren
         final User user = sessionContext.getUser();
-        if (user != null) {
-            // der Benutzername darf sich beim Aktualisieren nicht mehr ändern!
-            dto.withUsername(user.getProfile().getUsername());
-            final UserDtoToEntityContext ctx = ctxService.createContext(dto);
-            mapper.map(ctx);
-            return dao.saveOrUpdate(ctx.user) != null;
-        }
-        return false;
+        return updateUser(user.getId(), dto);
     }
     /**
@@ -89,9 +89,8 @@ public class UserViewController {
     @Produces(MediaType.APPLICATION_JSON)
     @RolesAllowed(RightState.Constants.ADMIN)
     public RestResponse createUser(UserDto dto) {
-        final Long id = userService.createUser(dto);
-        if (id != null) {
-            return new RestResponse(ResponseState.SUCCESS, String.valueOf(id));
+        if (userService.createUser(dto)) {
+            return new RestResponse();
         }
         return new RestResponse(ResponseState.ALREADY_EXISTING);
     }
@@ -29,12 +29,13 @@ public class UserServiceTest extends JpaTestUtils {
     private static String USERNAME_TEST = "username_test";
     private static String EXISTING_USERNAME_TEST = "username_profile";
+    private static Long EXISTING_USER_ID = 2L;
     private static boolean dbInitialized;
     private UserDto userDto;
-    private Long userId;
+    private boolean actual;
     @Inject
     private UserDao userDao;
@@ -74,7 +75,20 @@ public class UserServiceTest extends JpaTestUtils {
                 .createUser();
         then()
-                .assertCreateFail();
+                .assertCreateFailure();
+    }
+
+    @Test
+    public void testUpdateUserDoesntUpdateUsername() {
+
+        given()
+                .userDto(USERNAME_TEST);
+
+        when()
+                .updateUser(EXISTING_USER_ID);
+
+        then()
+                .assertUpdateSuccess();
     }
     // given
@@ -85,7 +99,7 @@ public class UserServiceTest extends JpaTestUtils {
     private UserServiceTest userDto(final String username) {
         userDto = new UserDtoTestDataBuilder()
-                .withUsername(USERNAME_TEST)
+                .withUsername(username)
                 .build();
         return this;
     }
@@ -98,7 +112,15 @@ public class UserServiceTest extends JpaTestUtils {
     private UserServiceTest createUser() {
         txBegin();
-        userId = underTest.createUser(userDto);
+        actual = underTest.createUser(userDto);
+        txCommit();
+
+        return this;
+    }
+
+    private UserServiceTest updateUser(final Long id) {
+        txBegin();
+        actual = underTest.updateUser(id, userDto);
         txCommit();
         return this;
@@ -111,14 +133,20 @@ public class UserServiceTest extends JpaTestUtils {
     }
     private void assertCreateSuccess() {
-        // wir suchen den soeben erstellten Benutzer; wenn er existiert, is alles gut
+        // wir suchen den soeben erstellten Benutzer; wenn er existiert, ist alles gut
         final User user = userDao.findByUsername(USERNAME_TEST);
         assertNotNull(user);
     }
-    private void assertCreateFail() {
-        // es darf kein neuer Benutzer mit identischem "username" erstellt worden sein
-        assertNull(userId);
+    private void assertCreateFailure() {
+        // es darf kein neuer Benutzer mit identischem Benutzernamen erstellt worden sein
+        assertFalse(actual);
+    }
+
+    private void assertUpdateSuccess() {
+        // wir suchen den soeben aktualisierten Benutzer; wenn sein Benutzername unverändert ist, ist alles gut
+        final User user = userDao.findById(EXISTING_USER_ID);
+        assertEquals(EXISTING_USERNAME_TEST, user.getProfile().getUsername());
     }
 }
 package net.ziemers.swxercise.ui;
-import net.ziemers.swxercise.lg.model.user.User;
 import net.ziemers.swxercise.lg.testdatabuilder.user.UserDtoTestDataBuilder;
-import net.ziemers.swxercise.lg.testdatabuilder.user.UserTestDataBuilder;
 import net.ziemers.swxercise.lg.user.dto.UserDto;
 import net.ziemers.swxercise.lg.user.service.UserService;
+import net.ziemers.swxercise.ui.enums.ResponseState;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.InjectMocks;
@@ -33,8 +32,6 @@ public class UserViewControllerTest {
     private UserDto userDto;
-    private User user;
-
     private RestResponse actual;
     @Test
@@ -59,12 +56,54 @@ public class UserViewControllerTest {
     @Test
     public void testCreateUserReturnsSuccess() {
-        // TODO Test ist noch zu implementieren
+
+        given()
+                .userDto();
+
+        doing()
+                .createUser(true);
+
+        then()
+                .assertCreateSuccess();
     }
     @Test
-    public void testUpdateUserReturnsSuccess() {
-        // TODO Test ist noch zu implementieren
+    public void testCreateUserReturnsFailure() {
+
+        given()
+                .userDto();
+
+        doing()
+                .createUser(false);
+
+        then()
+                .assertCreateFailure();
+    }
+
+    @Test
+    public void testUpdateLoggedInUserReturnsSuccess() {
+
+        given()
+                .userDto();
+
+        doing()
+                .updateUser();
+
+        then()
+                .assertUpdateSuccess();
+    }
+
+    @Test
+    public void testUpdateSpecificUserReturnsSuccess() {
+
+        given()
+                .userDto();
+
+        doing()
+                .updateUser(1L);
+
+        then()
+                .assertUpdateSuccess();
     }
     @Test
@@ -81,17 +120,29 @@ public class UserViewControllerTest {
     public void testLoginUserReturnsSuccess() {
         given()
-                .userDto()
-                .user();
+                .userDto();
         doing()
-                .loginUser();
+                .loginUser(true);
         then()
                 .assertLoginSuccess();
     }
     @Test
+    public void testLoginNonExistingUserReturnsFailure() {
+
+        given()
+                .userDto();
+
+        doing()
+                .loginUser(false);
+
+        then()
+                .assertLoginFailure();
+    }
+
+    @Test
     public void testLogoutUserReturnsSuccess() {
         doing()
@@ -112,23 +163,36 @@ public class UserViewControllerTest {
         return this;
     }
-    private UserViewControllerTest user() {
-        user = new UserTestDataBuilder().build();
-        return this;
-    }
-
     // doing
     private UserViewControllerTest doing() {
         return this;
     }
-    private void loginUser() {
+    private void createUser(final boolean result) {
         /*
          * 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.createUser(userDto)).thenReturn(result);
+
+        actual = underTest.createUser(userDto);
+    }
+
+    private void updateUser() {
+        when(userService.updateUser(userDto)).thenReturn(true);
+
+        actual = underTest.updateUser(userDto);
+    }
+
+    private void updateUser(final Long id) {
+        when(userService.updateUser(id, userDto)).thenReturn(true);
+
+        actual = underTest.updateUser(id, userDto);
+    }
+
+    private void loginUser(final boolean result) {
+        when(userService.loginUser(userDto)).thenReturn(result);
         actual = underTest.loginUser(userDto);
     }
@@ -145,11 +209,31 @@ public class UserViewControllerTest {
         return this;
     }
+    private void assertCreateSuccess() {
+        final RestResponse expected = new RestResponse();
+        assertEquals(expected, actual);
+    }
+
+    private void assertCreateFailure() {
+        final RestResponse expected = new RestResponse(ResponseState.ALREADY_EXISTING);
+        assertEquals(expected, actual);
+    }
+
+    private void assertUpdateSuccess() {
+        final RestResponse expected = new RestResponse();
+        assertEquals(expected, actual);
+    }
+
     private void assertLoginSuccess() {
         final RestResponse expected = new RestResponse();
         assertEquals(expected, actual);
     }
+    private void assertLoginFailure() {
+        final RestResponse expected = new RestResponse(ResponseState.FAILED);
+        assertEquals(expected, actual);
+    }
+
     private void assertLogoutSuccess() {
         final RestResponse expected = new RestResponse();
         assertEquals(expected, actual);
 <?xml version="1.0" encoding="UTF-8"?>
 <dataset>
     <!-- die Grosz-/Kleinschreibung scheint unter bestimmten Bedingungen von Bedeutung zu sein! -->
-    <Profile id="1" username="username_profile" passwordHash="passwordHash_profile" hashAlgorithm="hashAlgorithm_profile" salt="salt_profile"/>
+    <Profile id="1" username="username_profile" passwordHash="passwordHash_profile" hashAlgorithm="SHA512" salt="salt_profile"/>
     <User id="2" firstname="firstname_user" lastname="lastname_user" profile_id="1"/>
 </dataset>