Improve UI test coverage

pom-dependency-tree.txt

@@ -1,4 +1,4 @@
-ai.elimu:webapp:war:2.5.32-SNAPSHOT
+ai.elimu:webapp:war:2.5.33-SNAPSHOT
 +- com.github.elimu-ai:model:jar:model-2.0.83:compile
 |  \- com.google.code.gson:gson:jar:2.11.0:compile
 |     \- com.google.errorprone:error_prone_annotations:jar:2.27.0:compile

src/main/webapp/WEB-INF/jsp/content/letter-sound/list.jsp

@@ -81,6 +81,6 @@
     </div>
 
     <div class="fixed-action-btn" style="bottom: 2em; right: 2em;">
-        <a href="<spring:url value='/content/letter-sound/create' />" class="btn-floating btn-large tooltipped" data-position="left" data-delay="50" data-tooltip="<fmt:message key="add.letter.sound.correspondence" />"><i class="material-icons">add</i></a>
+        <a id="createButton" href="<spring:url value='/content/letter-sound/create' />" class="btn-floating btn-large tooltipped" data-position="left" data-delay="50" data-tooltip="<fmt:message key="add.letter.sound.correspondence" />"><i class="material-icons">add</i></a>
     </div>
 </content:section>

src/main/webapp/WEB-INF/jsp/content/letter/list.jsp

@@ -40,7 +40,7 @@
                             </td>
                             <td style="font-size: 2em;">
                                 <a name="${letter.id}"></a>
-                                <a href="<spring:url value='/content/letter/edit/${letter.id}' />">"<c:out value='${letter.text}' />"</a>
+                                <a class="editLink" href="<spring:url value='/content/letter/edit/${letter.id}' />">"<c:out value='${letter.text}' />"</a>
                             </td>
                             <td>
                                 <c:choose>
@@ -81,6 +81,6 @@
     </div>
 
     <div class="fixed-action-btn" style="bottom: 2em; right: 2em;">
-        <a href="<spring:url value='/content/letter/create' />" class="btn-floating btn-large tooltipped" data-position="left" data-delay="50" data-tooltip="<fmt:message key="add.letter" />"><i class="material-icons">add</i></a>
+        <a id="createButton" href="<spring:url value='/content/letter/create' />" class="btn-floating btn-large tooltipped" data-position="left" data-delay="50" data-tooltip="<fmt:message key="add.letter" />"><i class="material-icons">add</i></a>
     </div>
 </content:section>

src/main/webapp/WEB-INF/jsp/content/main.jsp

@@ -14,7 +14,7 @@
                     <span class="card-title"><i class="material-icons">text_fields</i> <fmt:message key="letters" /></span>
                 </div>
                 <div class="card-action">
-                    <a href="<spring:url value='/content/letter/list' />"><fmt:message key="view.list" /> (${letterCount})</a>
+                    <a id="letterListLink" href="<spring:url value='/content/letter/list' />"><fmt:message key="view.list" /> (${letterCount})</a>
                 </div>
             </div>
         </div>
@@ -25,7 +25,7 @@
                     <span class="card-title"><i class="material-icons">music_note</i> <fmt:message key="sounds" /></span>
                 </div>
                 <div class="card-action">
-                    <a href="<spring:url value='/content/sound/list' />"><fmt:message key="view.list" /> (${soundCount})</a>
+                    <a id="soundListLink" href="<spring:url value='/content/sound/list' />"><fmt:message key="view.list" /> (${soundCount})</a>
                 </div>
             </div>
         </div>
@@ -36,7 +36,7 @@
                     <span class="card-title"><i class="material-icons">emoji_symbols</i> <fmt:message key="letter.sounds" /></span>
                 </div>
                 <div class="card-action">
-                    <a href="<spring:url value='/content/letter-sound/list' />"><fmt:message key="view.list" /> (${letterSoundCount})</a>
+                    <a id="letterSoundListLink" href="<spring:url value='/content/letter-sound/list' />"><fmt:message key="view.list" /> (${letterSoundCount})</a>
                     <a href="<spring:url value='/content/letter-sound/peer-reviews' />"><fmt:message key="peer.review" /></a>
                 </div>
             </div>

src/main/webapp/WEB-INF/jsp/content/sound/list.jsp

@@ -42,7 +42,7 @@
                             </td>
                             <td style="font-size: 2em;">
                                 <a name="${sound.id}"></a>
-                                <a href="<spring:url value='/content/sound/edit/${sound.id}' />">/${sound.valueIpa}/</a>
+                                <a class="editLink" href="<spring:url value='/content/sound/edit/${sound.id}' />">/${sound.valueIpa}/</a>
                             </td>
                             <td>
                                 ${sound.valueSampa}
@@ -84,6 +84,6 @@
     </div>
 
     <div class="fixed-action-btn" style="bottom: 2em; right: 2em;">
-        <a href="<spring:url value='/content/sound/create' />" class="btn-floating btn-large tooltipped" data-position="left" data-delay="50" data-tooltip="<fmt:message key="add.sound" />"><i class="material-icons">add</i></a>
+        <a id="createButton" href="<spring:url value='/content/sound/create' />" class="btn-floating btn-large tooltipped" data-position="left" data-delay="50" data-tooltip="<fmt:message key="add.sound" />"><i class="material-icons">add</i></a>
     </div>
 </content:section>

src/test/java/selenium/content/MainContentPage.java

@@ -0,0 +1,31 @@
+package selenium.content;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebElement;
+
+public class MainContentPage {
+
+    private WebDriver driver;
+
+    public MainContentPage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("mainContentPage"));
+    }
+
+    public void pressLetterListLink() {
+        WebElement link = driver.findElement(By.id("letterListLink"));
+        link.click();
+    }
+
+    public void pressSoundListLink() {
+        WebElement link = driver.findElement(By.id("soundListLink"));
+        link.click();
+    }
+
+    public void pressLetterSoundListLink() {
+        WebElement link = driver.findElement(By.id("letterSoundListLink"));
+        link.click();
+    }
+}

src/test/java/selenium/content/letter/LetterCreatePage.java

@@ -0,0 +1,15 @@
+package selenium.content.letter;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+
+public class LetterCreatePage {
+
+    private WebDriver driver;
+
+    public LetterCreatePage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("letterCreatePage"));
+    }
+}

src/test/java/selenium/content/letter/LetterEditPage.java

@@ -0,0 +1,15 @@
+package selenium.content.letter;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+
+public class LetterEditPage {
+
+    private WebDriver driver;
+
+    public LetterEditPage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("letterEditPage"));
+    }
+}

src/test/java/selenium/content/letter/LetterListPage.java

@@ -0,0 +1,30 @@
+package selenium.content.letter;
+
+import java.util.List;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebElement;
+
+public class LetterListPage {
+
+    private WebDriver driver;
+
+    public LetterListPage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("letterListPage"));
+    }
+
+    public void pressRandomLetter() {
+        List<WebElement> links = driver.findElements(By.className("editLink"));
+        int randomIndex = (int)(Math.random() * links.size());
+        WebElement randomLink = links.get(randomIndex);
+        randomLink.click();
+    }
+
+    public void pressCreateButton() {
+        WebElement button = driver.findElement(By.id("createButton"));
+        button.click();
+    }
+}

src/test/java/selenium/content/letter/LetterTest.java

@@ -0,0 +1,70 @@
+package selenium.content.letter;
+
+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 lombok.extern.slf4j.Slf4j;
+import selenium.content.MainContentPage;
+import selenium.util.DomainHelper;
+
+@Slf4j
+public class LetterTest {
+
+    private WebDriver driver;
+
+    @BeforeEach
+    public void setUp() { 
+        log.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");
+        log.info("headlessSystemProperty: \"" + headlessSystemProperty + "\"");
+        if ("true".equals(headlessSystemProperty)) {
+            chromeOptions.addArguments("headless");
+        }
+
+        driver = new ChromeDriver(chromeOptions);
+
+        driver.get(DomainHelper.getBaseUrl() + "/content");
+    }
+
+    @AfterEach
+    public void tearDown() {
+        log.info("tearDown");
+
+        driver.quit();
+    }
+
+    @Test
+    public void testRandomLetterEditPage() {
+        log.info("testRandomLetterEditPage");
+
+        MainContentPage mainContentPage = new MainContentPage(driver);
+        mainContentPage.pressLetterListLink();
+
+        LetterListPage letterListPage = new LetterListPage(driver);
+        letterListPage.pressRandomLetter();
+
+        LetterEditPage letterEditPage = new LetterEditPage(driver);
+    }
+
+    @Test
+    public void testLetterCreatPage() {
+        log.info("testLetterCreatePage");
+
+        MainContentPage mainContentPage = new MainContentPage(driver);
+        mainContentPage.pressLetterListLink();
+
+        LetterListPage letterListPage = new LetterListPage(driver);
+        letterListPage.pressCreateButton();
+
+        LetterCreatePage letterCreatePage = new LetterCreatePage(driver);
+    }
+}

src/test/java/selenium/content/letter_sound/LetterSoundCreatePage.java

@@ -0,0 +1,15 @@
+package selenium.content.letter_sound;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+
+public class LetterSoundCreatePage {
+
+    private WebDriver driver;
+
+    public LetterSoundCreatePage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("letterSoundCreatePage"));
+    }
+}

src/test/java/selenium/content/letter_sound/LetterSoundEditPage.java

@@ -0,0 +1,15 @@
+package selenium.content.letter_sound;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+
+public class LetterSoundEditPage {
+
+    private WebDriver driver;
+
+    public LetterSoundEditPage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("letterSoundEditPage"));
+    }
+}

src/test/java/selenium/content/letter_sound/LetterSoundListPage.java

@@ -0,0 +1,30 @@
+package selenium.content.letter_sound;
+
+import java.util.List;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+import org.openqa.selenium.WebElement;
+
+public class LetterSoundListPage {
+
+    private WebDriver driver;
+
+    public LetterSoundListPage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("letterSoundListPage"));
+    }
+
+    public void pressRandomLetterSound() {
+        List<WebElement> links = driver.findElements(By.className("editLink"));
+        int randomIndex = (int)(Math.random() * links.size());
+        WebElement randomLink = links.get(randomIndex);
+        randomLink.click();
+    }
+
+    public void pressCreateButton() {
+        WebElement button = driver.findElement(By.id("createButton"));
+        button.click();
+    }
+}

src/test/java/selenium/content/letter_sound/LetterSoundTest.java

@@ -0,0 +1,70 @@
+package selenium.content.letter_sound;
+
+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 lombok.extern.slf4j.Slf4j;
+import selenium.content.MainContentPage;
+import selenium.util.DomainHelper;
+
+@Slf4j
+public class LetterSoundTest {
+
+    private WebDriver driver;
+
+    @BeforeEach
+    public void setUp() { 
+        log.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");
+        log.info("headlessSystemProperty: \"" + headlessSystemProperty + "\"");
+        if ("true".equals(headlessSystemProperty)) {
+            chromeOptions.addArguments("headless");
+        }
+
+        driver = new ChromeDriver(chromeOptions);
+
+        driver.get(DomainHelper.getBaseUrl() + "/content");
+    }
+
+    @AfterEach
+    public void tearDown() {
+        log.info("tearDown");
+
+        driver.quit();
+    }
+
+    @Test
+    public void testRandomLetterSoundEditPage() {
+        log.info("testRandomLetterSoundEditPage");
+
+        MainContentPage mainContentPage = new MainContentPage(driver);
+        mainContentPage.pressLetterSoundListLink();
+
+        LetterSoundListPage letterSoundListPage = new LetterSoundListPage(driver);
+        letterSoundListPage.pressRandomLetterSound();
+
+        LetterSoundEditPage letterSoundEditPage = new LetterSoundEditPage(driver);
+    }
+
+    @Test
+    public void testLetterSoundCreatPage() {
+        log.info("testLetterSoundCreatePage");
+
+        MainContentPage mainContentPage = new MainContentPage(driver);
+        mainContentPage.pressLetterSoundListLink();
+
+        LetterSoundListPage letterSoundListPage = new LetterSoundListPage(driver);
+        letterSoundListPage.pressCreateButton();
+
+        LetterSoundCreatePage letterSoundCreatePage = new LetterSoundCreatePage(driver);
+    }
+}

src/test/java/selenium/content/sound/SoundCreatePage.java

@@ -0,0 +1,15 @@
+package selenium.content.sound;
+
+import org.openqa.selenium.By;
+import org.openqa.selenium.WebDriver;
+
+public class SoundCreatePage {
+
+    private WebDriver driver;
+
+    public SoundCreatePage(WebDriver driver) {
+        this.driver = driver;
+
+        driver.findElement(By.id("soundCreatePage"));
+    }
+}