user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
 Warning: Initial query:
 %22t/multi-mid: don%27t access ~/.public-inbox/config%22
 returned no results, used:
 "t/multi-mid: don't access ~/.public-inbox/config"
 instead

Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH] t/multi-mid: don't access ~/.public-inbox/config
@ 2020-02-06  9:18  7% Eric Wong
  0 siblings, 0 replies; 1+ results
From: Eric Wong @ 2020-02-06  9:18 UTC (permalink / raw)
  To: meta

It can cause unpredictable behavior and also slow things down.

Followup-to: e4d3be19612b2082 ("t: localize the PI_CONFIG env")
---
 t/multi-mid.t | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/t/multi-mid.t b/t/multi-mid.t
index 0d283a0d..94c0e0a2 100644
--- a/t/multi-mid.t
+++ b/t/multi-mid.t
@@ -48,8 +48,8 @@ for my $order ([$bad, $good], [$good, $bad]) {
 	}
 	my $rdr = { 1 => \(my $out = ''), 2 => \(my $err = '') };
 	my $cmd = [ '-convert', $ibx->{inboxdir}, "$tmpdir/v2" ];
-	ok(run_script($cmd, undef, $rdr),
-		'convert to v2');
+	my $env = { PI_DIR => "$tmpdir/.public-inbox" };
+	ok(run_script($cmd, $env, $rdr), 'convert to v2');
 	$err =~ s!\AW: $tmpdir/v1 not configured[^\n]+\n!!s;
 	is($err, '', 'no errors or warnings from -convert');
 	$ibx->{version} = 2;

^ permalink raw reply related	[relevance 7%]

Results 1-1 of 1 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2020-02-06  9:18  7% [PATCH] t/multi-mid: don't access ~/.public-inbox/config Eric Wong

Code repositories for project(s) associated with this public inbox

	https://80x24.org/public-inbox.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).