From 999e8c1b3d54f4504cd4fa87949a077da325a1af Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sun, 26 Jul 2020 09:03:15 +0000 Subject: t/init.t: don't modify ~/.public-inbox/ Tests for failures should not leave junk temporary files lying around in a users' ~/.public-inbox/. On a side note, I'm not sure if PI_DIR is or was ever necessary. It's never been documented, so perhaps using $HOME for this is better... --- t/init.t | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 't/init.t') diff --git a/t/init.t b/t/init.t index b8f17b5c..16581955 100644 --- a/t/init.t +++ b/t/init.t @@ -50,12 +50,15 @@ sub quiet_fail { '-init did not unlink lock on failure'); } { + my $env = { PI_DIR => "$tmpdir/.public-inbox/" }; my $rdr = { 2 => \(my $err = '') }; my $cmd = [ '-init', 'alist', "$tmpdir/a\nlist", qw(http://example.com/alist alist@example.com) ]; - ok(!run_script($cmd, undef, $rdr), + ok(!run_script($cmd, $env, $rdr), 'public-inbox-init rejects LF in inboxdir'); like($err, qr/`\\n' not allowed in `/s, 'reported \\n'); + is_deeply([glob("$tmpdir/.public-inbox/pi-init-*")], [], + 'no junk files left behind'); } SKIP: { -- cgit v1.2.3-24-ge0c7