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

Улучшение для BladeTest №7 #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

neonicotinoid
Copy link

Проблема:
текущий тест BladeTest.php №7 просто проверяет наличие class="bg-red-500" на странице. Он покажет положительный результат даже если всем блокам добавить класс bg-red-500 (не зависимости от чётных или нечетных).

Предлагаемое решение:

  • Дополняем задачу – не просто добавляем bg-red-500 для нечетных классов, а добавляем ещё и класс bg-blue-500 для четных.
  • В тесте проверяем не просто наличие классов на странице, но и их правильный порядок (10 юзеров, 10 классов в нужном порядке)

Навыки, проверяемые в задаче, не изменятся – это умение работать с циклами в Blade, работать с индексом цикла и добавлять разные классы в зависимости от условий. Хотя сама задача станет чуть посложнее.

@lee-to
Copy link
Owner

lee-to commented Apr 13, 2022

Привет, спасибо за PR но мне не очень нравится код где дублирование классов в несколько строк, сделай через репит чтобы не повторять одно и тоже

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