Skip to content

Commit f4df5c9

Browse files
committed
[1.4.2-minor fixes] - common fixes
1 parent 25f57ca commit f4df5c9

File tree

1 file changed

+4
-27
lines changed

1 file changed

+4
-27
lines changed

src/main/java/util/validator/ResponsiveUIValidator.java

Lines changed: 4 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -373,31 +373,31 @@ void validateBottomOffsetForChunk(List<WebElement> elements) {
373373

374374
void validateRightOffsetForElements(WebElement element, String readableName) {
375375
if (!element.equals(rootElement)) {
376-
if (!elementsHaveEqualLeftRightOffset(false, element)) {
376+
if (!elementsHaveEqualLeftRightOffset(false, rootElement, element)) {
377377
putJsonDetailsWithElement(String.format("Element '%s' has not the same right offset as element '%s'", rootElementReadableName, readableName), element);
378378
}
379379
}
380380
}
381381

382382
void validateLeftOffsetForElements(WebElement element, String readableName) {
383383
if (!element.equals(rootElement)) {
384-
if (!elementsHaveEqualLeftRightOffset(true, element)) {
384+
if (!elementsHaveEqualLeftRightOffset(true, rootElement, element)) {
385385
putJsonDetailsWithElement(String.format("Element '%s' has not the same left offset as element '%s'", rootElementReadableName, readableName), element);
386386
}
387387
}
388388
}
389389

390390
void validateTopOffsetForElements(WebElement element, String readableName) {
391391
if (!element.equals(rootElement)) {
392-
if (!elementsHaveEqualTopBottomOffset(true, element)) {
392+
if (!elementsHaveEqualTopBottomOffset(true, rootElement, element)) {
393393
putJsonDetailsWithElement(String.format("Element '%s' has not the same top offset as element '%s'", rootElementReadableName, readableName), element);
394394
}
395395
}
396396
}
397397

398398
void validateBottomOffsetForElements(WebElement element, String readableName) {
399399
if (!element.equals(rootElement)) {
400-
if (!elementsHaveEqualTopBottomOffset(false, element)) {
400+
if (!elementsHaveEqualTopBottomOffset(false, rootElement, element)) {
401401
putJsonDetailsWithElement(String.format("Element '%s' has not the same bottom offset as element '%s'", rootElementReadableName, readableName), element);
402402
}
403403
}
@@ -893,17 +893,6 @@ private boolean elementsAreOverlapped(WebElement rootElement, WebElement element
893893
|| elementsAreOverlappedOnBorder(rootElement, elementOverlapWith);
894894
}
895895

896-
private boolean elementsHaveEqualLeftRightOffset(boolean isLeft, WebElement elementToCompare) {
897-
Point elLoc = elementToCompare.getLocation();
898-
Dimension elSize = elementToCompare.getSize();
899-
900-
if (isLeft) {
901-
return xRoot == elLoc.getX();
902-
} else {
903-
return (pageWidth - xRoot + widthRoot) == (pageWidth - elLoc.getX() + elSize.getWidth());
904-
}
905-
}
906-
907896
private boolean elementsHaveEqualLeftRightOffset(boolean isLeft, WebElement element, WebElement elementToCompare) {
908897
Point elLoc = elementToCompare.getLocation();
909898
Dimension elSize = elementToCompare.getSize();
@@ -917,18 +906,6 @@ private boolean elementsHaveEqualLeftRightOffset(boolean isLeft, WebElement elem
917906
}
918907
}
919908

920-
921-
private boolean elementsHaveEqualTopBottomOffset(boolean isTop, WebElement elementToCompare) {
922-
Point elLoc = elementToCompare.getLocation();
923-
Dimension elSize = elementToCompare.getSize();
924-
925-
if (isTop) {
926-
return yRoot == elLoc.getY();
927-
} else {
928-
return (pageHeight - yRoot + heightRoot) == (pageHeight - elLoc.getY() + elSize.getHeight());
929-
}
930-
}
931-
932909
private boolean elementsHaveEqualTopBottomOffset(boolean isTop, WebElement element, WebElement elementToCompare) {
933910
Point elLoc = elementToCompare.getLocation();
934911
Dimension elSize = elementToCompare.getSize();

0 commit comments

Comments
 (0)