]> Sergey Matveev's repositories - public-inbox.git/commitdiff
t/v2reindex.t: Swap the order of minmax tests so errors make sense
authorEric W. Biederman <ebiederm@xmission.com>
Wed, 18 Jul 2018 00:32:01 +0000 (19:32 -0500)
committerEric Wong <e@80x24.org>
Wed, 18 Jul 2018 10:24:21 +0000 (10:24 +0000)
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>
t/v2reindex.t

index 5bc307f1cac11ed34c3c3ec6e4c8258544c85652..8af30991f858d52537f9813b64a98945f7162227 100644 (file)
@@ -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();