Skip to content

YH-2130: Added mock error handling when deleting a skill#1303

Merged
PerelomaDenis merged 4 commits into
developfrom
feature/YH-2130
Jun 24, 2026
Merged

YH-2130: Added mock error handling when deleting a skill#1303
PerelomaDenis merged 4 commits into
developfrom
feature/YH-2130

Conversation

@IrinaYarutkina

Copy link
Copy Markdown
Collaborator

https://tracker.yandex.ru/YH-2130
В deleteSkillMock добавлена обработка ошибок при условиях, когда

  • Пользователь авторизован
  • Пользователь верифицирован
  • Навык не найден
  • Навык привязан к какой то специализации
  • Навык привязан к какому то вопросу
  • Навык указан в каком то профиле
  • Навык привязан к какой то другой сущности не относящихся к 4-6 пунктам - ресурсы и темы

@IrinaYarutkina IrinaYarutkina added review the task is currently being reviewed feature New functionality or improvements labels Jun 20, 2026
Comment thread src/features/skill/deleteSkill/api/__mocks__/deleteSkillMock.ts Outdated
Comment thread src/features/skill/deleteSkill/api/__mocks__/deleteSkillMock.ts Outdated
Comment thread src/features/skill/deleteSkill/api/__mocks__/deleteSkillMock.ts Outdated
@PerelomaDenis PerelomaDenis added comments Code improvements are required and removed review the task is currently being reviewed labels Jun 21, 2026
PerelomaDenis
PerelomaDenis previously approved these changes Jun 24, 2026
# Conflicts:
#	src/features/skill/deleteSkill/api/__mocks__/deleteSkillMock.ts
@PerelomaDenis PerelomaDenis added merge Can merge this branch and removed comments Code improvements are required labels Jun 24, 2026
@PerelomaDenis PerelomaDenis merged commit e8d3e2d into develop Jun 24, 2026
1 check passed
@PerelomaDenis PerelomaDenis deleted the feature/YH-2130 branch June 24, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New functionality or improvements merge Can merge this branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants