Feat/alert/UI: AlertScreen UI

.github/workflows/ci.yml

@@ -5,6 +5,8 @@ on:
   push:
     branches:
       - main
+      - 'setup/**'
+      - 'feat/**'
 
   pull_request:
     types:
@@ -40,6 +42,16 @@ jobs:
           distribution: "temurin"
           java-version: "17"
 
+      # Hello Hello, Alonso here ^_~
+      # For whatever reason Android SDK is weird and I can't get the emulator to work
+      # vvv this new plugin should install v12.0 that was published Sep 2024 to be updated
+      - name: Setup Android SDK
+        uses: Swisyn/setup-android-sdk@v1
+        with:
+          cmdline-tools-version: 11076708
+      ##TODO: Check if I need to install adb and avdmanager manually with cli cmds
+
+
       # Caching is a very useful part of a CI, as a workflow is executed in a clean environment every time,
       # this means that one would need to re-download and re-process gradle files for every run. Which is very time consuming.
       #
@@ -66,7 +78,7 @@ jobs:
           arch: x86_64
           force-avd-creation: false
           emulator-options: -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
-          disable-animations: false
+          disable-animations: true
           script: echo "Generated AVD snapshot for caching."
 
       - name: Grant execute permission for gradlew
@@ -97,8 +109,9 @@ jobs:
           api-level: 34
           target: google_apis
           arch: x86_64
+          avd-name: github
           force-avd-creation: false
-          emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
+          emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none -skin 1080x1920
           disable-animations: true
           script: ./gradlew connectedCheck --parallel --build-cache
 

app/build.gradle.kts

@@ -5,7 +5,7 @@ plugins {
   alias(libs.plugins.androidApplication)
   alias(libs.plugins.jetbrainsKotlinAndroid)
   alias(libs.plugins.ktfmt)
-  //alias(libs.plugins.sonar)
+  // alias(libs.plugins.sonar)
   alias(libs.plugins.compose.compiler)
   id("jacoco")
 
@@ -94,26 +94,28 @@ android {
   }
 }
 
-
 sonar {
   properties {
-    property("sonar.projectKey", "PeriodPals_periodpals")
+    property("sonar.projectKey", "periodpals_periodpals")
     property("sonar.organization", "periodpals")
     property("sonar.host.url", "https://sonarcloud.io")
     // Comma-separated paths to the various directories containing the *.xml JUnit report files.
     // Each path may be absolute or relative to the project base directory.
     property(
-        "sonar.junit.reportPaths",
-        "${project.layout.buildDirectory.get()}/test-results/testDebugunitTest/")
+      "sonar.junit.reportPaths",
+      "${project.layout.buildDirectory.get()}/test-results/testDebugunitTest/",
+    )
     // Paths to xml files with Android Lint issues. If the main flavor is changed, this file will
     // have to be changed too.
     property(
-        "sonar.androidLint.reportPaths",
-        "${project.layout.buildDirectory.get()}/reports/lint-results-debug.xml")
+      "sonar.androidLint.reportPaths",
+      "${project.layout.buildDirectory.get()}/reports/lint-results-debug.xml",
+    )
     // Paths to JaCoCo XML coverage report files.
     property(
-        "sonar.coverage.jacoco.xmlReportPaths",
-        "${project.layout.buildDirectory.get()}/reports/jacoco/jacocoTestReport/jacocoTestReport.xml")
+      "sonar.coverage.jacoco.xmlReportPaths",
+      "${project.layout.buildDirectory.get()}/reports/jacoco/jacocoTestReport/jacocoTestReport.xml",
+    )
   }
 }
 
@@ -134,6 +136,17 @@ dependencies {
   // implementation(libs.androidx.fragment.ktx)
   // implementation(libs.kotlinx.serialization.json)
 
+  implementation(libs.compose)
+  implementation(libs.mockk.v1120)
+  implementation(libs.androidx.ui.test.junit4.v105)
+  implementation(libs.androidx.ui.test.manifest.v105)
+
+  configurations.all {
+    resolutionStrategy {
+      force("androidx.test.ext:junit:1.1.5")
+      force("androidx.test.espresso:espresso-core:3.5.0")
+    }
+  }
   // supabase setup
   implementation(platform("io.github.jan-tennert.supabase:bom:3.0.0"))
   implementation(libs.github.postgrest.kt)
@@ -148,6 +161,7 @@ dependencies {
   implementation(libs.material)
   implementation(libs.androidx.lifecycle.runtime.ktx)
   implementation(platform(libs.compose.bom))
+  implementation(libs.androidx.navigation.compose.v282)
 
   testImplementation(libs.junit)
 
@@ -180,6 +194,12 @@ dependencies {
 
   // ----------       Robolectric     ------------
   testImplementation(libs.robolectric)
+
+  // Material Icons
+  implementation(libs.androidx.material.icons.extended)
+  // Mockito for unit testing
+  testImplementation(libs.mockito.kotlin)
+  testImplementation(libs.mockito.core.v540)
 }
 
 tasks.withType<Test> {
@@ -199,26 +219,27 @@ tasks.register("jacocoTestReport", JacocoReport::class) {
   }
 
   val fileFilter =
-      listOf(
-          "**/R.class",
-          "**/R$*.class",
-          "**/BuildConfig.*",
-          "**/Manifest*.*",
-          "**/*Test*.*",
-          "android/**/*.*",
-      )
+    listOf(
+      "**/R.class",
+      "**/R$*.class",
+      "**/BuildConfig.*",
+      "**/Manifest*.*",
+      "**/*Test*.*",
+      "android/**/*.*",
+    )
 
   val debugTree =
-      fileTree("${project.layout.buildDirectory.get()}/tmp/kotlin-classes/debug") {
-        exclude(fileFilter)
-      }
+    fileTree("${project.layout.buildDirectory.get()}/tmp/kotlin-classes/debug") {
+      exclude(fileFilter)
+    }
 
   val mainSrc = "${project.layout.projectDirectory}/src/main/java"
   sourceDirectories.setFrom(files(mainSrc))
   classDirectories.setFrom(files(debugTree))
   executionData.setFrom(
-      fileTree(project.layout.buildDirectory.get()) {
-        include("outputs/unit_test_code_coverage/debugUnitTest/testDebugUnitTest.exec")
-        include("outputs/code_coverage/debugAndroidTest/connected/*/coverage.ec")
-      })
+    fileTree(project.layout.buildDirectory.get()) {
+      include("outputs/unit_test_code_coverage/debugUnitTest/testDebugUnitTest.exec")
+      include("outputs/code_coverage/debugAndroidTest/connected/*/coverage.ec")
+    }
+  )
 }

app/src/androidTest/java/com/android/periodpals/ui/alert/AlertScreenTest.kt

@@ -0,0 +1,48 @@
+package com.android.periodpals.ui.alert
+
+import androidx.compose.material3.MaterialTheme
+import androidx.compose.ui.test.assertIsDisplayed
+import androidx.compose.ui.test.assertTextEquals
+import androidx.compose.ui.test.junit4.createComposeRule
+import androidx.compose.ui.test.onNodeWithTag
+import androidx.compose.ui.test.performClick
+import androidx.compose.ui.test.performTextInput
+import org.junit.Rule
+import org.junit.Test
+
+class AlertScreenTest {
+
+  @get:Rule val composeTestRule = createComposeRule()
+
+  @Test
+  fun displayAllComponents() {
+    composeTestRule.setContent { MaterialTheme { AlertScreen() } }
+
+    composeTestRule.onNodeWithTag("alertInstruction").assertIsDisplayed()
+    composeTestRule.onNodeWithTag("alertProduct").assertIsDisplayed()
+    composeTestRule.onNodeWithTag("alertUrgency").assertIsDisplayed()
+    composeTestRule.onNodeWithTag("alertLocation").assertIsDisplayed()
+    composeTestRule.onNodeWithTag("alertMessage").assertIsDisplayed()
+    composeTestRule
+        .onNodeWithTag("alertSubmit")
+        .assertIsDisplayed()
+        .assertTextEquals("Ask for Help")
+  }
+
+  @Test
+  fun interactWithComponents() {
+    composeTestRule.setContent { MaterialTheme { AlertScreen() } }
+
+    composeTestRule.onNodeWithTag("alertProduct").performClick()
+    composeTestRule.onNodeWithTag("Pads").performClick()
+    //        composeTestRule.onNodeWithTag("alertProduct").assertTextEquals("Pads")
+    composeTestRule.onNodeWithTag("alertUrgency").performClick()
+    composeTestRule.onNodeWithTag("!! Medium").performClick()
+    //        composeTestRule.onNodeWithTag("alertUrgency").assertTextEquals("!! Medium")
+
+    composeTestRule.onNodeWithTag("alertLocation").performTextInput("Rolex")
+    composeTestRule.onNodeWithTag("alertMessage").performTextInput("I need help finding a tampon")
+
+    composeTestRule.onNodeWithTag("alertSubmit").performClick()
+  }
+}

app/src/androidTest/java/com/android/periodpals/ui/navigation/BottomNavigationMenuTest.kt

@@ -0,0 +1,96 @@
+package com.android.periodpals.ui.navigation
+
+import android.annotation.SuppressLint
+import androidx.compose.runtime.getValue
+import androidx.compose.runtime.mutableStateOf
+import androidx.compose.runtime.setValue
+import androidx.compose.ui.test.assertIsDisplayed
+import androidx.compose.ui.test.assertIsNotSelected
+import androidx.compose.ui.test.assertIsSelected
+import androidx.compose.ui.test.junit4.createComposeRule
+import androidx.compose.ui.test.onNodeWithTag
+import androidx.compose.ui.test.performClick
+import org.junit.Rule
+import org.junit.Test
+
+class BottomNavigationMenuTest {
+
+  @get:Rule val composeTestRule = createComposeRule()
+
+  @Test
+  fun bottomNavigationMenu_displaysAllTabs() {
+    composeTestRule.setContent {
+      BottomNavigationMenu(
+          onTabSelect = {}, tabList = LIST_TOP_LEVEL_DESTINATION, selectedItem = "Map")
+    }
+
+    composeTestRule.onNodeWithTag("bottomNavigationMenu").assertIsDisplayed()
+    LIST_TOP_LEVEL_DESTINATION.forEach { tab ->
+      composeTestRule.onNodeWithTag(tab.textId).assertIsDisplayed()
+    }
+  }
+
+  @SuppressLint("UnrememberedMutableState")
+  @Test
+  fun bottomNavigationMenu_clickOnTab_changesSelection() {
+    var selectedTab by mutableStateOf(Route.MAP) // Initially selected tab is "MAP"
+
+    // Set the composable content with an initial selected tab
+    composeTestRule.setContent {
+      BottomNavigationMenu(
+          onTabSelect = { selectedTab = it.route },
+          tabList = LIST_TOP_LEVEL_DESTINATION,
+          selectedItem = selectedTab)
+    }
+
+    // Initially, verify that "MAP" is selected
+    composeTestRule.onNodeWithTag("Map").assertIsSelected()
+
+    // Perform a click on the "Alert" tab
+    composeTestRule.onNodeWithTag("Alert").performClick()
+
+    // Now check that the "Alert" tab is selected
+    composeTestRule.onNodeWithTag("Alert").assertIsSelected()
+
+    // Optionally, check that the previously selected "Map" tab is no longer selected
+    composeTestRule.onNodeWithTag("Map").assertIsNotSelected()
+  }
+
+  @Test
+  fun bottomNavigationMenu_iconAndLabelAreDisplayedCorrectly() {
+    composeTestRule.setContent {
+      BottomNavigationMenu(
+          onTabSelect = {}, tabList = LIST_TOP_LEVEL_DESTINATION, selectedItem = "Profile")
+    }
+
+    LIST_TOP_LEVEL_DESTINATION.forEach { tab ->
+      composeTestRule.onNodeWithTag(tab.textId).assertIsDisplayed()
+      composeTestRule.onNodeWithTag(tab.textId).assertIsDisplayed()
+    }
+  }
+
+  @Test
+  fun bottomNavigationMenu_initialSelectionIsCorrect() {
+    composeTestRule.setContent {
+      BottomNavigationMenu(
+          onTabSelect = {}, tabList = LIST_TOP_LEVEL_DESTINATION, selectedItem = "Timer")
+    }
+
+    composeTestRule.onNodeWithTag("Timer").assertIsSelected()
+  }
+
+  @Test
+  fun bottomNavigationMenu_selectingSameTabDoesNotCrash() {
+    var selectedTab = Route.ALERT_LIST
+
+    composeTestRule.setContent {
+      BottomNavigationMenu(
+          onTabSelect = { selectedTab = it.route },
+          tabList = LIST_TOP_LEVEL_DESTINATION,
+          selectedItem = selectedTab)
+    }
+
+    composeTestRule.onNodeWithTag("Alert List").performClick()
+    composeTestRule.onNodeWithTag("Alert List").assertIsSelected()
+  }
+}