Merge pull request cdapio#255 from cloudsufi/pipelineLogs-fix

close pipeline logs fix
cloudsufi · Dec 7, 2024 · 4d27679 · 4d27679
2 parents da76ba3 + 3fccdc8
commit 4d27679
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/main/java/io/cdap/e2e/pages/actions/CdfLogActions.java b/src/main/java/io/cdap/e2e/pages/actions/CdfLogActions.java
@@ -100,6 +100,6 @@ public static void verifyPipelineFailedLogIsShownWithErrorLevel() {
    * Click on the close button(x) for the Logs page
    */
   public static void closeLogs() {
-    ElementHelper.clickOnElement(CdfLogLocators.closeLogs);
+    ElementHelper.clickIfDisplayed(CdfLogLocators.closeLogs);
   }
 }
diff --git a/src/main/java/io/cdap/e2e/pages/locators/CdfLogLocators.java b/src/main/java/io/cdap/e2e/pages/locators/CdfLogLocators.java
@@ -46,9 +46,7 @@ public class CdfLogLocators {
 
   @FindBy(xpath = "//*[contains(text(),'Pipeline') and contains(text(),'succeeded')]")
   public static WebElement validateSucceeded;
-
-  @FindBy(xpath = "//*[contains(@class, 'close')]")
-  public static WebElement closeLogs;
+  public static By closeLogs = By.xpath("//*[contains(@class, 'close')]");
 
   public static WebElement getPipelineStatusFromLogs(String status) {
     return SeleniumDriver.getDriver()