diff options
author | Thomas Müller <DeepDiver1975@users.noreply.github.com> | 2016-09-06 15:27:30 +0200 |
---|---|---|
committer | Morris Jobke <hey@morrisjobke.de> | 2016-09-12 16:46:39 +0200 |
commit | 6eba111e0981100e9866e449c2198d5bca3079ec (patch) | |
tree | e2dfa822f2d5a6dd9f10f84a250be6295f8e51f0 /tests | |
parent | a9c243948b41db6f492cfdbb3e16cb8731f38b78 (diff) | |
download | nextcloud-server-6eba111e0981100e9866e449c2198d5bca3079ec.tar.gz nextcloud-server-6eba111e0981100e9866e449c2198d5bca3079ec.zip |
Fix failing preview unit tests (#26039)
Diffstat (limited to 'tests')
-rw-r--r-- | tests/lib/PreviewTest.php | 42 |
1 files changed, 21 insertions, 21 deletions
diff --git a/tests/lib/PreviewTest.php b/tests/lib/PreviewTest.php index cd228801815..2d6f53e2ce8 100644 --- a/tests/lib/PreviewTest.php +++ b/tests/lib/PreviewTest.php @@ -23,8 +23,10 @@ namespace Test; use OC\Files\FileInfo; +use OC\Files\Filesystem; use OC\Files\Storage\Temporary; use OC\Files\View; +use OC\Preview; use Test\Traits\MountProviderTrait; use Test\Traits\UserTrait; @@ -41,7 +43,7 @@ class PreviewTest extends TestCase { const TEST_PREVIEW_USER1 = "test-preview-user1"; - /** @var \OC\Files\View */ + /** @var View */ private $rootView; /** * Note that using 756 with an image with a ratio of 1.6 brings interesting rounding issues @@ -77,10 +79,10 @@ class PreviewTest extends TestCase { $this->createUser(self::TEST_PREVIEW_USER1, self::TEST_PREVIEW_USER1); $this->loginAsUser(self::TEST_PREVIEW_USER1); - $storage = new \OC\Files\Storage\Temporary([]); - \OC\Files\Filesystem::mount($storage, [], '/' . self::TEST_PREVIEW_USER1 . '/'); + $storage = new Temporary([]); + Filesystem::mount($storage, [], '/' . self::TEST_PREVIEW_USER1 . '/'); - $this->rootView = new \OC\Files\View(''); + $this->rootView = new View(''); $this->rootView->mkdir('/' . self::TEST_PREVIEW_USER1); $this->rootView->mkdir('/' . self::TEST_PREVIEW_USER1 . '/files'); @@ -133,7 +135,7 @@ class PreviewTest extends TestCase { $x = 50; $y = 50; - $preview = new \OC\Preview(self::TEST_PREVIEW_USER1, 'files/', 'test.txt', $x, $y); + $preview = new Preview(self::TEST_PREVIEW_USER1, 'files/', 'test.txt', $x, $y); $preview->getPreview(); $fileInfo = $this->rootView->getFileInfo($sampleFile); @@ -164,14 +166,14 @@ class PreviewTest extends TestCase { $x = 50; $y = 50; - $preview = new \OC\Preview(self::TEST_PREVIEW_USER1, 'files/', 'test.txt', $x, $y); + $preview = new Preview(self::TEST_PREVIEW_USER1, 'files/', 'test.txt', $x, $y); $preview->getPreview(); $fileInfo = $this->rootView->getFileInfo($sampleFile); /** @var int $fileId */ $fileId = $fileInfo['fileid']; - $thumbCacheFolder = '/' . self::TEST_PREVIEW_USER1 . '/' . \OC\Preview::THUMBNAILS_FOLDER . + $thumbCacheFolder = '/' . self::TEST_PREVIEW_USER1 . '/' . Preview::THUMBNAILS_FOLDER . '/' . $fileId . '/'; $this->assertSame(true, $this->rootView->is_dir($thumbCacheFolder), "$thumbCacheFolder \n"); @@ -203,7 +205,7 @@ class PreviewTest extends TestCase { $sample = '/' . self::TEST_PREVIEW_USER1 . '/files/test.' . $extension; $this->rootView->file_put_contents($sample, $data); - $preview = new \OC\Preview( + $preview = new Preview( self::TEST_PREVIEW_USER1, 'files/', 'test.' . $extension, $x, $y ); @@ -233,7 +235,7 @@ class PreviewTest extends TestCase { $this->rootView->file_put_contents($imgPath, $imgData); $preview = - new \OC\Preview(self::TEST_PREVIEW_USER1, 'files/', 'testimage.odt', $width, $height); + new Preview(self::TEST_PREVIEW_USER1, 'files/', 'testimage.odt', $width, $height); $preview->getPreview(); $image = $preview->getPreview(); @@ -310,8 +312,6 @@ class PreviewTest extends TestCase { public function testCreateMaxAndNormalPreviewsAtFirstRequest( $sampleId, $widthAdjustment, $heightAdjustment, $keepAspect = false, $scalingUp = false ) { - //$this->markTestSkipped('Not testing this at this time'); - // Get the right sample for the experiment $this->getSample($sampleId); $sampleWidth = $this->sampleWidth; @@ -328,7 +328,7 @@ class PreviewTest extends TestCase { $preview = $this->createPreview($previewWidth, $previewHeight); // There should be no cached thumbnails - $thumbnailFolder = '/' . self::TEST_PREVIEW_USER1 . '/' . \OC\Preview::THUMBNAILS_FOLDER . + $thumbnailFolder = '/' . self::TEST_PREVIEW_USER1 . '/' . Preview::THUMBNAILS_FOLDER . '/' . $sampleFileId; $this->assertSame(false, $this->rootView->is_dir($thumbnailFolder)); @@ -562,10 +562,10 @@ class PreviewTest extends TestCase { * @param int $width * @param int $height * - * @return \OC\Preview + * @return Preview */ private function createPreview($width, $height) { - $preview = new \OC\Preview( + $preview = new Preview( self::TEST_PREVIEW_USER1, 'files/', $this->sampleFilename, $width, $height ); @@ -581,7 +581,7 @@ class PreviewTest extends TestCase { /** * Creates the Max preview which will be used in the rest of the test * - * @return \OC\Preview + * @return Preview */ private function createMaxPreview() { $this->keepAspect = true; @@ -741,7 +741,7 @@ class PreviewTest extends TestCase { $userPath = '/' . self::TEST_PREVIEW_USER1 . '/'; } - return $userPath . \OC\Preview::THUMBNAILS_FOLDER . '/' . $fileId + return $userPath . Preview::THUMBNAILS_FOLDER . '/' . $fileId . '/' . $width . '-' . $height . $postfix . '.png'; } @@ -891,7 +891,7 @@ class PreviewTest extends TestCase { $originalHeight = 1050; $originalAspectRation = $originalWidth / $originalHeight; - $preview = new \OC\Preview( + $preview = new Preview( self::TEST_PREVIEW_USER1, 'files/', 'testimage.jpg', 150, 150 @@ -911,13 +911,13 @@ class PreviewTest extends TestCase { $originalHeight = 1050; $originalAspectRation = $originalWidth / $originalHeight; - $preview = new \OC\Preview( + $preview = new Preview( self::TEST_PREVIEW_USER1, 'files/', 'testimage.jpg', 150, 150 ); $preview->setKeepAspect(true); - $preview->setMode(\OC\Preview::MODE_COVER); + $preview->setMode(Preview::MODE_COVER); $image = $preview->getPreview(); $aspectRatio = $image->width() / $image->height(); @@ -929,7 +929,7 @@ class PreviewTest extends TestCase { public function testSetFileWithInfo() { $info = new FileInfo('/foo', null, '/foo', ['mimetype' => 'foo/bar'], null); - $preview = new \OC\Preview(); + $preview = new Preview(); $preview->setFile('/foo', $info); $this->assertEquals($info, $this->invokePrivate($preview, 'getFileInfo')); } @@ -945,7 +945,7 @@ class PreviewTest extends TestCase { \OC_Util::tearDownFS(); \OC_Util::setupFS($userId); - $preview = new \OC\Preview($userId, 'files'); + $preview = new Preview($userId, 'files'); $view = new View('/' . $userId . '/files'); $view->file_put_contents('test.png', file_get_contents($sourceFile)); $info = $view->getFileInfo('test.png'); |