Add test for form validator

The validator requires some changes to already existing tests.

Co-authored-by: Simon Bruder <simon.bruder@mailbox.tu-dresden.de>

src/main/java/catering/users/User.java

@@ -71,4 +71,25 @@ public class User {
 	public boolean hasRole(String role) {
 		return userAccount.getRoles().stream().toList().contains(Role.of(role));
 	}
+
+	// Der Ficker muss so. -- sbruder
+	@Override
+	public int hashCode() {
+		final int prime = 31;
+		int result = 1;
+		result = prime * result + ((id == null) ? 0 : id.hashCode());
+		result = prime * result + ((address == null) ? 0 : address.hashCode());
+		result = prime * result + ((fullName == null) ? 0 : fullName.hashCode());
+		result = prime * result + ((userAccount == null) ? 0 : userAccount.getId().hashCode());
+		return result;
+	}
+
+	@Override
+	public boolean equals(Object obj) {
+		if (obj instanceof User other) {
+			return address.equals(other.address) && fullName.equals(other.fullName)
+					&& userAccount.getPassword().equals(other.userAccount.getPassword());
+		}
+		return false;
+	}
 }

src/test/java/catering/users/UserControllerIntegrationTests.java

@@ -1,16 +1,26 @@
 package catering.users;
 
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.tuple;
+import static org.hamcrest.CoreMatchers.containsString;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post;
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.redirectedUrl;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.content;
+
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
+import org.salespointframework.useraccount.UserAccountManagement;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
 import org.springframework.boot.test.context.SpringBootTest;
 import org.springframework.security.test.context.support.WithMockUser;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.request.MockHttpServletRequestBuilder;
-
+import org.springframework.security.test.context.support.WithAnonymousUser;
-import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post;
-import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.redirectedUrl;
 
 @AutoConfigureMockMvc
 @SpringBootTest
@@ -18,14 +28,38 @@ class UserControllerIntegrationTests {
 	@Autowired
 	MockMvc mvc;
 
+	@Autowired
+	private UserManagement userManagement;
+
+	@Autowired
+	private UserAccountManagement userAccountManagement;
+
+	void createTestUser(String name) {
+		if (userAccountManagement.findByUsername(name).isEmpty()) {
+			userManagement.createCustomer(name, "Baum Weg", "123",
+					name.substring(0, 1).toUpperCase() + name.substring(1) + " Klaus");
+		}
+	}
+
 	@BeforeEach
-	void init() throws Exception {
+	void setup() throws Exception {
-		MockHttpServletRequestBuilder createCustomer = post("/register")
+		createTestUser("dieter");
-			.param("username", "dieter")
+		createTestUser("hans");
-			.param("password", "123")
+		createTestUser("tim");
-			.param("fullName", "Dieter")
+		createTestUser("gina");
-			.param("address", "Baum Weg");
+		createTestUser("mona");
-		mvc.perform(createCustomer);
+		createTestUser("john");
+		createTestUser("stefan");
+	}
+
+	@Test
+	@WithMockUser(username = "john", password = "123")
+	void profileLoadsProperly() throws Exception {
+		mvc.perform(get("/profile"))
+				.andExpect(status().isOk())
+				.andExpect(content().string(containsString("john")))
+				.andExpect(content().string(containsString("John Klaus")))
+				.andExpect(content().string(containsString("Baum Weg")));
 	}
 
 	@Test
@@ -37,6 +71,168 @@ class UserControllerIntegrationTests {
 				.param("fullName", "Dieter")
 				.param("address", "Baum Weg");
 		mvc.perform(createMessage).andExpect(redirectedUrl("/logout"));
+		assertTrue(userAccountManagement.findByUsername("diete").isPresent());
+		assertFalse(userAccountManagement.findByUsername("dieter").isPresent());
 	}
 
+	@Test
+	@WithMockUser(username = "hans", password = "123")
+	void profileRefuseNullByte() throws Exception {
+		User pre = userManagement.getUserByName("hans").get();
+
+		mvc.perform(post("/profile")
+				.param("username", "hansi")
+				.param("password", "\0")
+				.param("fullName", "Hans Klaus")
+				.param("address", "Baum Weg"))
+				.andExpect(status().isOk())
+				.andExpect(content().string(containsString("Ungültiges Passwort")));
+
+		User post = userManagement.getUserByName("hans").get();
+
+		assertThat(pre).isEqualTo(post);
+	}
+
+	@Test
+	@WithMockUser(username = "tim", password = "123")
+	void allowEmptyPasswordInProfile() throws Exception {
+		String streetPre = userManagement.getUserByName("tim").get().getAddress();
+
+		MockHttpServletRequestBuilder createMessage = post("/profile")
+				.param("username", "tim")
+				.param("password", "")
+				.param("fullName", "Tim Klaus")
+				.param("address", "Baum Straße");
+		mvc.perform(createMessage)
+				.andExpect(status().is3xxRedirection())
+				.andExpect(redirectedUrl("/profile"));
+
+		String streetPost = userManagement.getUserByName("tim").get().getAddress();
+
+		assertThat(streetPost).isNotEqualTo(streetPre);
+	}
+
+	// This test is necessary, because the password in the profile form is not
+	// valided by "validationutils"
+	@Test
+	@WithMockUser(username = "mona", password = "123")
+	void refuseNullPasswordInProfile() throws Exception {
+		assertTrue(userManagement.getUserByName("mona").isPresent());
+		assertFalse(userManagement.getUserByName("mon").isPresent());
+		MockHttpServletRequestBuilder createMessage = post("/profile")
+				.param("username", "mon")
+				// .param("password", null) can be achieved by simply not setting the password
+				// parameter
+				.param("fullName", "Mona Klaus")
+				.param("address", "Baum Straße");
+		mvc.perform(createMessage)
+				.andExpect(status().isOk())
+				.andExpect(content().string(containsString("Ungültiges Passwort")));
+		assertTrue(userAccountManagement.findByUsername("mona").isPresent());
+		assertFalse(userAccountManagement.findByUsername("mon").isPresent());
+
+	}
+
+	@Test
+	@WithMockUser(username = "gina", password = "123")
+	void updateEverythingInProfile() throws Exception {
+		assertTrue(userManagement.getUserByName("gina").isPresent());
+		assertFalse(userManagement.getUserByName("giina").isPresent());
+		MockHttpServletRequestBuilder createMessage = post("/profile")
+				.param("username", "giina")
+				.param("password", "gina")
+				.param("fullName", "Gina")
+				.param("address", "Hotel");
+		mvc.perform(createMessage)
+				.andExpect(status().is3xxRedirection())
+				.andExpect(redirectedUrl("/logout"));
+		assertFalse(userAccountManagement.findByUsername("gina").isPresent());
+		assertThat(userManagement.getUsers().findAll())
+				.extracting("username", "fullName", "address")
+				.contains(tuple("giina", "Gina", "Hotel"));
+		MockHttpServletRequestBuilder login = post("/login")
+				.param("username", "giina")
+				.param("password", "gina");
+		mvc.perform(login)
+				.andExpect(status().is3xxRedirection())
+				.andExpect(redirectedUrl("/"));
+	}
+
+	@Test
+	@WithMockUser(username = "stefan", password = "123")
+	void updateEverythingInProfileWrong() throws Exception {
+		assertTrue(userManagement.getUserByName("stefan").isPresent());
+		mvc.perform(post("/profile"))
+				.andExpect(status().isOk())
+				.andExpect(content().string(containsString("Ungültiger Nutzername")))
+				.andExpect(content().string(containsString("Ungültiges Passwort")))
+				.andExpect(content().string(containsString("Ungültiger Name")))
+				.andExpect(content().string(containsString("Ungültige Addresse")));
+		assertFalse(userAccountManagement.findByUsername("steafan").isPresent());
+		assertThat(userManagement.getUsers().findAll())
+				.extracting("username", "fullName", "address")
+				.contains(tuple("stefan", "Stefan Klaus", "Baum Weg"));
+	}
+
+	@Test
+	@WithAnonymousUser
+	void refuseEmptyPasswordDuringRegistration() throws Exception {
+		MockHttpServletRequestBuilder createEmptyCustomer = post("/register")
+				.param("username", "emptyCustomer")
+				.param("password", "")
+				.param("fullName", "Dieter")
+				.param("address", "Baum Weg");
+		mvc.perform(createEmptyCustomer)
+				.andExpect(status().isOk())
+				.andExpect(content().string(containsString("Ungültiges Passwort")));
+		assertFalse(userAccountManagement.findByUsername("emptyCustomer").isPresent());
+	}
+
+	@Test
+	@WithAnonymousUser
+	void createCustomer() throws Exception {
+		assertFalse(userAccountManagement.findByUsername("tom").isPresent());
+		MockHttpServletRequestBuilder createCustomer = post("/register")
+				.param("username", "tom")
+				.param("password", "123")
+				.param("fullName", "Tom Klaus")
+				.param("address", "Baum Weg");
+		mvc.perform(createCustomer);
+		assertTrue(userAccountManagement.findByUsername("tom").isPresent());
+		assertThat(userManagement.getUsers().findAll())
+				.extracting("username", "fullName", "address")
+				.contains(tuple("tom", "Tom Klaus", "Baum Weg"));
+	}
+
+	@Test
+	@WithAnonymousUser
+	void registrationRefuseNullByte() throws Exception {
+		MockHttpServletRequestBuilder createNullCustomer = post("/register")
+				.param("username", "didi")
+				.param("password", "\0")
+				.param("fullName", "Dieter")
+				.param("address", "Baum Weg");
+		mvc.perform(createNullCustomer)
+				.andExpect(status().isOk())
+				.andExpect(content().string(containsString("Ungültiges Passwort")));
+		assertFalse(userAccountManagement.findByUsername("didi").isPresent());
+	}
+
+	@Test
+	@WithAnonymousUser
+	void registrationFormLoadsProperly() throws Exception {
+		mvc.perform(get("/register"))
+				.andExpect(status().isOk());
+	}
+
+	@Test
+	@WithAnonymousUser
+	void registerTotallyWrong() throws Exception {
+		mvc.perform(post("/register"))
+				.andExpect(status().isOk())
+				.andExpect(content().string(containsString("Ungültiger Nutzername")))
+				.andExpect(content().string(containsString("Ungültiges Passwort")))
+				.andExpect(content().string(containsString("Ungültiger Name")))
+				.andExpect(content().string(containsString("Ungültige Addresse")));
+	}
 }