about summary refs log tree commit homepage
path: root/t/extsearch.t
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2023-11-15 04:32:39 +0000
committerEric Wong <e@80x24.org>2023-11-15 08:02:57 +0000
commit4bf3fe6a811c6e024af3580f2da15c234fb53011 (patch)
tree6ba664947cc5cbe2cec26e065dff02071b6c706f /t/extsearch.t
parent7d06b126e9395d96a03b37daf49019925ff0ec76 (diff)
downloadpublic-inbox-4bf3fe6a811c6e024af3580f2da15c234fb53011.tar.gz
Avoid mixing autodie use in different scopes since it's likely
to cause problems like it did in Gcf2.  While none of these
fix known problems with test cases, it's likely worthwhile to
avoid it anyways to avoid future surprises.

For Process::IO, we'll add some additional tests in t/io.t
to ensure we don't get unintended exceptions for try_cat.
Diffstat (limited to 't/extsearch.t')
-rw-r--r--t/extsearch.t3
1 files changed, 1 insertions, 2 deletions
diff --git a/t/extsearch.t b/t/extsearch.t
index 1a1eb350..090f6db5 100644
--- a/t/extsearch.t
+++ b/t/extsearch.t
@@ -7,7 +7,7 @@ use PublicInbox::Config;
 use PublicInbox::InboxWritable;
 require_git(2.6);
 require_mods(qw(json DBD::SQLite Xapian));
-use autodie qw(open rename truncate);
+use autodie qw(open rename truncate unlink);
 require PublicInbox::Search;
 use_ok 'PublicInbox::ExtSearch';
 use_ok 'PublicInbox::ExtSearchIdx';
@@ -577,7 +577,6 @@ test_lei(sub {
         is_deeply($dst, \@md1,
                 "convert from extindex w/ or w/o `extindex' prefix");
 
-        use autodie qw(unlink);
         my @o = glob "$home/extindex/ei*/over.sqlite*";
         unlink(@o);
         ok(!lei('convert', '-o', "$home/fail", "extindex:$d"));