2020-05 - junit

2026-05-30 10:50:42 +00:00 · 2020-05-26 15:39:35 +06:00
parent c4a7862dfa
commit 6fc303fdca
10 changed files with 267 additions and 0 deletions
@@ -0,0 +1,24 @@
+target/
+
+### STS ###
+.apt_generated
+.classpath
+.factorypath
+.project
+.settings
+.springBeans
+.sts4-cache
+
+### IntelliJ IDEA ###
+.idea
+*.iws
+*.iml
+*.ipr
+
+### NetBeans ###
+/nbproject/private/
+/build/
+/nbbuild/
+/dist/
+/nbdist/
+/.nb-gradle/
@@ -0,0 +1,38 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project xmlns="http://maven.apache.org/POM/4.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+
+    <groupId>ru.otus</groupId>
+    <artifactId>junit</artifactId>
+    <version>1.0</version>
+
+    <!-- Здесь находится много настроек, в частности maven-surefire-plugin -->
+    <parent>
+        <groupId>org.springframework.boot</groupId>
+        <artifactId>spring-boot-starter-parent</artifactId>
+        <version>2.3.0.RELEASE</version>
+        <relativePath/>
+    </parent>
+
+    <properties>
+        <java.version>11</java.version>
+    </properties>
+
+    <dependencies>
+        <!-- В этой зависимости и JUnit 5 (без JUnit 4), Mockito, AssertJ -->
+        <dependency>
+            <groupId>org.springframework.boot</groupId>
+            <artifactId>spring-boot-starter-test</artifactId>
+            <scope>test</scope>
+            <exclusions>
+                <exclusion>
+                    <!-- Таким образом отключается Junit 4 -->
+                    <groupId>org.junit.vintage</groupId>
+                    <artifactId>junit-vintage-engine</artifactId>
+                </exclusion>
+            </exclusions>
+        </dependency>
+    </dependencies>
+</project>
@@ -0,0 +1,16 @@
+package ru.otus.junit.dao;
+
+import ru.otus.junit.domain.Person;
+
+import java.util.List;
+
+public interface PersonDao {
+
+    Person getByName(String name) throws PersonNotFoundException;
+
+    List<Person> getAll();
+
+    void deleteByName(String name) throws PersonNotFoundException;
+
+    void save(Person person);
+}
@@ -0,0 +1,36 @@
+package ru.otus.junit.dao;
+
+import ru.otus.junit.domain.Person;
+
+import java.util.List;
+
+public class PersonDaoImpl implements PersonDao {
+
+    // TODO: добавить поле - список Person-ов
+
+    @Override
+    public Person getByName(String name) throws PersonNotFoundException {
+        // TODO: реализовать поиск в списке по имени (по технике Test-First)
+        // TODO: да, этот метод может бросать Exception, и это нужно покрыть другим тестом
+        return null;
+    }
+
+    @Override
+    public List<Person> getAll() {
+        // TODO: реализовать получние всех Person (по технике Test-First)
+        return null;
+    }
+
+    @Override
+    public void deleteByName(String name) throws PersonNotFoundException {
+        // TODO: реализовать удаление по имени (по технике Test-First)
+        // TODO: да, этот метод может бросать Exception, и это нужно покрыть другим тестом
+    }
+
+    @Override
+    public void save(Person person) {
+        // TODO: этот метод должен найти по имени в списке
+        // TODO: если такой есть - заменить
+        // TODO: если такого нет - добавить
+    }
+}
@@ -0,0 +1,8 @@
+package ru.otus.junit.dao;
+
+public class PersonNotFoundException extends RuntimeException {
+
+    public PersonNotFoundException(String message) {
+        super(message);
+    }
+}
@@ -0,0 +1,32 @@
+package ru.otus.junit.domain;
+
+@SuppressWarnings({"WeakerAccess", "unused"})
+public class Person {
+
+    private int age;
+    private String name;
+
+    public Person(int age, String name) {
+        this.age = age;
+        this.name = name;
+    }
+
+    public int getAge() {
+        return age;
+    }
+
+    public void setAge(int age) {
+    }
+
+    public String getName() {
+        return name;
+    }
+
+    public void setName(String name) {
+        this.name = name;
+    }
+
+    public void birthDay() {
+        this.age++;
+    }
+}
@@ -0,0 +1,16 @@
+package ru.otus.junit.service;
+
+import ru.otus.junit.domain.Person;
+
+import java.util.List;
+
+public interface PersonService {
+
+    Person getByName(String name);
+
+    List<Person> getAll();
+
+    boolean existsWithName(String name);
+
+    void save(Person p);
+}
@@ -0,0 +1,37 @@
+package ru.otus.junit.service;
+
+import ru.otus.junit.dao.PersonDao;
+import ru.otus.junit.domain.Person;
+
+import java.util.List;
+
+public class PersonServiceImpl implements PersonService {
+
+    private final PersonDao dao;
+
+    PersonServiceImpl(PersonDao dao) {
+        this.dao = dao;
+    }
+
+    @Override
+    public Person getByName(String name) {
+        return dao.getByName(name);
+    }
+
+    @Override
+    public List<Person> getAll() {
+        // TODO: реализовать данный метод по технике Test-First
+        return null;
+    }
+
+    @Override
+    public boolean existsWithName(String name) {
+        // TODO: реализовать данный метод по технике Test-First
+        return false;
+    }
+
+    @Override
+    public void save(Person p) {
+        // TODO: реализовать данный метод по технике Test-First
+    }
+}
@@ -0,0 +1,23 @@
+package ru.otus.junit.domain;
+
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+@DisplayName("Класс Person")
+class PersonTest {
+
+    @DisplayName("корректно создаётся конструктором")
+    @Test
+    void shouldHaveCorrectConstructor() {
+        Person person = new Person(42, "Ivan");
+
+        assertEquals("Ivan", person.getName());
+        assertEquals(42, person.getAge());
+    }
+
+    // TODO: @DisplayName("должен")
+
+    // TODO: @DisplayName("должен увеличивать возраст при вызове birthDay")
+}
@@ -0,0 +1,37 @@
+package ru.otus.junit.service;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+import ru.otus.junit.dao.PersonDao;
+import ru.otus.junit.domain.Person;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.BDDMockito.given;
+
+@ExtendWith(MockitoExtension.class)
+class PersonServiceImplTest {
+
+    @Mock
+    private PersonDao personDao;
+
+    private PersonService personService;
+
+    @BeforeEach
+    void setUp() {
+        personService = new PersonServiceImpl(personDao);
+    }
+
+    @Test
+    void getByName() {
+        // TODO: используйте eq() mapper вместо any()
+        given(personDao.getByName(any()))
+                .willReturn(new Person(10, "Ivan"));
+
+        assertThat(personService.getByName("Ivan"))
+                .isNotNull(); // TODO: сравните с помощью equals
+    }
+}