[SwitchTask] Added test case for numeric case values

src/Phing/Task/System/CaseTask.php

@@ -30,9 +30,9 @@ class CaseTask extends SequentialTask
     private $value;
 
     /**
-     * @param mixed $value
+     * @param string $value
      */
-    public function setValue($value): void
+    public function setValue(string $value): void
     {
         $this->value = $value;
     }

src/Phing/Task/System/SwitchTask.php

@@ -152,7 +152,7 @@ public function main()
         foreach ($this->cases as $case) {
             $cValue = $case->getValue();
 
-            if (empty($cValue)) {
+            if ($cValue === null || $cValue === '') {
                 throw new BuildException('Value is required for case.');
             }
 

tests/Phing/Test/Task/System/SwitchTaskTest.php

@@ -43,6 +43,11 @@ public function testSwitchCase(): void
         $this->expectLogContaining(__FUNCTION__, 'The value of property foo is bar');
     }
 
+    public function testSwitchCaseNumbers(): void
+    {
+        $this->expectLogContaining(__FUNCTION__, 'The value of property foo is 1');
+    }
+
     public function testSwitchDefault(): void
     {
         $this->expectLogContaining(__FUNCTION__, 'The value of property bar is not sensible');

tests/etc/tasks/system/SwitchTest.xml

@@ -14,6 +14,20 @@
             </default>
         </switch>
     </target>
+    <target name="testSwitchCaseNumbers">
+        <property name="foo" value="1"/>
+        <switch value="${foo}">
+            <case value="0">
+                <echo message="The value of property foo is 0" />
+            </case>
+            <case value="1">
+                <echo message="The value of property foo is 1" />
+            </case>
+            <default>
+                <echo message="The value of property foo is not sensible" />
+            </default>
+        </switch>
+    </target>
     <target name="testSwitchDefault">
         <property name="bar" value="test"/>
         <switch value="${bar}">