minor #17071 [Filesystem] fix tests on 2.3 (Christian Flothmann)

fabpot · fabpot · commit 36a514181f3e · 2015-12-20T20:57:55.000+01:00
This PR was merged into the 2.3 branch. Discussion ---------- [Filesystem] fix tests on 2.3 | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #16797 | License | MIT | Doc PR | The test introduced in #16797 used the `assertFilePermissions()` method to test for the expected result. This worked quite well for the PR as it was submitted for the `master` branch. However, the tests now fail on 2.3 as the `FilesystemTestCase` class which contains this method was introduced with Symfony 2.4. Commits ------- 9050f67 [Filesystem] fix tests on 2.3
diff --git a/src/Symfony/Component/Filesystem/Tests/FilesystemTest.php b/src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
@@ -492,7 +492,7 @@ public function testChmodChangesZeroModeOnSubdirectoriesOnRecursive()
 
         $this->filesystem->chmod($directory, 0753, 0000, true);
 
-        $this->assertFilePermissions(753, $subdirectory);
+        $this->assertEquals(753, $this->getFilePermissions($subdirectory));
     }
 
     public function testChown()

Original file line number	Diff line number	Diff line change
`@@ -492,7 +492,7 @@ public function testChmodChangesZeroModeOnSubdirectoriesOnRecursive()`
`492`	`492`
`493`	`493`	`$this->filesystem->chmod($directory, 0753, 0000, true);`
`494`	`494`
`495`		`- $this->assertFilePermissions(753, $subdirectory);`
	`495`	`+ $this->assertEquals(753, $this->getFilePermissions($subdirectory));`
`496`	`496`	`}`
`497`	`497`
`498`	`498`	`public function testChown()`