From a96a90190c2d348ef87c32cc26ffdc72e9463b9c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Franti=C5=A1ek=20Ma=C5=A1a?= Date: Mon, 22 Apr 2024 20:30:27 +0200 Subject: [PATCH 1/3] Destroy all Firebase instances before each tests --- src/test/kotlin/AppTest.kt | 2 +- src/test/kotlin/FirebaseTest.kt | 9 +++++++++ src/test/kotlin/FirestoreTest.kt | 2 +- 3 files changed, 11 insertions(+), 2 deletions(-) create mode 100644 src/test/kotlin/FirebaseTest.kt diff --git a/src/test/kotlin/AppTest.kt b/src/test/kotlin/AppTest.kt index 72598e6..d017e38 100644 --- a/src/test/kotlin/AppTest.kt +++ b/src/test/kotlin/AppTest.kt @@ -5,7 +5,7 @@ import com.google.firebase.FirebasePlatform import com.google.firebase.initialize import org.junit.Test -class AppTest { +class AppTest : FirebaseTest() { @Test fun testInitialize() { FirebasePlatform.initializeFirebasePlatform(object : FirebasePlatform() { diff --git a/src/test/kotlin/FirebaseTest.kt b/src/test/kotlin/FirebaseTest.kt new file mode 100644 index 0000000..224b473 --- /dev/null +++ b/src/test/kotlin/FirebaseTest.kt @@ -0,0 +1,9 @@ +import com.google.firebase.FirebaseApp +import org.junit.Before + +abstract class FirebaseTest { + @Before + fun beforeEach() { + FirebaseApp.clearInstancesForTest() + } +} diff --git a/src/test/kotlin/FirestoreTest.kt b/src/test/kotlin/FirestoreTest.kt index 4f27899..50677f4 100644 --- a/src/test/kotlin/FirestoreTest.kt +++ b/src/test/kotlin/FirestoreTest.kt @@ -11,7 +11,7 @@ import org.junit.Before import org.junit.Test import java.io.File -class FirestoreTest { +class FirestoreTest : FirebaseTest() { @Before fun initialize() { FirebasePlatform.initializeFirebasePlatform(object : FirebasePlatform() { From d398f1e5d69cf8cf4060d7072cb2c869a873b703 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Franti=C5=A1ek=20Ma=C5=A1a?= Date: Mon, 22 Apr 2024 20:41:10 +0200 Subject: [PATCH 2/3] Fix test result reporting for PRs opened from forks --- .github/workflows/build-pr.yml | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/.github/workflows/build-pr.yml b/.github/workflows/build-pr.yml index c2f6bb1..ba3c825 100644 --- a/.github/workflows/build-pr.yml +++ b/.github/workflows/build-pr.yml @@ -5,6 +5,11 @@ jobs: gradle: name: Build and Test runs-on: ubuntu-latest + permissions: + checks: write + contents: read + issues: write + pull-requests: write steps: - uses: actions/checkout@v3 - uses: actions/setup-java@v3 From ce94b59163d5896e698b1c240c1c6e82c2db21ca Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Franti=C5=A1ek=20Ma=C5=A1a?= Date: Tue, 7 May 2024 19:14:14 +0200 Subject: [PATCH 3/3] Remove test report step See https://github.com/ScaCap/action-surefire-report/issues/31 --- .github/workflows/build-pr.yml | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/.github/workflows/build-pr.yml b/.github/workflows/build-pr.yml index ba3c825..daf4e10 100644 --- a/.github/workflows/build-pr.yml +++ b/.github/workflows/build-pr.yml @@ -5,11 +5,6 @@ jobs: gradle: name: Build and Test runs-on: ubuntu-latest - permissions: - checks: write - contents: read - issues: write - pull-requests: write steps: - uses: actions/checkout@v3 - uses: actions/setup-java@v3 @@ -20,9 +15,3 @@ jobs: uses: eskatos/gradle-command-action@v1 with: arguments: build - - name: Test Report - uses: scacap/action-surefire-report@v1 - if: always() - with: - report_paths: "build/test-results/test/TEST-*.xml" - github_token: ${{ secrets.GITHUB_TOKEN }}