5378 Fix failing tests for Snackbar by igor-korost · Pull Request #5379 · jdi-testing/jdi-light

igor-korost · 2024-01-26T10:20:00Z

No description provided.

pnatashap · 2024-01-26T14:37:19Z

...angular-tests/src/test/java/io/github/epam/angular/tests/elements/complex/SnackbarTests.java

+        snackBarPage.durationInput.setValue(String.valueOf(DURATION));
+        snackBarPage.customSnackbarOpenButton.click();

        //duration(DURATION, 1000, action);


удалить

Сделано

pnatashap · 2024-01-26T14:44:05Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

-    protected String messageLocator = "./span";
+    protected String messageLocator = "//*[@matsnackbarlabel]";

    protected UIElement action;


это кнопка, а не UIElement

Заменил protected UIElement action на public Button action;
Модификатор доступа изменил на public так как action после правок используется в SnackbarTests

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

pnatashap · 2024-01-26T14:44:56Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

+        return message.getText();
    }

    @JDIAction("Get '{name}' action")


и ниже все методы тоже требуют переименования

Переименовал getMessageText() в messageText() и getActionText() в actionText()

...angular-tests/src/test/java/io/github/epam/angular/tests/elements/complex/SnackbarTests.java

pnatashap · 2024-01-26T14:47:49Z

...angular-tests/src/test/java/io/github/epam/angular/tests/elements/complex/SnackbarTests.java

+        snackBarPage.openButton.click();

-        snackbarSection.basicSnackbar.has().action();
+        snackBarPage.basicSnackbar.has().action();


тест про NO ACTION TEST
но в валидации проверяем, что action есть - не логично

Переименовал. Теперь эта строчка выглядит так snackBarPage.basicSnackbar.has().noAction();

…ments/complex/Snackbar.java Co-authored-by: Natalia Pozhidaeva <Natalia_Pozhidaeva@epam.com>

…ests/elements/complex/SnackbarTests.java Co-authored-by: Natalia Pozhidaeva <Natalia_Pozhidaeva@epam.com>

pnatashap

ревью компонента еще раз нужно сделать

pnatashap · 2024-02-02T03:21:24Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

-    protected UIElement action;
-    protected String actionLocator = ".//button";
+    public Button action;
+    protected String actionLocator = "//button";


а было правильно

Вернул назад

pnatashap · 2024-02-02T03:23:41Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

 public class Snackbar extends UIBaseElement<SnackbarAssert> {
    protected UIElement message;
-    protected String messageLocator = "./span";
+    protected String messageLocator = "//*[@matsnackbarlabel]";


стиль mdc-snackbar__label гораздо более понятнее смотрится

Пофиксил. Теперь так protected String messageLocator = ".mdc-snackbar__label";

pnatashap · 2024-02-02T03:31:19Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

+    protected String actionLocator = "//button";

    public Snackbar() {
        message = new UIElement();


А зачем нам эти элементы хранить?

После введения проверок из элемента Button кнопка action теперь используется в тестах (сейчас кнопка переименована в actionButton). Если я правильно понял вопрос.

вопрос был зачем это делать в конструкторе, а не так же, как это сделано в остальных элементах

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

pnatashap · 2024-02-02T03:36:41Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java


    @JDIAction("Get '{name}' action")
-    public String getActionText() {
+    public String actionText() {


на самом деле при наличии доступа к кнопки мы можем на ней и проверить, что с ней, на кнопке есть свои валидаторы

Выполнено. Теперь во всех тестах SnackbarTests используются валидаторы из Button

pnatashap · 2024-02-11T04:46:20Z

@igor-korost если работа закончена, то надо перезапрашивать ревью, мысли еще никто читать не научился

pnatashap

а почему PR называется исправление падающих тестов?

pnatashap · 2024-02-11T04:47:30Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

+    protected String actionLocator = "//button";

    public Snackbar() {
        message = new UIElement();


вопрос был зачем это делать в конструкторе, а не так же, как это сделано в остальных элементах

pnatashap · 2024-02-11T04:48:20Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java


-        action = new UIElement();
-        action.core().setLocator(actionLocator);
+        actionButton = new Button();


кнопок технически можно сделать несколько, надо предусмотреть возможность с ними работать

У элементов snackbar при нормальном проектировании может быть только 1 кнопка или не быть её вообще. Так как это исчезающий элемент и можно не успеть отреагировать. 💁 Вот тут подробнее написано https://stackoverflow.com/questions/67642607/reactmaterial-ui-multiple-snackbar

Конструктор убрал из Snackbar.java ✅

pnatashap · 2024-02-11T04:49:15Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/elements/complex/Snackbar.java

 */

 public class Snackbar extends UIBaseElement<SnackbarAssert> {
    protected UIElement message;


если message это элемент с текстом, то это должен быть не UIElement

Заменил на public String messageText(), конструктор убрал вообще

pnatashap · 2024-02-11T04:50:29Z

jdi-light-angular/src/main/java/com/epam/jdi/light/angular/asserts/SnackbarAssert.java

@@ -11,19 +11,7 @@ public class SnackbarAssert extends UIAssert<SnackbarAssert, Snackbar> {

    @JDIAction(value = "Assert that '{name}' has message '{0}'", isAssert = true)
    public SnackbarAssert message(String expected) {


на текст всегда ставим валидатор с матчером, чтобы не только на полное соответствие можно было проверить

Заменил на Matchers.containsString(expected)

… 5378_fix_snackbar_tests

igor-korost · 2024-03-01T04:25:59Z

а почему PR называется исправление падающих тестов?

Название не вполне верное. Скорее это рефакторинг и исправление. Цель этого PR сдать уже сделанную работу поскольку я ухожу с проекта.

This reverts commit 3058d29.

5378: Fix failing tests for Snackbar

bffd263

igor-korost added Angular Refactoring code refactoring ReWork labels Jan 26, 2024

igor-korost self-assigned this Jan 26, 2024

igor-korost changed the title ~~5378: Fix failing tests for Snackbar~~ 5378 Fix failing tests for Snackbar Jan 26, 2024

pnatashap requested changes Jan 26, 2024

View reviewed changes

pnatashap linked an issue Jan 26, 2024 that may be closed by this pull request

Tests refactoring: fix Snackbar element tests #5378

Closed

3 tasks

igor-korost and others added 3 commits January 31, 2024 19:47

Update jdi-light-angular/src/main/java/com/epam/jdi/light/angular/ele…

5bcaab1

…ments/complex/Snackbar.java Co-authored-by: Natalia Pozhidaeva <Natalia_Pozhidaeva@epam.com>

Update jdi-light-angular-tests/src/test/java/io/github/epam/angular/t…

952c65c

…ests/elements/complex/SnackbarTests.java Co-authored-by: Natalia Pozhidaeva <Natalia_Pozhidaeva@epam.com>

5378 Fix review

234d252

pnatashap requested changes Feb 2, 2024

View reviewed changes

igor-korost added 2 commits February 2, 2024 14:50

5378 Fix locator

c501379

5378 Fix review

16c6929

pnatashap requested changes Feb 11, 2024

View reviewed changes

igor-korost added 3 commits February 28, 2024 21:28

5378 Fix review

e919568

Merge remote-tracking branch 'origin/angular_rework_development' into…

1057f50

… 5378_fix_snackbar_tests

5378 Fix typo

90b4fe2

igor-korost mentioned this pull request Feb 28, 2024

Tests refactoring: Snackbar element #4942

Open

3 tasks

igor-korost added 2 commits March 6, 2024 15:35

Update Cypress

3058d29

Revert "Update Cypress"

c6e5fb5

This reverts commit 3058d29.

		@@ -11,19 +11,7 @@ public class SnackbarAssert extends UIAssert<SnackbarAssert, Snackbar> {

		@JDIAction(value = "Assert that '{name}' has message '{0}'", isAssert = true)
		public SnackbarAssert message(String expected) {

Conversation

igor-korost commented Jan 26, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pnatashap left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

pnatashap commented Feb 11, 2024

Uh oh!

pnatashap left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

igor-korost commented Mar 1, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants