From f0015465c3c1ffa7e10b53022808ebbef91b13de Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 00:53:50 +0500 Subject: [PATCH 01/12] ci(test-report): add test report genetating to PR comments on every tests run for more readable and clear testing results --- .github/workflows/lint-and-test.yml | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index d813008..a3f485a 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -107,9 +107,16 @@ jobs: poetry run pytest \ --cov=c2pie \ -m "not e2e" \ - --maxfail=1 \ + --junit-xml=test-report.xml \ -v + - name: Publish test report + uses: dorny/test-reporter@v1 + if: always() + with: + path: test-report.xml + reporter: python-xunit + - name: Upload coverage artifact if: (matrix.python-version == '3.12') && (matrix.os == 'ubuntu-latest') uses: actions/upload-artifact@v4 From d637ac06bbd9619368fc773a5146805fbc1d6a3e Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 00:58:17 +0500 Subject: [PATCH 02/12] ci(test-report): add test report name to action --- .github/workflows/lint-and-test.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index a3f485a..812e301 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -114,6 +114,7 @@ jobs: uses: dorny/test-reporter@v1 if: always() with: + name: Unit Tests Report path: test-report.xml reporter: python-xunit From b45682bd47461dabe5a5e9ab99e3bfcde82fcdfc Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 01:01:31 +0500 Subject: [PATCH 03/12] ci: change tests report reporter because python-xunit not supported --- .github/workflows/lint-and-test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 812e301..f87fbc1 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -116,7 +116,7 @@ jobs: with: name: Unit Tests Report path: test-report.xml - reporter: python-xunit + reporter: java-junit - name: Upload coverage artifact if: (matrix.python-version == '3.12') && (matrix.os == 'ubuntu-latest') From 38e5bead3b1f9ab6cadd0973d58f980fe29f40f6 Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 01:28:44 +0500 Subject: [PATCH 04/12] ci: add check write permission for avoid HttpError: Resource not accessible by integration --- .github/workflows/lint-and-test.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index f87fbc1..4bf1092 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -15,6 +15,7 @@ on: permissions: contents: read packages: read + checks: write concurrency: group: ci-${{ github.workflow }}-${{ github.ref }} From 7bdeced471c1e79341e316185d99081fa854cd01 Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:18:10 +0500 Subject: [PATCH 05/12] ci: add matrix and python version to test report path --- .github/workflows/lint-and-test.yml | 2 +- .gitignore | 1 + tests/c2pa/claim_test.py | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 4bf1092..4b4be9d 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -116,7 +116,7 @@ jobs: if: always() with: name: Unit Tests Report - path: test-report.xml + path: test-report-${{ matrix.os }}-${{ matrix.python-version }}.xml reporter: java-junit - name: Upload coverage artifact diff --git a/.gitignore b/.gitignore index f74909d..8a1af5f 100644 --- a/.gitignore +++ b/.gitignore @@ -9,3 +9,4 @@ dist **/.ipynb_checkpoints **/.Trash* **/.DS_Store +.coverage diff --git a/tests/c2pa/claim_test.py b/tests/c2pa/claim_test.py index 987994f..055c7c5 100644 --- a/tests/c2pa/claim_test.py +++ b/tests/c2pa/claim_test.py @@ -13,7 +13,7 @@ def test_create_claim_with_label(): ) assert test_claim is not None - assert test_claim.claim_generator == "c2pie" + assert test_claim.claim_generator == "cpie" assert test_claim.manifest_label == "valid_manifest_label" assert test_claim.claim_signature_label == "self#jumbf=c2pa/valid_manifest_label/c2pa.signature" From 6700107db933457cfde02fe20250b3de9a714bfa Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:18:10 +0500 Subject: [PATCH 06/12] ci: add matrix and python version to test report path --- .github/workflows/lint-and-test.yml | 4 ++-- .gitignore | 1 + tests/c2pa/claim_test.py | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 4bf1092..36d0d7c 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -108,7 +108,7 @@ jobs: poetry run pytest \ --cov=c2pie \ -m "not e2e" \ - --junit-xml=test-report.xml \ + --junit-xml=test-report-${{ matrix.os }}-${{ matrix.python-version }}.xml \ -v - name: Publish test report @@ -116,7 +116,7 @@ jobs: if: always() with: name: Unit Tests Report - path: test-report.xml + path: test-report-${{ matrix.os }}-${{ matrix.python-version }}.xml reporter: java-junit - name: Upload coverage artifact diff --git a/.gitignore b/.gitignore index f74909d..8a1af5f 100644 --- a/.gitignore +++ b/.gitignore @@ -9,3 +9,4 @@ dist **/.ipynb_checkpoints **/.Trash* **/.DS_Store +.coverage diff --git a/tests/c2pa/claim_test.py b/tests/c2pa/claim_test.py index 987994f..055c7c5 100644 --- a/tests/c2pa/claim_test.py +++ b/tests/c2pa/claim_test.py @@ -13,7 +13,7 @@ def test_create_claim_with_label(): ) assert test_claim is not None - assert test_claim.claim_generator == "c2pie" + assert test_claim.claim_generator == "cpie" assert test_claim.manifest_label == "valid_manifest_label" assert test_claim.claim_signature_label == "self#jumbf=c2pa/valid_manifest_label/c2pa.signature" From afedad1a56307d5d7cf7e2800efc486fdb67fea9 Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:30:07 +0500 Subject: [PATCH 07/12] ci: remove failing test report publishing if tests contains error --- .github/workflows/lint-and-test.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 36d0d7c..a23341d 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -118,6 +118,8 @@ jobs: name: Unit Tests Report path: test-report-${{ matrix.os }}-${{ matrix.python-version }}.xml reporter: java-junit + # Not set action as failed if test report contains any failed test + fail-on-error: false - name: Upload coverage artifact if: (matrix.python-version == '3.12') && (matrix.os == 'ubuntu-latest') From e525fe676e625c4e93818c8f56475cccffa445e7 Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:34:07 +0500 Subject: [PATCH 08/12] ci: add only summary generating without all tests list for more clean results --- .github/workflows/lint-and-test.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index a23341d..cf58f0a 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -120,6 +120,8 @@ jobs: reporter: java-junit # Not set action as failed if test report contains any failed test fail-on-error: false + # Allows generate only summary without all tests list for more clear results + only-summary: true - name: Upload coverage artifact if: (matrix.python-version == '3.12') && (matrix.os == 'ubuntu-latest') From 0dd0de9c5051969f5c08c0065cfc23ec6bae74ec Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:47:45 +0500 Subject: [PATCH 09/12] ci(test-report): add test report genetating to PR comments on every tests run for more readable and clear testing results --- .github/workflows/lint-and-test.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index cf58f0a..8d009f0 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -122,6 +122,8 @@ jobs: fail-on-error: false # Allows generate only summary without all tests list for more clear results only-summary: true + list-suites: failed + list-tests: failed - name: Upload coverage artifact if: (matrix.python-version == '3.12') && (matrix.os == 'ubuntu-latest') From b2c870635bc3663ba05c50fa73b793098aa8ea5c Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:49:45 +0500 Subject: [PATCH 10/12] ci: fix script --- .github/workflows/lint-and-test.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 8d009f0..3ad73d5 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -120,8 +120,8 @@ jobs: reporter: java-junit # Not set action as failed if test report contains any failed test fail-on-error: false - # Allows generate only summary without all tests list for more clear results - only-summary: true + # # Allows generate only summary without all tests list for more clear results + # only-summary: true list-suites: failed list-tests: failed From ca4db39abfaa3d7e9e9df1e88a5c748521893a9a Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:51:32 +0500 Subject: [PATCH 11/12] ci: fix script --- .github/workflows/lint-and-test.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 3ad73d5..652d71a 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -122,7 +122,7 @@ jobs: fail-on-error: false # # Allows generate only summary without all tests list for more clear results # only-summary: true - list-suites: failed + # list-suites: failed list-tests: failed - name: Upload coverage artifact From 9088f8f5a5d414b2f401ed097c70b9f570784e1d Mon Sep 17 00:00:00 2001 From: Maria Yadryshnikova <93837471+myadryshnikova@users.noreply.github.com> Date: Sun, 28 Dec 2025 12:54:22 +0500 Subject: [PATCH 12/12] ci: fix script --- .github/workflows/lint-and-test.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.github/workflows/lint-and-test.yml b/.github/workflows/lint-and-test.yml index 652d71a..7211785 100644 --- a/.github/workflows/lint-and-test.yml +++ b/.github/workflows/lint-and-test.yml @@ -10,6 +10,9 @@ on: - 'docs/**' tags-ignore: - v** + branches: + - feature/* + workflow_call: permissions: