[Config] Make ifFalse() consistent between value and closure based checks

Author: arjenm

src/Symfony/Component/Config/Definition/Builder/ExprBuilder.php

@@ -76,7 +76,7 @@ public function ifTrue(?\Closure $closure = null): static
      */
     public function ifFalse(?\Closure $closure = null): static
     {
-        $this->ifPart = $closure ?? static fn ($v) => false === $v;
+        $this->ifPart = $closure ? static fn ($v) => !$closure($v) : static fn ($v) => false === $v;
         $this->allowedTypes = self::TYPE_ANY;
 
         return $this;

src/Symfony/Component/Config/Tests/Definition/Builder/ExprBuilderTest.php

@@ -42,11 +42,23 @@ public function testIfTrueExpression()
         ->end();
         $this->assertFinalizedValueIs('new_value', $test);
 
+        $test = $this->getTestBuilder()
+            ->ifTrue(fn () => 1)
+            ->then($this->returnClosure('new_value'))
+        ->end();
+        $this->assertFinalizedValueIs('new_value', $test);
+
         $test = $this->getTestBuilder()
             ->ifTrue(fn () => false)
             ->then($this->returnClosure('new_value'))
         ->end();
         $this->assertFinalizedValueIs('value', $test);
+
+        $test = $this->getTestBuilder()
+            ->ifTrue(fn () => 0)
+            ->then($this->returnClosure('new_value'))
+        ->end();
+        $this->assertFinalizedValueIs('value', $test);
     }
 
     public function testIfFalseExpression()
@@ -58,16 +70,28 @@ public function testIfFalseExpression()
         $this->assertFinalizedValueIs('new_value', $test, ['key' => false]);
 
         $test = $this->getTestBuilder()
-            ->ifFalse(fn () => true)
+            ->ifFalse(fn ($v) => 'value' === $v)
             ->then($this->returnClosure('new_value'))
         ->end();
-        $this->assertFinalizedValueIs('new_value', $test);
+        $this->assertFinalizedValueIs('value', $test);
 
         $test = $this->getTestBuilder()
-            ->ifFalse(fn () => false)
+            ->ifFalse(fn ($v) => 1)
             ->then($this->returnClosure('new_value'))
         ->end();
         $this->assertFinalizedValueIs('value', $test);
+
+        $test = $this->getTestBuilder()
+            ->ifFalse(fn ($v) => 'other_value' === $v)
+            ->then($this->returnClosure('new_value'))
+        ->end();
+        $this->assertFinalizedValueIs('new_value', $test);
+
+        $test = $this->getTestBuilder()
+            ->ifFalse(fn ($v) => 0)
+            ->then($this->returnClosure('new_value'))
+            ->end();
+        $this->assertFinalizedValueIs('new_value', $test);
     }
 
     public function testIfStringExpression()