From 6b584ee452cefd5b7d06075b6b7c5d96659c35e1 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 26 Aug 2025 19:50:50 +0000 Subject: [PATCH] test_common: delete wal param when creating inbox We don't want the `{wal}' field leaking into the Inbox object, it only needs to be in the hashref populated by Getopt::Long. --- lib/PublicInbox/TestCommon.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/PublicInbox/TestCommon.pm b/lib/PublicInbox/TestCommon.pm index 7b173b611..295b51d6e 100644 --- a/lib/PublicInbox/TestCommon.pm +++ b/lib/PublicInbox/TestCommon.pm @@ -948,7 +948,7 @@ sub create_inbox ($;@) { $opt{-primary_address} //= $addr->[0] // "$ident\@example.com"; my $parallel = delete($opt{importer_parallel}) // 0; my $creat_opt = { nproc => delete($opt{nproc}) // 1 }; - $creat_opt->{wal} = 1 if $opt{wal}; + $creat_opt->{wal} = 1 if delete $opt{wal}; my $ibx = PublicInbox::InboxWritable->new({ %opt }, $creat_opt); if (!-f "$dir/creat.stamp") { my $im = $ibx->importer($parallel); -- 2.47.3