[SassTask] [core] - Fixed Filesystem::listContents for numeric filenames.

src/Phing/Io/FileSystem.php

@@ -869,14 +869,17 @@ public function compareMTimes($file1, $file2)
      */
     public function listContents(File $f)
     {
-        return array_keys(
+        $filenames = array_keys(
             iterator_to_array(
                 new FilesystemIterator(
                     $f->getAbsolutePath(),
                     FilesystemIterator::KEY_AS_FILENAME
                 )
             )
         );
+
+        // make sure numeric filenames (e.g. 11) are properly treated as strings
+        return array_map('strval', $filenames);
     }
 
     /**

tests/Phing/Io/FileSystemTest.php

@@ -102,4 +102,16 @@ public function testWhichHinkyExecutableNameWithSeparator()
         $path = $fs->which('zx:\tasword.bin');
         $this->assertEquals($path, false);
     }
+
+    public function testListContentsWithNumericName()
+    {
+        $fs = FileSystem::getFileSystem();
+
+        $parentDir = new File(__DIR__ . '/../../etc/system/io/testdir');
+        $contents = $fs->listContents($parentDir);
+
+        foreach ($contents as $filename) {
+            self::assertIsString($filename);
+        }
+    }
 }

tests/classes/phing/tasks/ext/SassTask/Integration/SassCompilerTest.php

@@ -72,14 +72,14 @@ public function testItNotProducesAnyCompiledOutputWhenNoInput(): void
 
     public function testItThrowsExceptionWhenFailOnErrorIsSet(): void
     {
+        $this->expectException(BuildException::class);
+
         $this->compiler->compile(
             self::SASS_TEST_BASE . 'non-existing.sass',
             self::SASS_TEST_BASE . 'test.css',
             true
         );
 
-        $this->expectException(BuildException::class);
-
         $this->assertFileDoesNotExist(self::SASS_TEST_BASE . 'test.css');
     }
 }