From 67df3e1e85335169afb591619a605768bdd1a625 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Date: Tue, 14 Apr 2026 13:41:18 +0100
Subject: [PATCH] Bug 41901: (follow-up) Fix perlcritic violation in unit tests

Remove explicit "return undef" in favour of bare "return" to
satisfy Perl::Critic (PBP p.199).

Signed-off-by: Brian J. Barr <brian.barr@churchofjesuschrist.org>
Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
---
 t/db_dependent/api/v1/authorities.t | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/t/db_dependent/api/v1/authorities.t b/t/db_dependent/api/v1/authorities.t
index 6beedf8463..5aac5790f4 100755
--- a/t/db_dependent/api/v1/authorities.t
+++ b/t/db_dependent/api/v1/authorities.t
@@ -180,7 +180,7 @@ subtest 'post() tests' => sub {
         'fetch',
         sub {
             my ( $class, $id ) = @_;
-            return undef if $id == 999999;
+            return if $id == 999999;
             my $rt      = ( $id == 42 ) ? 'biblio' : 'authority';
             my @matches = ();
             @matches = (