Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(23UG-69): добавляет тесты #38

Merged
merged 7 commits into from
Mar 25, 2023
Merged

Conversation

cap-Bernardito
Copy link
Contributor

@cap-Bernardito cap-Bernardito commented Mar 24, 2023

Какую задачу решаем

  • env переменная VITE_API_ENDPOINT переименована в API_ENDPOINT (нужно добавить на хостинге)
  • настроен jest
  • для auth-slice добавлены тесты синхронных и асинхронных экшонов
  • для App добавлены проверки перехода на страницы при клике по ссылкам навигационного меню
  • добавлены тесты для ErrorBoundary
  • на ci добавлено создание файла .env

В задепплоенной версии пользователь не отображается, т.к. VITE_API_ENDPOINT была переименована и ломаются запросы к API. Починится, когда на хостинге исправим/добавим. Вообще это конечно баг, так быть не должно, нужна валидация ответа от API. Добавил todo на исправление в тасктрекер.

@vercel
Copy link

vercel bot commented Mar 24, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
uno-game-client ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 24, 2023 at 10:15AM (UTC)

@cap-Bernardito cap-Bernardito requested a review from Olegas March 24, 2023 19:06
@cap-Bernardito cap-Bernardito merged commit 5f58057 into dev Mar 25, 2023
@cap-Bernardito cap-Bernardito deleted the dev-tests branch March 25, 2023 07:29
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.

5 participants