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

[Critical] Удалить несуществующий метод data-qa #1659

Open
mobilisf opened this issue Feb 23, 2024 · 8 comments · Fixed by #1737 or #1761
Open

[Critical] Удалить несуществующий метод data-qa #1659

mobilisf opened this issue Feb 23, 2024 · 8 comments · Fixed by #1737 or #1761
Assignees
Labels
bug Something isn't working PizzaFeedback tasks related to user's feedback
Milestone

Comments

@mobilisf
Copy link

mobilisf commented Feb 23, 2024

Environment details (please complete the following information):
JDN v 3.15.2 Back-end v 0.2.64

Describe the bug
Убрать несуществующие (тестовые qa) методы data-qa и data-qa-id.

To Reproduce

  1. Запустить Вивидус на сайте https://www.otto.de/damen/
  2. Отфильтровать по TextArea
  3. Сохранить с data-qa или data-qa-id
    image
  4. Локатор изменился - проблема в том, что метода dataQaId не существует
    image

Expected result
На самом деле - для вивидуса такие дата атрибуты нужно давать пользователю выбор из двух вариатов

  1. variables.DamenmodeOnlineKaufenFashionShopFuerDamenOttoPage.TextArea.textarea=By.cssSelector(*[data-qa-id='search-field'])
  2. variables.DamenmodeOnlineKaufenFashionShopFuerDamenOttoPage.TextArea.textarea=By.xPath(//*[@data-qa-id='search-field'])

Это должно отображаться в выпадушке выбора типа локатора.
Сейчас там - image
А должно быть две опции
CSS Selector for data-qa-id "search-field"
и
xPath for data-qa-id "search-field"

Выбор одного из вариантов соответственно превращается в

  1. variables.DamenmodeOnlineKaufenFashionShopFuerDamenOttoPage.TextArea.textarea=By.cssSelector(*[data-qa-id='search-field'])
  2. variables.DamenmodeOnlineKaufenFashionShopFuerDamenOttoPage.TextArea.textarea=By.xPath(//*[@data-qa-id='search-field'])
    соответственно
@mobilisf mobilisf added the bug Something isn't working label Feb 23, 2024
@AlexeyGirin AlexeyGirin added this to the Release 3.16 milestone Feb 23, 2024
@AlexeyGirin AlexeyGirin added the PizzaFeedback tasks related to user's feedback label Feb 23, 2024
@savinovava savinovava changed the title Удалить несуществующий метод data-qa [Critical] Удалить несуществующий метод data-qa Feb 26, 2024
@Iogsotot Iogsotot linked a pull request May 17, 2024 that will close this issue
@Iogsotot
Copy link
Contributor

Iogsotot commented May 17, 2024

#1737
3.15.33

Метод data-qa оставили, изменили его отображение для Vividus

@Iogsotot
Copy link
Contributor

Временно отключена фича с расчетом CSS selector на бэкенде и download all as zip (см #1284 #1734)
Без этих фич можно смотреть билд версии 3.15.34

@Iogsotot
Copy link
Contributor

Включены фичи с расчетом CSS selector на бэкенде и download all as zip (см #1284 #1734) для локального бэкенда ( issue_1284-backend-calculation-of-css-selectors -l pr-192)
3.15.35

@Iogsotot
Copy link
Contributor

Iogsotot commented Jun 5, 2024

закрыто, тк @mobilisf не нашёл проблем

@Iogsotot Iogsotot closed this as completed Jun 5, 2024
@savinovava savinovava moved this to Done in JDN.AI Board Jun 20, 2024
@AlexeyGirin AlexeyGirin moved this from Done to To do in JDN.AI Board Jun 20, 2024
@AlexeyGirin
Copy link
Contributor

Not fixed.
For VIVDUS option - data attributes are missed:
image

To discuss:
For JDI Light option - we offer only css variant for data attribute. At the same time xPath is also possible

@AlexeyGirin AlexeyGirin reopened this Jun 20, 2024
@Iogsotot
Copy link
Contributor

Iogsotot commented Jun 24, 2024

если у компонента 5 data-attr, то в режиме Vividus мы хотим предоставлять пользователю выбор из 6 xPath и 6 Css Selectors (5 data и 1 оригинальный сгенерированный на бэке)?

текущая реализация - 1 xPath и 1 Css Selector вне зависимости от количества data-attr (берутся с бэка, не гененирируются на фронте)

Пример:
Снимок экрана 2024-06-24 в 10 45 54
Снимок экрана 2024-06-24 в 10 47 49

@AlexeyGirin
Copy link
Contributor

если у компонента 5 data-attr, то в режиме Vividus мы хотим предоставлять пользователю выбор из 6 xPath и 6 Css Selectors (5 data и 1 оригинальный сгенерированный на бэке)?

текущая реализация - 1 xPath и 1 Css Selector вне зависимости от количества data-attr (берутся с бэка, не гененирируются на фронте)

Пример: Снимок экрана 2024-06-24 в 10 45 54 Снимок экрана 2024-06-24 в 10 47 49

Да, нужно давать выбрать любой дата атрибут

@Iogsotot Iogsotot moved this from To do to In progress in JDN.AI Board Jun 24, 2024
@Iogsotot Iogsotot moved this from In progress to To do in JDN.AI Board Jul 2, 2024
@AlexeyGirin AlexeyGirin moved this from To do to QA in JDN.AI Board Jul 3, 2024
@AlexeyGirin AlexeyGirin moved this from QA to In progress in JDN.AI Board Jul 3, 2024
@Iogsotot Iogsotot pinned this issue Jul 25, 2024
@Iogsotot Iogsotot linked a pull request Aug 1, 2024 that will close this issue
@Iogsotot
Copy link
Contributor

Iogsotot commented Aug 1, 2024

#1761
3.16.2

@Iogsotot Iogsotot moved this from In progress to QA in JDN.AI Board Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working PizzaFeedback tasks related to user's feedback
Projects
Status: No status
Status: QA
3 participants