From db881d7c3dcfba0426baaa0642b523cbe787344b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?K=C5=8Dan?= Date: Thu, 19 Mar 2026 00:10:21 -0600 Subject: [PATCH] ci: use prove directly instead of make test fallback Replace `make && ( make test || prove -wbvm t/*.t )` with `make && prove -wbvm t/*.t` in all CI jobs. The old pattern ran tests twice on failure: first via `make test` (TAP summary only), then via `prove -wbvm` (verbose). This doubled CI time on failures and flooded logs with redundant output. Since `prove -wbvm` already provides better diagnostics than `make test`, just use it directly. Co-Authored-By: Claude Opus 4.6 --- .github/workflows/testsuite.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/testsuite.yml b/.github/workflows/testsuite.yml index a4c0153..01bddb3 100644 --- a/.github/workflows/testsuite.yml +++ b/.github/workflows/testsuite.yml @@ -38,7 +38,7 @@ jobs: cpanfile: "cpanfile" - name: Makefile.PL run: perl -I$(pwd) Makefile.PL - - run: make && ( make test || prove -wbvm t/*.t ) + - run: make && prove -wbvm t/*.t openssl-matrix: timeout-minutes: 5 @@ -136,7 +136,7 @@ jobs: # not available < 5.12 cpanm --notest Test::Kwalitee ||: - run: perl Makefile.PL - - run: make && ( make test || prove -wbvm t/*.t ) + - run: make && prove -wbvm t/*.t # # Windows @@ -177,7 +177,7 @@ jobs: - name: Deps for testing run: cpanm --notest Crypt::OpenSSL::Random Crypt::OpenSSL::Guess - run: perl Makefile.PL - - run: make && ( make test || prove -wbvm t/*.t ) + - run: make && prove -wbvm t/*.t # # macOS — covers LibreSSL (system default) and Homebrew OpenSSL 3.x @@ -223,4 +223,4 @@ jobs: run: cpanm --notest Crypt::OpenSSL::Guess Crypt::OpenSSL::Random Crypt::OpenSSL::Bignum - name: Makefile.PL run: perl Makefile.PL - - run: make && ( make test || prove -wbvm t/*.t ) \ No newline at end of file + - run: make && prove -wbvm t/*.t \ No newline at end of file