From: Eric W. Biederman <ebiederm@xmission.com>
Date: Wed, 18 Jul 2018 00:32:01 +0000 (-0500)
Subject: t/v2reindex.t: Swap the order of minmax tests so errors make sense
X-Git-Tag: v1.2.0~485
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=90577102f4eabdf57b570457062de07398b72415;p=public-inbox.git

t/v2reindex.t: Swap the order of minmax tests so errors make sense

Previously if a minmax test failed it would say it was expecting the
incorrect value, which is confusing when looking into why the test
fails.

Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
---

diff --git a/t/v2reindex.t b/t/v2reindex.t
index 5bc307f1..8af30991 100644
--- a/t/v2reindex.t
+++ b/t/v2reindex.t
@@ -63,7 +63,7 @@ $im->done;
 ok(-d $xap, 'Xapian directories recreated');
 
 delete $ibx->{mm};
-is_deeply($minmax, [ $ibx->mm->minmax ], 'minmax unchanged');
+is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
 
 ok(unlink "$mainrepo/msgmap.sqlite3", 'remove msgmap');
 remove_tree($xap);
@@ -77,7 +77,7 @@ ok(!-d $xap, 'Xapian directories removed again');
 	$im->done;
 	ok(-d $xap, 'Xapian directories recreated');
 	delete $ibx->{mm};
-	is_deeply($minmax, [ $ibx->mm->minmax ], 'minmax unchanged');
+	is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
 }
 
 ok(unlink "$mainrepo/msgmap.sqlite3", 'remove msgmap');
@@ -92,7 +92,7 @@ ok(!-d $xap, 'Xapian directories removed again');
 	$im->done;
 	ok(-d $xap, 'Xapian directories recreated');
 	delete $ibx->{mm};
-	is_deeply($minmax, [ $ibx->mm->minmax ], 'minmax unchanged');
+	is_deeply([ $ibx->mm->minmax ], $minmax, 'minmax unchanged');
 }
 
 done_testing();