Готово

java-exceptions-template.iml

@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<module org.jetbrains.idea.maven.project.MavenProjectsManager.isMavenModule="true" type="JAVA_MODULE" version="4">
+  <component name="NewModuleRootManager" LANGUAGE_LEVEL="JDK_1_8">
+    <output url="file://$MODULE_DIR$/target/classes" />
+    <output-test url="file://$MODULE_DIR$/target/test-classes" />
+    <content url="file://$MODULE_DIR$">
+      <sourceFolder url="file://$MODULE_DIR$/src/main/java" isTestSource="false" />
+      <sourceFolder url="file://$MODULE_DIR$/src/test/java" isTestSource="true" />
+      <excludeFolder url="file://$MODULE_DIR$/target" />
+    </content>
+    <orderEntry type="inheritedJdk" />
+    <orderEntry type="sourceFolder" forTests="false" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.junit.jupiter:junit-jupiter:5.5.2" level="project" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.junit.jupiter:junit-jupiter-api:5.5.2" level="project" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.apiguardian:apiguardian-api:1.1.0" level="project" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.opentest4j:opentest4j:1.2.0" level="project" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.junit.platform:junit-platform-commons:1.5.2" level="project" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.junit.jupiter:junit-jupiter-params:5.5.2" level="project" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.junit.jupiter:junit-jupiter-engine:5.5.2" level="project" />
+    <orderEntry type="library" scope="TEST" name="Maven: org.junit.platform:junit-platform-engine:1.5.2" level="project" />
+    <orderEntry type="library" scope="PROVIDED" name="Maven: org.projectlombok:lombok:1.18.10" level="project" />
+  </component>
+</module>

src/main/java/com/epam/izh/rd/online/exception/NotAccessException.java

@@ -0,0 +1,7 @@
+package com.epam.izh.rd.online.exception;
+
+public class NotAccessException extends Exception {
+    public NotAccessException(String message) {
+        super(message);
+    }
+}

src/main/java/com/epam/izh/rd/online/exception/NotCorrectPasswordException.java

@@ -0,0 +1,7 @@
+package com.epam.izh.rd.online.exception;
+
+public class NotCorrectPasswordException extends Exception{
+    public NotCorrectPasswordException(String message) {
+        super(message);
+    }
+}

src/main/java/com/epam/izh/rd/online/exception/SimplePasswordException.java

@@ -0,0 +1,7 @@
+package com.epam.izh.rd.online.exception;
+
+public class SimplePasswordException extends Exception {
+    public SimplePasswordException(String message) {
+        super(message);
+    }
+}

src/main/java/com/epam/izh/rd/online/exception/UserAlreadyRegisteredException.java

@@ -0,0 +1,7 @@
+package com.epam.izh.rd.online.exception;
+
+public class UserAlreadyRegisteredException extends Exception {
+    public UserAlreadyRegisteredException(String message) {
+        super(message);
+    }
+}

src/main/java/com/epam/izh/rd/online/exception/UserNotFoundException.java

@@ -0,0 +1,7 @@
+package com.epam.izh.rd.online.exception;
+
+public class UserNotFoundException extends Exception{
+    public UserNotFoundException(String message) {
+        super(message);
+    }
+}

src/main/java/com/epam/izh/rd/online/repository/UserRepository.java

@@ -9,7 +9,7 @@
 
 /**
  * Готовый класс репозитория.
- * Можно добавлять свои методы, принеобходимости, но нельзя исправлять текущие.
+ * Можно добавлять свои методы, при необходимости, но нельзя исправлять текущие.
  * <p>
  * Позволяет сохранять\обновлять пользователей.
  * Позволяет находить пользователя по логину.

src/main/java/com/epam/izh/rd/online/service/AuthenticationService.java

@@ -1,6 +1,8 @@
 package com.epam.izh.rd.online.service;
 
 import com.epam.izh.rd.online.entity.User;
+import com.epam.izh.rd.online.exception.NotCorrectPasswordException;
+import com.epam.izh.rd.online.exception.UserNotFoundException;
 import com.epam.izh.rd.online.repository.IUserRepository;
 
 public class AuthenticationService implements IAuthenticationService {
@@ -25,20 +27,22 @@ public AuthenticationService(IUserRepository userRepository) {
      * @param user - пользователь проходящий авторизацию
      */
     @Override
-    public User login(User user) {
-        // Находим пользователя в базе
+    public User login(User user) throws UserNotFoundException, NotCorrectPasswordException {
         User foundUser = userRepository.findByLogin(user.getLogin());
+        validateUserLoginAndPassword(user);
 
-        //
-        // Здесь необходимо реализовать перечисленные выше проверки
-        //
-
-        // Устанавливаем найденного пользователя, который прошел все проверки, как вошедшего в систему.
         CurrentUserManager.setCurrentLoggedInUser(foundUser);
-
         return foundUser;
     }
 
+    private void validateUserLoginAndPassword(User user) throws UserNotFoundException,NotCorrectPasswordException {
+        if (userRepository.findByLogin(user.getLogin()) == null) {
+            throw new UserNotFoundException("Пользователь с таким логином не найден");
+        } else if (!userRepository.findByLogin(user.getLogin()).getPassword().equals(user.getPassword())) {
+            throw new NotCorrectPasswordException("Пароль введен неверно!");
+        }
+    }
+
     /**
      * Данный метод очищает данные о текущем (активном) пользователе.
      */

src/main/java/com/epam/izh/rd/online/service/IAuthenticationService.java

@@ -1,9 +1,11 @@
 package com.epam.izh.rd.online.service;
 
 import com.epam.izh.rd.online.entity.User;
+import com.epam.izh.rd.online.exception.NotCorrectPasswordException;
+import com.epam.izh.rd.online.exception.UserNotFoundException;
 
 public interface IAuthenticationService {
-    User login(User user);
+    User login(User user) throws UserNotFoundException, NotCorrectPasswordException;
 
     void logout();
 }

src/main/java/com/epam/izh/rd/online/service/IUserService.java

@@ -1,10 +1,13 @@
 package com.epam.izh.rd.online.service;
 
 import com.epam.izh.rd.online.entity.User;
+import com.epam.izh.rd.online.exception.NotAccessException;
+import com.epam.izh.rd.online.exception.SimplePasswordException;
+import com.epam.izh.rd.online.exception.UserAlreadyRegisteredException;
 
 public interface IUserService {
 
-    User register(User user);
+    User register(User user) throws UserAlreadyRegisteredException, SimplePasswordException;
 
-    void delete(String login);
+    void delete(String login) throws NotAccessException;
 }

src/main/java/com/epam/izh/rd/online/service/UserService.java

@@ -1,6 +1,9 @@
 package com.epam.izh.rd.online.service;
 
 import com.epam.izh.rd.online.entity.User;
+import com.epam.izh.rd.online.exception.NotAccessException;
+import com.epam.izh.rd.online.exception.SimplePasswordException;
+import com.epam.izh.rd.online.exception.UserAlreadyRegisteredException;
 import com.epam.izh.rd.online.repository.IUserRepository;
 import com.epam.izh.rd.online.repository.UserRepository;
 
@@ -30,14 +33,33 @@ public UserService(IUserRepository userRepository) {
      * @param user - даныне регистрирующегося пользователя
      */
     @Override
-    public User register(User user) {
+    public User register(User user) throws UserAlreadyRegisteredException, SimplePasswordException {
+        validateUserData(user);
+        return userRepository.save(user);
+    }
 
-        //
-        // Здесь необходимо реализовать перечисленные выше проверки
-        //
+    private void validateUserData(User user) throws UserAlreadyRegisteredException, SimplePasswordException {
+        isEmptyLoginOrPasswordUserValue(user);
+        isUserAlreadyRegistered(user);
+        isUserSimplePassword(user);
+    }
 
-        // Если все проверки успешно пройдены, сохраняем пользователя в базу
-        return userRepository.save(user);
+    private void isEmptyLoginOrPasswordUserValue(User user) {
+        if (user.getLogin().isEmpty() || user.getPassword().isEmpty()) {
+            throw new IllegalArgumentException("Ошибка в заполнении полей");
+        }
+    }
+
+    private void isUserSimplePassword(User user) throws UserAlreadyRegisteredException {
+        if (userRepository.findByLogin(user.getLogin()) != null) {
+            throw new UserAlreadyRegisteredException("Пользователь с логином " + user.getLogin() + " уже зарегистрирован");
+        }
+    }
+
+    private void isUserAlreadyRegistered(User user) throws SimplePasswordException {
+        if (user.getPassword().matches("\\d*")) {
+            throw new SimplePasswordException("Пароль не соответствует требованиям безопасности");
+        }
     }
 
     /**
@@ -58,14 +80,11 @@ public User register(User user) {
      *
      * @param login
      */
-    public void delete(String login) {
-
-        // Здесь необходимо сделать доработку метод
-
+    public void delete(String login) throws NotAccessException {
+        try {
             userRepository.deleteByLogin(login);
-
-        // Здесь необходимо сделать доработку метода
-
+        } catch (UnsupportedOperationException e) {
+            throw new NotAccessException("Недостаточно прав для выполнения операции");
+        }
     }
-
 }

src/test/java/com/epam/izh/rd/online/Assert.java

@@ -7,12 +7,12 @@
 
 public class Assert<T extends Throwable> {
 
-    public T assertThrowsWithClassName(String className, Executable executable, String message) {
+    public T assertThrowsWithClassName(String className, Executable executable, String message) throws ClassNotFoundException {
         return assertThrows(forName(className), executable, message);
     }
 
     @SneakyThrows
-    private Class<T> forName(String className) {
+    private Class<T> forName(String className) throws ClassNotFoundException {
         return (Class<T>) Class.forName("com.epam.izh.rd.online.exception." + className);
     }
 }

src/test/java/com/epam/izh/rd/online/AuthenticationServiceTest.java

@@ -25,14 +25,14 @@ private void setup() {
 
     @Test
     @DisplayName("Тест метода IAuthenticationService.login(User user)")
-    void testRegisterCase1() {
+    void testRegisterCase1() throws ClassNotFoundException {
         assertion.assertThrowsWithClassName("UserNotFoundException", () -> authenticationService.login(getUser()),
                 "Пользователь с таким логином не найден");
     }
 
     @Test
     @DisplayName("Тест метода IAuthenticationService.login(User user)")
-    void testRegisterCase2() {
+    void testRegisterCase2() throws ClassNotFoundException {
         User user = getUser();
         userRepository.save(user);
 

src/test/java/com/epam/izh/rd/online/UserServiceTest.java

@@ -1,11 +1,11 @@
 package com.epam.izh.rd.online;
 
 import com.epam.izh.rd.online.entity.User;
+import com.epam.izh.rd.online.exception.SimplePasswordException;
+import com.epam.izh.rd.online.exception.UserAlreadyRegisteredException;
 import com.epam.izh.rd.online.repository.IUserRepository;
 import com.epam.izh.rd.online.repository.UserRepository;
-import com.epam.izh.rd.online.service.CurrentUserManager;
-import com.epam.izh.rd.online.service.IUserService;
-import com.epam.izh.rd.online.service.UserService;
+import com.epam.izh.rd.online.service.*;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -39,7 +39,7 @@ void testRegisterCase1(User user) {
 
     @Test
     @DisplayName("Тест метода IUserService.register(User user) кейс 2")
-    void testRegisterCase2() throws Exception {
+    void testRegisterCase2() throws Exception, SimplePasswordException, UserAlreadyRegisteredException {
         User user = Providers.getUser();
 
         userService.register(user);
@@ -49,7 +49,7 @@ void testRegisterCase2() throws Exception {
 
     @Test
     @DisplayName("Тест метода IUserService.register(User user) кейс 3")
-    void testRegisterCase3() {
+    void testRegisterCase3() throws ClassNotFoundException {
         User user = getUserWithNumberPassword();
 
         assertion.assertThrowsWithClassName("SimplePasswordException", () -> userService.register(user),
@@ -60,7 +60,7 @@ void testRegisterCase3() {
     @ParameterizedTest
     @MethodSource("com.epam.izh.rd.online.Providers#testDelete")
     @DisplayName("Тест метода IUserService.delete(String login)")
-    void testDelete(User user) {
+    void testDelete(User user) throws ClassNotFoundException {
         CurrentUserManager.setCurrentLoggedInUser(user);
         assertion.assertThrowsWithClassName("NotAccessException", () -> userService.delete("123"),
                 "Недостаточно прав для выполнения операции");