Run tests

fyreplace · Jul 31, 2024 · 268c2fb · 268c2fb
1 parent f380759
commit 268c2fb
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 9 deletions.
diff --git a/.github/workflows/publishing.yml b/.github/workflows/publishing.yml
@@ -77,10 +77,31 @@ jobs:
           name: Fyreplace.libre.aab
           path: app/build/outputs/bundle/libreRelease/app-libre-release.aab
           if-no-files-found: error
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+          submodules: true
+
+      - name: Setup Java
+        uses: actions/setup-java@v4
+        with:
+          distribution: 'temurin'
+          java-version: '17'
+          cache: 'gradle'
+
+      - name: Run tests
+        run: ./gradlew test
 
   publish:
     name: Publish
-    needs: build
+    needs:
+      - build
+      - test
     runs-on: ubuntu-latest
     environment: google-play-store
     steps:

diff --git a/app/src/test/kotlin/app/fyreplace/fyreplace/test/navigation/DestinationTests.kt b/app/src/test/kotlin/app/fyreplace/fyreplace/test/navigation/DestinationTests.kt
@@ -2,12 +2,11 @@ package app.fyreplace.fyreplace.test.navigation
 
 import app.fyreplace.fyreplace.ui.views.navigation.Destination
 import org.junit.Assert.assertNotEquals
-import org.junit.Assert.assertTrue
 import org.junit.Test
 
 class DestinationTests {
     @Test
-    fun testDestinationsAreUnique() {
+    fun `Destinations are unique`() {
         for (first in Destination.entries) {
             for (second in Destination.entries.filterNot { it == first }) {
                 assertNotEquals(first.route, second.route)
@@ -20,12 +19,7 @@ class DestinationTests {
     }
 
     @Test
-    fun testEssentialsAreFewer() {
-        assertTrue(Destination.Set.topLevel(flatten = false).size < Destination.Set.topLevel(flatten = true).size)
-    }
-
-    @Test
-    fun testDestinationsDoNotLoop() {
+    fun `Destinations do not loop`() {
         for (destination in Destination.entries) {
             assertNotEquals(destination, destination.parent)
             assertNotEquals(destination, destination.parent?.parent)