about summary refs log tree commit homepage
path: root/t
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2023-10-10 09:03:09 +0000
committerEric Wong <e@80x24.org>2023-10-10 19:11:44 +0000
commitc3599884198dd5db51ed82058b08fc6851a3e375 (patch)
tree4c2febf4f9c14b12254983e02724b6161e279f87 /t
parent0205e902056f9900c77840138894bf8884ea6568 (diff)
downloadpublic-inbox-c3599884198dd5db51ed82058b08fc6851a3e375.tar.gz
Occasionally, t/nntp.t spews undefined variable warnings under
`make check-run'.  While the test doesn't fail, it's annoying
to see them and it could be a source of deeper problems.
Diffstat (limited to 't')
-rw-r--r--t/nntp.t7
1 files changed, 5 insertions, 2 deletions
diff --git a/t/nntp.t b/t/nntp.t
index 0d3384d7..42a4ea97 100644
--- a/t/nntp.t
+++ b/t/nntp.t
@@ -3,14 +3,17 @@
 use v5.12;
 use PublicInbox::TestCommon;
 use PublicInbox::Eml;
-require_mods(qw(DBD::SQLite Data::Dumper));
+require_mods(qw(DBD::SQLite));
 use_ok 'PublicInbox::NNTP';
 use PublicInbox::Config;
 use POSIX qw(strftime);
+use Data::Dumper;
 
 {
         my $quote_str = sub {
-                my (undef, $s) = split(/ = /, Data::Dumper::Dumper($_[0]), 2);
+                my ($orig) = @_;
+                my (undef, $s) = split(/ = /, Dumper($orig), 2);
+                $s // diag explain(['$s undefined, $orig = ', $orig]);
                 $s =~ s/;\n//;
                 $s;
         };