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

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

Merged

Conversation

EldarMuhamethanov
Copy link
Contributor


  • Unit-тесты
  • 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 EldarMuhamethanov requested a review from a team as a code owner October 4, 2024 16:42
Copy link
Contributor

github-actions bot commented Oct 4, 2024

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

Copy link

codesandbox-ci bot commented Oct 4, 2024

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.

Copy link
Contributor

github-actions bot commented Oct 4, 2024

e2e tests

Playwright Report

Copy link
Contributor

github-actions bot commented Oct 4, 2024

👀 Docs deployed

Commit c36d960

Copy link

codecov bot commented Oct 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vkcom-publisher vkcom-publisher added pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности and removed pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности labels Oct 12, 2024
Copy link
Contributor

@andrey-medvedev-vk andrey-medvedev-vk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Класс 👏 👏 👏

Поправишь, пожалуйста, конфликты?

…dars

# Conflicts:
#	packages/vkui/src/components/CalendarDay/CalendarDay.test.tsx
#	packages/vkui/src/components/CalendarRange/CalendarRange.test.tsx
Copy link
Contributor

@andrey-medvedev-vk andrey-medvedev-vk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💅

@vkcom-publisher vkcom-publisher added pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности and removed pr-needs-work Автоматизация: PR автоматически закроется через 14 дней при отсутствии активности labels Oct 29, 2024
…dars

# Conflicts:
#	packages/vkui/src/components/CalendarTime/CalendarTime.tsx
#	packages/vkui/src/components/DateInput/DateInput.tsx
Copy link
Contributor

@andrey-medvedev-vk andrey-medvedev-vk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@andrey-medvedev-vk andrey-medvedev-vk requested a review from a team November 6, 2024 13:00
@EldarMuhamethanov
Copy link
Contributor Author

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

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

@andrey-medvedev-vk
Copy link
Contributor

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

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

@EldarMuhamethanov
Copy link
Contributor Author

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

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

@inomdzhon inomdzhon added patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча v6 Автоматизация: PR продублируется в ветку v6 and removed patch Автоматизация: PR продублируется в ветку последнего минорного релиза для выпуска патча labels Nov 7, 2024
@inomdzhon
Copy link
Contributor

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

@EldarMuhamethanov EldarMuhamethanov merged commit 12e1d2b into master Nov 7, 2024
29 checks passed
@EldarMuhamethanov EldarMuhamethanov deleted the e.muhamethanov/7688/add-test-ids-for-calendars branch November 7, 2024 09:54
@vkcom-publisher
Copy link
Contributor

❌ 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"

EldarMuhamethanov added a commit that referenced this pull request Nov 7, 2024
…, DateInput (#7705)

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

* fix(CalendarRange): fix tests

* fix: fix tests after merge conflicts

(cherry picked from commit 12e1d2b)
EldarMuhamethanov added a commit that referenced this pull request Nov 7, 2024
…, DateInput (#7705)

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

* fix(CalendarRange): fix tests

* fix: fix tests after merge conflicts

(cherry picked from commit 12e1d2b)
This was referenced Nov 7, 2024
EldarMuhamethanov added a commit that referenced this pull request Nov 7, 2024
…, DateInput (#7705) (#7912)

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

* fix(CalendarRange): fix tests

* fix: fix tests after merge conflicts

(cherry picked from commit 12e1d2b)
EldarMuhamethanov added a commit that referenced this pull request Nov 7, 2024
…, DateInput (#7705) (#7911)

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

* fix(CalendarRange): fix tests

* fix: fix tests after merge conflicts

(cherry picked from commit 12e1d2b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v6 Автоматизация: PR продублируется в ветку v6
Projects
None yet
5 participants