feat: add tests ids props for Calendar, CalendarRange, DateInputRange, DateInput
- close #7688
- [x] Unit-тесты
- [x] Release notes
Описание
Нужно добавить прокидывание тестовых айдишников для важных для тестирования элементов. Это нужно сделать для компонентов CalendarDay, CalendarDays, Calendar, CalendarHeader, CalendarRange, DateInput, DateRangeInput
Изменения
- Добавил необходимые свойства с id для тестов для прокидывания до всех интерактивных компонентов в
CalendarиCalendarRange - Переписал тесты для компонентов с использованием новых свойств
Release notes
Улучшения
- Calendar: Добавлены свойства для установки
data-testidу вложенных элементов - CalendarRange: Добавлены свойства для установки
data-testidу вложенных элементов - DateInput: Добавлены свойства для установки
data-testidу вложенных элементов - DateRangeInput: Добавлены свойства для установки
data-testidу вложенных элементов
size-limit report 📦
| Path | Size |
|---|---|
| JS | 384.92 KB (+0.43% 🔺) |
| JS (gzip) | 123.02 KB (+0.33% 🔺) |
| JS (brotli) | 101.96 KB (+0.29% 🔺) |
| JS import Div (tree shaking) | 1.47 KB (0%) |
| CSS | 342.28 KB (0%) |
| CSS (gzip) | 49.35 KB (0%) |
| CSS (brotli) | 40 KB (0%) |
This pull request is automatically built and testable in CodeSandbox.
To see build info of the built libraries, click here or the icon next to each commit SHA.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.38%. Comparing base (
5775640) to head (8c31c6b). Report is 2 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #7705 +/- ##
=======================================
Coverage 95.37% 95.38%
=======================================
Files 378 378
Lines 11160 11169 +9
Branches 3711 3720 +9
=======================================
+ Hits 10644 10653 +9
Misses 516 516
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 95.38% <100.00%> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Сможем в v6 закинуть?
Возможно не особо имеет смысл в v6 засовывать, так как это не баг. Плюс придется конфликты неприятные разруливать
Возможно не особо имеет смысл в v6 засовывать, так как это не баг.
Да, не баг, но мы сейчас и минорные изменения в последние патчи v6 кидаем. 🙈 Не breaking changе, зато у пользователей была бы возможность уже в v6 использовать, не закладывая время на переход на мажорный релиз v7. Конфликты, это конечно, не очень, но вдруг не всё так страшно. 🙃
Да, не баг, но мы сейчас и минорные изменения в последние патчи v6 кидаем. 🙈 Не breaking changе, зато у пользователей была бы возможность уже в v6 использовать, не закладывая время на переход на мажорный релиз v7. Конфликты, это конечно, не очень, но вдруг не всё так страшно. 🙃
В целом да, можно и в v6 зарелизить👍
PS: добавил лейбл v6, который упадёт с конфликтом, как мету инфу, что PR в v6 также отправится
❌ Patch
Не удалось автоматически применить исправление на ветке v6.
Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core
Чтобы изменение попало в ветку v6, выполните следующие действия:
- Создайте новую ветку от v6 и примените изменения используя cherry-pick
git stash # опционально
git fetch origin v6
git checkout -b patch/pr7705 origin/v6
git cherry-pick --no-commit 12e1d2b4cddbafc8dfe8883e3b38ea8869c4f193
git checkout HEAD **/__image_snapshots__/*.png
git diff --quiet HEAD || git commit --no-verify --no-edit
- Исправьте конфликты, следуя инструкциям из терминала
- Отправьте ветку на GitHub и создайте новый PR с веткой v6 (установка лейбла не требуется!)
git push --set-upstream origin patch/pr7705
gh pr create --base v6 --title "patch: pr7705" --body "- patch #7705"