From f7d836bba99a41ee3fad00541eef5df6570852d1 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 11:56:08 +0200 Subject: [PATCH 01/12] Remove post in pr-java-ci.yml --- .github/workflows/pr-java-ci.yml | 2 -- 1 file changed, 2 deletions(-) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index 3f68172bf32..3b8a308bbda 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -39,7 +39,5 @@ jobs: report_paths: '**/target/surefire-reports/TEST-*.xml' check_name: 'JUnit Test Report' include_passed: true - comment: true fail_on_failure: true - updateComment: true token: ${{ secrets.GITHUB_TOKEN }} From f743398165192a96d71054f6ede97a04e8ad8765 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 12:18:33 +0200 Subject: [PATCH 02/12] Test commit for endpoint --- .../org/prebid/server/bidder/generic/GenericBidderTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java index 3ba06b5d4a9..15c2d846dfa 100644 --- a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java +++ b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java @@ -56,7 +56,7 @@ public void makeHttpRequestsShouldCreateExpectedUrl() { assertThat(result.getErrors()).isEmpty(); assertThat(result.getValue()).hasSize(1) .extracting(HttpRequest::getUri) - .containsExactly("https://randomurl.com"); + .containsExactly(ENDPOINT_URL); } @Test From 0a4fe61a0223960fbcbd131388c43c5aacc8d0d1 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 13:21:36 +0200 Subject: [PATCH 03/12] Add permissions: checks: write --- .github/workflows/pr-java-ci.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index 3b8a308bbda..cd5d6e1e5f8 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -1,5 +1,8 @@ name: Java CI +permissions: + checks: write + on: pull_request: paths-ignore: From 5c0c6f097c2dc4cdfe84fa2d629622dbe55552c6 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 13:32:37 +0200 Subject: [PATCH 04/12] Remove permissions: checks: write and add with: annotate_only: 'true' --- .github/workflows/pr-java-ci.yml | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index cd5d6e1e5f8..9e52837bc19 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -1,8 +1,5 @@ name: Java CI -permissions: - checks: write - on: pull_request: paths-ignore: @@ -39,8 +36,4 @@ jobs: - name: Publish JUnit Report uses: mikepenz/action-junit-report@v5 with: - report_paths: '**/target/surefire-reports/TEST-*.xml' - check_name: 'JUnit Test Report' - include_passed: true - fail_on_failure: true - token: ${{ secrets.GITHUB_TOKEN }} + annotate_only: 'true' From 414dc6516bb45bbcb8e9a70c0acfd2cb80895a43 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 15:00:08 +0200 Subject: [PATCH 05/12] Include passes --- .github/workflows/pr-java-ci.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index 9e52837bc19..47a57bad2b7 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -37,3 +37,4 @@ jobs: uses: mikepenz/action-junit-report@v5 with: annotate_only: 'true' + include_passed: 'true' From 5ed2797c1fb6529882f1187a37cee70e7074cd03 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 15:26:43 +0200 Subject: [PATCH 06/12] Make tests failed --- .../org/prebid/server/bidder/generic/GenericBidderTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java index 15c2d846dfa..6c06b71fd97 100644 --- a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java +++ b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java @@ -56,7 +56,7 @@ public void makeHttpRequestsShouldCreateExpectedUrl() { assertThat(result.getErrors()).isEmpty(); assertThat(result.getValue()).hasSize(1) .extracting(HttpRequest::getUri) - .containsExactly(ENDPOINT_URL); + .containsExactly("failedTest"); } @Test From e3ec4bba0c87199d208a7027d7e0086aa5e25b74 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 15:45:10 +0200 Subject: [PATCH 07/12] Add: check_name: 'JUnit Test Report' report_paths:'**/target/surefire-reports/TEST-*.xml' --- .github/workflows/pr-java-ci.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index 47a57bad2b7..97fa5ab7601 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -36,5 +36,7 @@ jobs: - name: Publish JUnit Report uses: mikepenz/action-junit-report@v5 with: + check_name: 'JUnit Test Report' + report_paths: '**/target/surefire-reports/TEST-*.xml' annotate_only: 'true' include_passed: 'true' From fb842f06718c34e3649ae71ca933d0b24bc4e1bc Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 16:03:26 +0200 Subject: [PATCH 08/12] Add: fail_on_failure: true --- .github/workflows/pr-java-ci.yml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index 97fa5ab7601..279eee722d8 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -38,5 +38,6 @@ jobs: with: check_name: 'JUnit Test Report' report_paths: '**/target/surefire-reports/TEST-*.xml' - annotate_only: 'true' - include_passed: 'true' + fail_on_failure: true + annotate_only: true' + include_passed: true From 11aefb1922abd17cbef2b64274e46520d90bbaa5 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 16:20:17 +0200 Subject: [PATCH 09/12] Add: if: always() --- .github/workflows/pr-java-ci.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index 279eee722d8..732407ae330 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -35,9 +35,9 @@ jobs: - name: Publish JUnit Report uses: mikepenz/action-junit-report@v5 + if: always() with: + report_paths: '**/build/test-results/test/*.xml' check_name: 'JUnit Test Report' - report_paths: '**/target/surefire-reports/TEST-*.xml' - fail_on_failure: true - annotate_only: true' - include_passed: true + fail_on_failure: false + annotate_only: true From fbd5cbd7c26f42cdf96ec90fa3339cabde86d63a Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 16:28:58 +0200 Subject: [PATCH 10/12] Change pr-java-ci config: Add: report_paths: '**/target/surefire-reports/TEST-*.xml' Replace fail_on_failure to true --- .github/workflows/pr-java-ci.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/pr-java-ci.yml b/.github/workflows/pr-java-ci.yml index 732407ae330..1a198dac001 100644 --- a/.github/workflows/pr-java-ci.yml +++ b/.github/workflows/pr-java-ci.yml @@ -37,7 +37,7 @@ jobs: uses: mikepenz/action-junit-report@v5 if: always() with: - report_paths: '**/build/test-results/test/*.xml' check_name: 'JUnit Test Report' - fail_on_failure: false + report_paths: '**/target/surefire-reports/TEST-*.xml' + fail_on_failure: true annotate_only: true From 8008cc25cc438ab847aa33d483711c128e4b219c Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 16:56:14 +0200 Subject: [PATCH 11/12] Revert "Make tests failed" This reverts commit 5ed2797c1fb6529882f1187a37cee70e7074cd03. --- .../org/prebid/server/bidder/generic/GenericBidderTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java index 6c06b71fd97..15c2d846dfa 100644 --- a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java +++ b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java @@ -56,7 +56,7 @@ public void makeHttpRequestsShouldCreateExpectedUrl() { assertThat(result.getErrors()).isEmpty(); assertThat(result.getValue()).hasSize(1) .extracting(HttpRequest::getUri) - .containsExactly("failedTest"); + .containsExactly(ENDPOINT_URL); } @Test From 60d48d9ac8ba550531885fa8baa16511cdfe6422 Mon Sep 17 00:00:00 2001 From: markiian Date: Wed, 3 Dec 2025 16:57:07 +0200 Subject: [PATCH 12/12] Rever 'failed test' --- .../org/prebid/server/bidder/generic/GenericBidderTest.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java index 15c2d846dfa..3ba06b5d4a9 100644 --- a/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java +++ b/src/test/java/org/prebid/server/bidder/generic/GenericBidderTest.java @@ -56,7 +56,7 @@ public void makeHttpRequestsShouldCreateExpectedUrl() { assertThat(result.getErrors()).isEmpty(); assertThat(result.getValue()).hasSize(1) .extracting(HttpRequest::getUri) - .containsExactly(ENDPOINT_URL); + .containsExactly("https://randomurl.com"); } @Test