]> Sergey Matveev's repositories - public-inbox.git/commitdiff
t/config.t: always compare against git bool behavior
authorEric Wong <e@yhbt.net>
Wed, 10 Jun 2020 07:04:43 +0000 (07:04 +0000)
committerEric Wong <e@yhbt.net>
Sat, 13 Jun 2020 07:55:45 +0000 (07:55 +0000)
We'll use the xqx() to avoid losing too much performance
compared to normal `backtick` (qx) when testing using
"make check-run" + Inline::C.

t/config.t

index 1f50bb86a0963bd5ac63e0817701a45c796846a5..3f41c0042a9e962b0628b01c05af8b2dfcc9c6ba 100644 (file)
@@ -209,20 +209,17 @@ EOF
 }
 
 {
-       my $check_git = !!$ENV{CHECK_GIT_BOOL};
        for my $t (qw(TRUE true yes on 1 +1 -1 13 0x1 0x12 0X5)) {
                is(PublicInbox::Config::_git_config_bool($t), 1, "$t is true");
-               if ($check_git) {
-                       is(`git -c test.val=$t config --bool test.val`,
-                               "true\n", "$t matches git-config behavior");
-               }
+               is(xqx([qw(git -c), "test.val=$t",
+                       qw(config --bool test.val)]),
+                       "true\n", "$t matches git-config behavior");
        }
        for my $f (qw(FALSE false no off 0 +0 +000 00 0x00 0X0)) {
                is(PublicInbox::Config::_git_config_bool($f), 0, "$f is false");
-               if ($check_git) {
-                       is(`git -c test.val=$f config --bool test.val`,
-                               "false\n", "$f matches git-config behavior");
-               }
+               is(xqx([qw(git -c), "test.val=$f",
+                       qw(config --bool test.val)]),
+                       "false\n", "$f matches git-config behavior");
        }
        is(PublicInbox::Config::_git_config_bool('bogus'), undef,
                'bogus is undef');