Skip to content

Develop#5

Open
Anastasiaaok wants to merge 5 commits intomainfrom
develop
Open

Develop#5
Anastasiaaok wants to merge 5 commits intomainfrom
develop

Conversation

@Anastasiaaok
Copy link
Copy Markdown
Owner

Реализованы:

  • Инъекция зависимости в Lion
  • Тесты на Feline, Cat, Lion
  • Mockito
  • Параметризация
  • 100% покрытие
  • Отчёт Jacoco

Comment thread .idea/.gitignore Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⛔️Нужно исправить. Папку .idea не нужно было загружать в репозиторий. Эта папка должна быть добавлена в .gitignore.

public void getFoodTest() throws Exception {
Feline feline = Mockito.mock(Feline.class);

Mockito.when(feline.eatMeat())
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⛔️Нужно исправить. Мок должен вести себя как реальный объект и возвращать List.of("Животные", "Птицы", "Рыба");


@Test
public void getFamilyTest() {
Feline feline = new Feline();
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️Можно улучшить. Инициализацию объекта можно вынести в метод с аннотацией before

Comment thread src/test/java/com/example/LionTest.java Outdated
public class LionTest {

@Test
public void maleLionTest() throws Exception {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⛔️Нужно исправить. Проверяется в параметризированном классе. Удали дубли тестов

Comment thread target/classes/read.me Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⛔️Нужно исправить. Нужно оставить только папку site, остальное, пожалуйста, удали

}

@Parameterized.Parameters(name = "Пол: {0} → есть грива: {1}")
public static Collection<Object[]> data() {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⛔️Нужно исправить. Вижу только один тест для класса lion. Проверь, пожалуйста, пулл реквест

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants