Merge pull request #5327 from jdi-testing/check_mat_ui_pages

The list of the tasks to check in Material UI
jdi-testing · Jan 14, 2024 · c7fcea4 · c7fcea4
2 parents 27838d8 + 10a9a12
commit c7fcea4
Show file tree

Hide file tree

Showing 16 changed files with 17 additions and 1 deletion.
diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/displaydata/BadgePage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/displaydata/BadgePage.java
@@ -18,6 +18,7 @@ public class BadgePage extends WebPage {
     @UI("#secondaryColorBadge")
     public static Badge secondaryColorBadgeContainer;
 
+    // @todo #5297 Button should be used from ButtomGruop, not directly as single
     @UI(".MuiButtonGroup-root  button[aria-label='reduce']")
     public static Button buttonReduce;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/displaydata/ChipsPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/displaydata/ChipsPage.java
@@ -18,9 +18,11 @@ public class ChipsPage extends WebPage {
     @UI("#lastDeleteInfo")
     public static Text lastDeleteBasicInfo;
 
+    // @todo #5297 Fix locator, it is not stable
     @UI("//h2[text()='Outlined Chips']/following-sibling::div/div/div[contains(@class, 'MuiChip-root')]")
     public static List<Chip> outlinedChips;
 
+    // @todo #5297 Fix to css locator
     @UI("//a[contains(@class, 'MuiChip-root')]")
     public static List<Chip> linkChips;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/displaydata/IconsPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/displaydata/IconsPage.java
@@ -12,6 +12,7 @@ public class IconsPage extends WebPage {
     @UI(".MuiGrid-grid-xs-8 > svg")
     public static List<Icon> simpleIcons;
 
+    // @todo #5297 Fix locator to be more stable
     @UI("//p[@id='colorLastClick']/..//div/*")
     public static List<Icon> colorIcons;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/feedback/ProgressPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/feedback/ProgressPage.java
@@ -13,6 +13,7 @@ public class ProgressPage extends WebPage {
     @JProgress(root = "#circularIndeterminateProgress")
     public static CircularProgress circularProgressIndeterminate;
 
+    // @todo #5297 Check locators with aria-valuenow, very stange to use them
     @JProgress(root = "div[aria-valuenow='25'][1]")
     public static CircularProgress circularProgressDeterminateWithValue25;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/inputs/ButtonGroupPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/inputs/ButtonGroupPage.java
@@ -9,6 +9,7 @@
 
 public class ButtonGroupPage extends WebPage {
 
+    // @todo #5297 Check locators, the class is the same is for Button
     @UI("div[aria-label = 'outlined primary button group'] .MuiButton-root")
     public static ButtonGroup basicButtonGroup;
 

diff --git a/...light-material-ui-tests/src/main/java/io/github/com/pages/inputs/DateTimePickersPage.java b/...light-material-ui-tests/src/main/java/io/github/com/pages/inputs/DateTimePickersPage.java
@@ -7,6 +7,7 @@
 
 public class DateTimePickersPage extends WebPage {
 
+    // @todo #5297 Check if locators with classses can be used
     @UI("//*[@id = 'date-picker-inline-label']/parent::div")
     public static DateTimePicker inlineDatePicker;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/inputs/TextFieldPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/inputs/TextFieldPage.java
@@ -9,6 +9,7 @@
 
 public class TextFieldPage extends WebPage {
 
+    // @todo #5297 Fix locators for more stable variant
     @UI("//input[@id='standard-required']/../..")
     public static TextField requiredTextField;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/layout/BoxPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/layout/BoxPage.java
@@ -7,6 +7,7 @@
 
 public class BoxPage extends WebPage {
 
+    // @todo #5297 Do not see any Box class, do we really need this page?
     @UI(".MuiButton-contained")
     public static Button buttonContainedBox;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/layout/HiddenPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/layout/HiddenPage.java
@@ -6,6 +6,7 @@
 import com.epam.jdi.light.ui.html.elements.common.Text;
 
 public class HiddenPage extends WebPage {
+    // @todo #5297 Check the page usage
     @UI(".MuiTypography-subtitle1")
     public static Text currentWidth;
 

diff --git a/...light-material-ui-tests/src/main/java/io/github/com/pages/navigation/ContextMenuPage.java b/...light-material-ui-tests/src/main/java/io/github/com/pages/navigation/ContextMenuPage.java
@@ -10,6 +10,7 @@ public class ContextMenuPage extends WebPage {
     @UI("p.MuiTypography-root")
     public static Text pageText;
 
+    // @todo #5297 Check the locator, looks like it points to item, not for Menu
     @UI(".MuiMenuItem-root")
     public static Menu contextMenuList;
 }
diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/navigation/TabPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/navigation/TabPage.java
@@ -10,6 +10,7 @@ public class TabPage extends WebPage {
     @UI("h2+div[1] .MuiTabs-root")
     public static Tabs simpleTabs;
 
+    // @todo #5297 Fix locator to be more stable
     @UI("//div[./h2[text()='Simple tabs']]/div[1]/div[@role='tabpanel' and not(@hidden)]")
     public static Container simplePanels;
 

diff --git a/...t-material-ui-tests/src/main/java/io/github/com/pages/navigation/TemporaryDrawerPage.java b/...t-material-ui-tests/src/main/java/io/github/com/pages/navigation/TemporaryDrawerPage.java
@@ -6,6 +6,7 @@
 
 public class TemporaryDrawerPage extends WebPage {
 
+    // @todo #5297 Check the locators for drawer, looks like they must be relative, while they a written as absolute
     @JButtonWithDrawer(
             root = "//span[text() = 'left']/parent::button",
             drawer = "//div[contains(@class, 'MuiDrawer-root')][./div[contains(@class, 'MuiDrawer-paperAnchorLeft')]]"

diff --git a/...ght-material-ui-tests/src/main/java/io/github/com/pages/surfaces/BackToTopAppBarPage.java b/...ght-material-ui-tests/src/main/java/io/github/com/pages/surfaces/BackToTopAppBarPage.java
@@ -7,6 +7,7 @@
 
 public class BackToTopAppBarPage extends WebPage {
 
+    // @todo #5297 Check the locator, looks like it is not for AppBar (or use correct classname)
     @UI(".MuiBox-root[1]")
     public static AppBar backToTopAppBar;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/surfaces/CardPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/surfaces/CardPage.java
@@ -18,6 +18,7 @@ public class CardPage extends WebPage {
     @UI("#complexInteraction")
     public static ComplexInteractionCard complexCard;
 
+    // @todo #5297 Class should be created to describe the card with image or image is a legal part of the card
     @UI("#complexInteraction .MuiCardMedia-root")
     public static Image complexCardImage;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/utils/PopoverHoverFrame.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/utils/PopoverHoverFrame.java
@@ -7,7 +7,7 @@
 
 
 public class PopoverHoverFrame extends Section {
-
+    // @todo #5297 Page is not used, check it
     @UI(".MuiTypography-root")
     public Typography labelToHover;
 

diff --git a/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/utils/TransitionPage.java b/jdi-light-material-ui-tests/src/main/java/io/github/com/pages/utils/TransitionPage.java
@@ -9,6 +9,7 @@
 
 public class TransitionPage extends WebPage {
 
+    // @todo #5297 Check the locators
     @UI("//h2[text()='Collapse']/following::div[contains(@class,'MuiCollapse-root')]")
     public static List<Transition> collapseTransitions;