/account/activate: prohibit repeated special characters.

Fix #454

Author: bodom91

src/main/java/ru/mystamps/web/model/ActivateAccountForm.java

@@ -62,12 +62,19 @@ public class ActivateAccountForm implements ActivateAccountDto {
 			groups = Login2Checks.class
 		)
 	})
-	@Pattern(
-		regexp = ValidationRules.LOGIN_REGEXP,
-		message = "{login.invalid}",
-		groups = Login3Checks.class
-	)
-	@UniqueLogin(groups = Login4Checks.class)
+	@Pattern.List({
+		@Pattern(
+			regexp = ValidationRules.LOGIN_REGEXP,
+			message = "{login.invalid}",
+			groups = Login3Checks.class
+		),
+		@Pattern(
+			regexp = ValidationRules.LOGIN_REPEATING_CHARS_REGEXP,
+			message = "{login.repetition_chars}",
+			groups = Login4Checks.class
+		)
+	})
+	@UniqueLogin(groups = Login5Checks.class)
 	private String login;
 
 	@Size(
@@ -120,7 +127,8 @@ public class ActivateAccountForm implements ActivateAccountDto {
 		Login1Checks.class,
 		Login2Checks.class,
 		Login3Checks.class,
-		Login4Checks.class
+		Login4Checks.class,
+		Login5Checks.class
 	})
 	public interface LoginChecks {
 	}
@@ -137,6 +145,9 @@ public interface Login3Checks {
 	public interface Login4Checks {
 	}
 
+	public interface Login5Checks {
+	}
+	
 	@GroupSequence({
 		Name1Checks.class,
 		Name2Checks.class,

src/main/java/ru/mystamps/web/validation/ValidationRules.java

@@ -24,6 +24,7 @@ public final class ValidationRules {
 	public static final int LOGIN_MIN_LENGTH = 2;
 	public static final int LOGIN_MAX_LENGTH = Db.User.LOGIN_LENGTH;
 	public static final String LOGIN_REGEXP = "[-_\\.a-zA-Z0-9]+";
+	public static final String LOGIN_REPEATING_CHARS_REGEXP = "(?!.+[-_.]{2,}).+";
 
 	public static final int NAME_MAX_LENGTH = Db.User.NAME_LENGTH;
 	public static final String NAME_REGEXP = "[- \\p{L}]+";

src/main/resources/ru/mystamps/i18n/ValidationMessages.properties

@@ -33,6 +33,7 @@ country-name-en.invalid = Country name must consist only latin letters, hyphen o
 country-name-ru.invalid = Country name must consist only Russian letters, hyphen or spaces
 
 login.invalid = Login must consist only latin letters, digits, dot, hyphen or underscore
+login.repetition_chars = Login must not contain repetition of hyphen, dot or underscore
 
 password.mismatch = Password mismatch
 password.login.match = Password and login must be different

src/main/resources/ru/mystamps/i18n/ValidationMessages_ru.properties

@@ -33,6 +33,7 @@ country-name-en.invalid = Название страны может содерж
 country-name-ru.invalid = Название страны может содержать только кирилические буквы, дефис или пробел
 
 login.invalid = Логин может состоять только из латинских букв, цифр, точки, дефиса или символа подчёркивания
+login.repetition_chars = Логин не должен содержать повторяющиеся символы дефиса, точки или символа подчёркивания
 
 password.mismatch = Пароли не совпадают
 password.login.match = Пароль и логин не должны совпадать

src/test/java/ru/mystamps/web/tests/cases/WhenAnonymousUserActivateAccount.java

@@ -150,6 +150,13 @@ public void loginShouldBeUnique() {
 			.field("login")
 			.hasError(tr("ru.mystamps.web.validation.jsr303.UniqueLogin.message"));
 	}
+
+	@Test(groups = "invalid", dependsOnGroups = "std", dataProvider = "invalidLogins")
+	public void loginShouldNotContainRepetitionSpecialCharacters(String login, Object whatever) {
+		page.activateAccount(login, null, null, null, null);
+
+		assertThat(page).field("login").hasError(tr("login.repetition_chars"));
+	}
 
 	@Test(groups = "misc", dependsOnGroups = "std")
 	public void loginShouldBeStripedFromLeadingAndTrailingSpaces() {
@@ -305,5 +312,16 @@ public Object[][] getValidNames() {
 			{"Семён Якушев", null}
 		};
 	}
+
+	@DataProvider(name = "invalidLogins")
+	public Object[][] getInvalidLogins() {
+		return new Object[][] {
+			{"te__st", null},
+			{"te--st", null},
+			{"te..st", null},
+			{"te_-st", null},
+			{"te-._st", null}
+		};
+	}
 
 }