[#2532][#2512] fix(IT): fix the web e2e test errors when running on the github actions

docs/webui.md

@@ -295,3 +295,21 @@ Design draft preview:
 | Catalog  | _`View`_ ✔ / _`Create`_ ✔ / _`Edit`_ ✔ / _`Delete`_ ✔ |
 | Schema   | _`View`_ ✔ / _`Create`_ ✘ / _`Edit`_ ✘ / _`Delete`_ ✘ |
 | Table    | _`View`_ ✔ / _`Create`_ ✘ / _`Edit`_ ✘ / _`Delete`_ ✘ |
+
+## E2E test
+End-to-end testing for web frontends is conducted using the [Selenium](https://www.selenium.dev/documentation/) testing framework, which is Java-based.
+
+Test cases can be found in the project directory: `integration-test/src/test/java/com/datastrato/gravitino/integration/test/web/ui`, where the `pages` directory is designated for storing definitions of frontend elements, among others.
+The root directory contains the actual steps for the test cases.
+
+:::tip
+ While writing test cases, running them in a local environment may not pose any issues.
+
+ However, due to the limited performance capabilities of GitHub Actions, scenarios involving delayed DOM loading—such as the time taken for a popup animation to open—can result in test failures.
+
+ To circumvent this issue, it is necessary to manually insert a delay operation, for instance, by adding such as `Thread.sleep(sleepTimeMillis)`.
+
+ This ensures that the test waits for the completion of the delay animation before proceeding with the next operation, thereby avoiding the problem.
+
+ It is advisable to utilize the [`waits`](https://www.selenium.dev/documentation/webdriver/waits/) methods inherent to Selenium as a substitute for `Thread.sleep()`.
+:::

integration-test/src/test/java/com/datastrato/gravitino/integration/test/web/ui/MetalakePageTest.java

@@ -4,7 +4,7 @@
  */
 package com.datastrato.gravitino.integration.test.web.ui;
 
-import com.datastrato.gravitino.integration.test.web.ui.Pages.MetalakePage;
+import com.datastrato.gravitino.integration.test.web.ui.pages.MetalakePage;
 import com.datastrato.gravitino.integration.test.web.ui.utils.AbstractWebIT;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.MethodOrderer;
@@ -16,16 +16,19 @@
 public class MetalakePageTest extends AbstractWebIT {
   MetalakePage metalakePage = new MetalakePage();
 
+  String metalakeName = "metalake_name";
+  String editedMetalakeName = metalakeName + "_edited";
+
   // Create a metalake by name, set the default comment and properties.
-  public void createMetalakeAction(String name) {
-    metalakePage.createMetalakeBtn.click();
+  public void createMetalakeAction(String name) throws InterruptedException {
+    clickAndWait(metalakePage.createMetalakeBtn);
     metalakePage.setMetalakeNameField(name);
     metalakePage.setMetalakeCommentField("metalake comment");
     metalakePage.addMetalakePropsBtn.click();
     metalakePage.setMetalakeProps(0, "key1", "value1");
     metalakePage.addMetalakePropsBtn.click();
     metalakePage.setMetalakeProps(1, "key2", "value2");
-    metalakePage.submitHandleMetalakeBtn.click();
+    clickAndWait(metalakePage.submitHandleMetalakeBtn);
   }
 
   @Test
@@ -35,78 +38,79 @@ public void homePage() {
     Assertions.assertEquals("Gravitino", title);
   }
 
-  //  @Test
-  //  @Order(1)
-  public void testCreateMetalake() {
-    String name = "metalake_name";
-    createMetalakeAction(name);
-    Assertions.assertTrue(metalakePage.verifyCreateMetalake(name));
+  @Test
+  @Order(1)
+  public void testCreateMetalake() throws InterruptedException {
+    createMetalakeAction(metalakeName);
+    Assertions.assertTrue(metalakePage.verifyCreateMetalake(metalakeName));
   }
 
-  //  @Test
-  //  @Order(2)
-  public void testViewMetalakeDetails() {
-    String name = "metalake_name";
-    metalakePage.clickViewMetalakeBtn(name);
-    Assertions.assertTrue(metalakePage.verifyShowMetalakeDetails(name));
+  @Test
+  @Order(2)
+  public void testViewMetalakeDetails() throws InterruptedException {
+    metalakePage.clickViewMetalakeBtn(metalakeName);
+    Assertions.assertTrue(metalakePage.verifyShowMetalakeDetails(metalakeName));
   }
 
-  //  @Test
-  //  @Order(3)
+  @Test
+  @Order(3)
   public void testEditMetalake() {
-    metalakePage.clickEditMetalakeBtn("metalake_name");
-    metalakePage.setMetalakeNameField("metalake_name_edited");
+    metalakePage.clickEditMetalakeBtn(metalakeName);
+    metalakePage.setMetalakeNameField(editedMetalakeName);
     metalakePage.submitHandleMetalakeBtn.click();
-    Assertions.assertTrue(metalakePage.verifyEditedMetalake("metalake_name_edited"));
+    Assertions.assertTrue(metalakePage.verifyEditedMetalake(editedMetalakeName));
   }
 
-  //  @Test
-  //  @Order(4)
+  @Test
+  @Order(4)
   public void testDeleteMetalake() {
-    metalakePage.clickDeleteMetalakeBtn("metalake_name_edited");
+    metalakePage.clickDeleteMetalakeBtn(editedMetalakeName);
     metalakePage.confirmDeleteBtn.click();
     Assertions.assertTrue(metalakePage.verifyEmptyMetalake());
   }
 
-  //  @Test
-  //  @Order(5)
-  public void testCreateMultipleMetalakes() {
+  @Test
+  @Order(5)
+  public void testCreateMultipleMetalakes() throws InterruptedException {
     int twoPagesCount = 11;
 
     for (int i = 0; i < twoPagesCount; i++) {
+      try {
+        Thread.sleep(ACTION_SLEEP_MILLIS);
+      } catch (Exception e) {
+        LOG.error(e.getMessage(), e);
+      }
       String name = "metalake_" + (i + 1);
       createMetalakeAction(name);
     }
 
     Assertions.assertTrue(metalakePage.verifyChangePagination());
   }
 
-  //  @Test
-  //  @Order(6)
-  public void testQueryMetalake() {
+  @Test
+  @Order(6)
+  public void testQueryMetalake() throws InterruptedException {
     String name = "query";
     createMetalakeAction(name);
     Assertions.assertTrue(metalakePage.verifyQueryMetalake(name));
   }
 
-  //  @Test
-  //  @Order(7)
-  public void testCreateInvalidMetalake() {
+  @Test
+  @Order(7)
+  public void testCreateInvalidMetalake() throws InterruptedException {
     String name = "1!@#$";
     metalakePage.createMetalakeBtn.click();
     metalakePage.setMetalakeNameField(name);
     metalakePage.submitHandleMetalakeBtn.click();
     Assertions.assertTrue(metalakePage.checkIsErrorName());
   }
 
-  //  https://github.com/datastrato/gravitino/issues/2512
-  //  @Test
-  //  @Order(7)
-  //  public void testLinkToCatalogsPage() {
-  //    String name = "a_link";
-  //    createMetalakeAction(true, name);
-  //    metalakePage.clickMetalakeLink(name);
-  //    metalakePage.waitLinkElementDisplayed(name);
-  //    Assertions.assertTrue(metalakePage.verifyLinkToCatalogsPage(name));
-  //  }
+  @Test
+  @Order(8)
+  public void testLinkToCatalogsPage() throws InterruptedException {
+    String name = "a_test_link";
+    createMetalakeAction(name);
+    metalakePage.clickMetalakeLink(name);
+    Assertions.assertTrue(metalakePage.verifyLinkToCatalogsPage(name));
+  }
 }