[ReflexiveTask] Fixed behavior, if file attribute points to a directory.

src/Phing/Task/System/ReflexiveTask.php

@@ -61,6 +61,7 @@ class ReflexiveTask extends Task
 
     /**
      * Single file to process.
+     * @var File
      */
     private $file;
 
@@ -82,6 +83,10 @@ public function main()
             throw new BuildException("You must specify a file or fileset(s) for the <reflexive> task.");
         }
 
+        if ($this->file->isDirectory()) {
+            throw new BuildException('File cannot be a directory.');
+        }
+
         // compile a list of all files to modify, both file attrib and fileset elements
         // can be used.
 
@@ -133,7 +138,7 @@ public function main()
                 if ($in) {
                     $in->close();
                 }
-                $this->log("Erorr reading file: " . $e->getMessage(), Project::MSG_WARN);
+                $this->log("Error reading file: " . $e->getMessage(), Project::MSG_WARN);
             }
 
             try {

tests/Phing/Task/System/ReflexiveTaskTest.php

@@ -0,0 +1,55 @@
+<?php
+/**
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * This software consists of voluntary contributions made by many individuals
+ * and is licensed under the LGPL. For more information please see
+ * <http://phing.info>.
+ */
+
+namespace Phing\Task\System;
+
+use Phing\Project;
+use Phing\Support\BuildFileTest;
+
+/**
+ * Tests the Copy Task
+ *
+ * @author Siad Ardroumli <siad.ardroumli@gmail.com>
+ */
+class ReflexiveTaskTest extends BuildFileTest
+{
+    public function setUp(): void
+    {
+        $this->configureProject(
+            PHING_TEST_BASE
+            . "/etc/tasks/system/ReflexiveTaskTest.xml"
+        );
+        $this->executeTarget("setup");
+    }
+
+    public function tearDown(): void
+    {
+        $this->executeTarget("clean");
+    }
+
+    public function test1()
+    {
+        $this->expectBuildException(__FUNCTION__, 'You must specify a file or fileset(s) for the <reflexive> task.');
+    }
+
+    public function test2()
+    {
+        $this->expectBuildException(__FUNCTION__, 'File cannot be a directory.');
+    }
+}

tests/etc/tasks/system/ReflexiveTaskTest.xml

@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<project name="CopyTaskTest" default="main">
+    <property name="tmp.dir" value="tmp"/>
+
+    <target name="setup">
+        <mkdir dir="${tmp.dir}"/>
+    </target>
+
+    <target name="clean">
+        <delete dir="${tmp.dir}"/>
+    </target>
+
+    <target name="test1">
+        <reflexive/>
+    </target>
+
+    <target name="test2">
+        <reflexive file=""/>
+    </target>
+
+    <target name="main"/>
+</project>