From eb10c8f62a44c53e0cbd73213401d428a8d29d43 Mon Sep 17 00:00:00 2001
From: jo-elimu <1451036+jo-elimu@users.noreply.github.com>
Date: Sun, 29 Sep 2024 19:22:36 +0700
Subject: [PATCH] test: contributor page

#1896
---
 .../WEB-INF/jsp/contributor/contributor.jsp   |  2 +-
 .../webapp/WEB-INF/jsp/contributor/list.jsp   |  2 +-
 .../contributor/ContributorListPage.java      | 16 ++++++
 .../contributor/ContributorListPageTest.java  |  3 +
 .../selenium/contributor/ContributorPage.java | 18 ++++++
 .../contributor/ContributorPageTest.java      | 55 +++++++++++++++++++
 6 files changed, 94 insertions(+), 2 deletions(-)
 create mode 100644 src/test/java/selenium/contributor/ContributorPage.java
 create mode 100644 src/test/java/selenium/contributor/ContributorPageTest.java

diff --git a/src/main/webapp/WEB-INF/jsp/contributor/contributor.jsp b/src/main/webapp/WEB-INF/jsp/contributor/contributor.jsp
index 79b95964e..dee2cc0d7 100644
--- a/src/main/webapp/WEB-INF/jsp/contributor/contributor.jsp
+++ b/src/main/webapp/WEB-INF/jsp/contributor/contributor.jsp
@@ -9,7 +9,7 @@
     </c:choose>
 </content:title>
 
-<content:section cssId="contributorStoryBooksPage">
+<content:section cssId="contributorPage">
     <%@ include file="/WEB-INF/jsp/contributor/contributor-summarized.jsp" %>
     
     <ul class="tabs tabs-transparent deep-purple lighten-1 z-depth-1" style="border-radius: 8px;">
diff --git a/src/main/webapp/WEB-INF/jsp/contributor/list.jsp b/src/main/webapp/WEB-INF/jsp/contributor/list.jsp
index 1814dba30..6fb048c2f 100644
--- a/src/main/webapp/WEB-INF/jsp/contributor/list.jsp
+++ b/src/main/webapp/WEB-INF/jsp/contributor/list.jsp
@@ -22,7 +22,7 @@
         <c:forEach var="contributor" items="${contributors}">
             <div class="col s6 m4 l3">
                 <div class="card contributor">
-                    <a href="<spring:url value='/contributor/${contributor.id}' />">
+                    <a class="contributorCardImageLink" href="<spring:url value='/contributor/${contributor.id}' />">
                         <div class="card-image">
                             <c:choose>
                                 <c:when test="${not empty contributor.imageUrl}">
diff --git a/src/test/java/selenium/contributor/ContributorListPage.java b/src/test/java/selenium/contributor/ContributorListPage.java
index 5a7703895..ea6086053 100644
--- a/src/test/java/selenium/contributor/ContributorListPage.java
+++ b/src/test/java/selenium/contributor/ContributorListPage.java
@@ -1,12 +1,19 @@
 package selenium.contributor;
 
+import java.util.List;
+
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 import org.openqa.selenium.By;
 import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebElement;
 
 import selenium.util.ErrorHelper;
 
 public class ContributorListPage {
 
+    private Logger logger = LogManager.getLogger();
+
     private WebDriver driver;
 
     public ContributorListPage(WebDriver driver) {
@@ -16,4 +23,13 @@ public ContributorListPage(WebDriver driver) {
         
         ErrorHelper.verifyNoScriptOrMarkupError(driver);
     }
+
+    public void pressRandomContributor() {
+        List<WebElement> contributorLinks = driver.findElements(By.className("contributorCardImageLink"));
+        logger.info("contributorLinks.size(): " + contributorLinks.size());
+        int randomIndex = (int) (Math.random() * contributorLinks.size());
+        logger.info("randomIndex: " + randomIndex);
+        WebElement contributorLink = contributorLinks.get(randomIndex);
+        contributorLink.click();
+    }
 }
diff --git a/src/test/java/selenium/contributor/ContributorListPageTest.java b/src/test/java/selenium/contributor/ContributorListPageTest.java
index 55539d604..c6f75c534 100644
--- a/src/test/java/selenium/contributor/ContributorListPageTest.java
+++ b/src/test/java/selenium/contributor/ContributorListPageTest.java
@@ -48,5 +48,8 @@ public void testContributorListPage() {
         logger.info("testContributorListPage");
         
         ContributorListPage contributorListPage = new ContributorListPage(driver);
+        contributorListPage.pressRandomContributor();
+
+        ContributorPage contributorPage = new ContributorPage(driver);
     }
 }
diff --git a/src/test/java/selenium/contributor/ContributorPage.java b/src/test/java/selenium/contributor/ContributorPage.java
new file mode 100644
index 000000000..53e1d1522
--- /dev/null
+++ b/src/test/java/selenium/contributor/ContributorPage.java
@@ -0,0 +1,18 @@
+package selenium.contributor;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+import selenium.util.ErrorHelper;
+
+public class ContributorPage {
+
+    private WebDriver driver;
+
+    public ContributorPage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("contributorPage"));
+        
+        ErrorHelper.verifyNoScriptOrMarkupError(driver);
+    }
+}
diff --git a/src/test/java/selenium/contributor/ContributorPageTest.java b/src/test/java/selenium/contributor/ContributorPageTest.java
new file mode 100644
index 000000000..526209811
--- /dev/null
+++ b/src/test/java/selenium/contributor/ContributorPageTest.java
@@ -0,0 +1,55 @@
+package selenium.contributor;
+
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.chrome.ChromeDriver;
+import org.openqa.selenium.chrome.ChromeOptions;
+
+import selenium.util.DomainHelper;
+
+public class ContributorPageTest {
+    
+    private final Logger logger = LogManager.getLogger();
+    
+    private WebDriver driver;
+
+    @BeforeEach
+    public void setUp() { 
+        logger.info("setUp");
+
+        ChromeOptions chromeOptions = new ChromeOptions();
+
+        // Read "headless" property set on the command line: 
+        //    mvn clean verify -P regression-test-ui -D headless=true
+        String headlessSystemProperty = System.getProperty("headless");
+        logger.info("headlessSystemProperty: \"" + headlessSystemProperty + "\"");
+        if ("true".equals(headlessSystemProperty)) {
+            chromeOptions.addArguments("headless");
+        }
+        
+        driver = new ChromeDriver(chromeOptions);
+
+        driver.get(DomainHelper.getBaseUrl() + "/contributor/list");
+    }
+
+    @AfterEach
+    public void tearDown() {
+        logger.info("tearDown");
+
+        driver.quit();
+    }
+
+    @Test
+    public void testContributorPage() {
+        logger.info("testContributorPage");
+        
+        ContributorListPage contributorListPage = new ContributorListPage(driver);
+        contributorListPage.pressRandomContributor();
+
+        ContributorPage contributorPage = new ContributorPage(driver);
+    }
+}