VKUI icon indicating copy to clipboard operation
VKUI copied to clipboard

feat: add tests ids props for Calendar, CalendarRange, DateInputRange, DateInput

Open EldarMuhamethanov opened this issue 1 year ago • 5 comments

  • 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 у вложенных элементов

EldarMuhamethanov avatar Oct 04 '24 16:10 EldarMuhamethanov

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%)

github-actions[bot] avatar Oct 04 '24 16:10 github-actions[bot]

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.

codesandbox-ci[bot] avatar Oct 04 '24 16:10 codesandbox-ci[bot]

e2e tests

Playwright Report

github-actions[bot] avatar Oct 04 '24 16:10 github-actions[bot]

👀 Docs deployed

Commit c36d9602db31c50120e970083adef9f0ba3b2458

github-actions[bot] avatar Oct 04 '24 16:10 github-actions[bot]

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.

codecov[bot] avatar Oct 04 '24 16:10 codecov[bot]

Сможем в v6 закинуть?

Возможно не особо имеет смысл в v6 засовывать, так как это не баг. Плюс придется конфликты неприятные разруливать

EldarMuhamethanov avatar Nov 06 '24 13:11 EldarMuhamethanov

Возможно не особо имеет смысл в v6 засовывать, так как это не баг.

Да, не баг, но мы сейчас и минорные изменения в последние патчи v6 кидаем. 🙈 Не breaking changе, зато у пользователей была бы возможность уже в v6 использовать, не закладывая время на переход на мажорный релиз v7. Конфликты, это конечно, не очень, но вдруг не всё так страшно. 🙃

andrey-medvedev-vk avatar Nov 06 '24 13:11 andrey-medvedev-vk

Да, не баг, но мы сейчас и минорные изменения в последние патчи v6 кидаем. 🙈 Не breaking changе, зато у пользователей была бы возможность уже в v6 использовать, не закладывая время на переход на мажорный релиз v7. Конфликты, это конечно, не очень, но вдруг не всё так страшно. 🙃

В целом да, можно и в v6 зарелизить👍

EldarMuhamethanov avatar Nov 07 '24 07:11 EldarMuhamethanov

PS: добавил лейбл v6, который упадёт с конфликтом, как мету инфу, что PR в v6 также отправится

inomdzhon avatar Nov 07 '24 09:11 inomdzhon

❌ Patch

Не удалось автоматически применить исправление на ветке v6.

Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core

Чтобы изменение попало в ветку v6, выполните следующие действия:

  1. Создайте новую ветку от 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
  1. Исправьте конфликты, следуя инструкциям из терминала
  2. Отправьте ветку на GitHub и создайте новый PR с веткой v6 (установка лейбла не требуется!)
git push --set-upstream origin patch/pr7705
gh pr create --base v6 --title "patch: pr7705" --body "- patch #7705"

vkcom-publisher avatar Nov 07 '24 09:11 vkcom-publisher