Skip to content

werewolf/BJS2-95842-add-unit-tests#3428

Open
Derzhko wants to merge 11 commits intowerewolf-master-stream12from
werewolf/BJS2-95842-add-unit-tests
Open

werewolf/BJS2-95842-add-unit-tests#3428
Derzhko wants to merge 11 commits intowerewolf-master-stream12from
werewolf/BJS2-95842-add-unit-tests

Conversation

@Derzhko
Copy link
Copy Markdown

@Derzhko Derzhko commented Oct 17, 2025

No description provided.

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

@github-actions
Copy link
Copy Markdown

⚠️ ОШЫБКА: Сборка завалилась: либо ошибки компиляции, либо не прошли тесты, либо возникли ошибки в стиле кода. Пожалуйста, проверь логи и внеси соответствующие изменения в ПР. После этого красная ошибка ниже должна пропасть, но этот комментарий останется - так и должно быть.

Copy link
Copy Markdown

@YasinJD YasinJD left a comment

Choose a reason for hiding this comment

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

Придраться не к чему - всё чисто. Мне нравится как контроллер с сервисом, так и тесты.

import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.times;
import static org.mockito.Mockito.when;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лишняя пустая строка

import school.faang.user_service.repository.user.SkillRepository;

import java.util.List;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Лишняя пустая строка


@ExtendWith(MockitoExtension.class)
public class SkillServiceImplTest {
@Value("${skill.offers.min.count}")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Зачем оно тут? В этом же тесте нет спринга

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.

3 participants