update tests

This commit is contained in:
Robin Appelman 2015-04-09 14:46:25 +02:00
parent 6a59502759
commit cbcee34eb0
2 changed files with 45 additions and 26 deletions

View File

@ -71,7 +71,7 @@ class File extends Node implements IFile {
* different object on a subsequent GET you are strongly recommended to not * different object on a subsequent GET you are strongly recommended to not
* return an ETag, and just return null. * return an ETag, and just return null.
* *
* @param resource $data * @param resource|string $data
* *
* @throws Forbidden * @throws Forbidden
* @throws UnsupportedMediaType * @throws UnsupportedMediaType
@ -82,13 +82,18 @@ class File extends Node implements IFile {
* @return string|null * @return string|null
*/ */
public function put($data) { public function put($data) {
if (is_string($data)) {
$stream = fopen('php://temp', 'r+');
fwrite($stream, $data);
$data = $stream;
};
try { try {
$exists = $this->fileView->file_exists($this->path); $exists = $this->fileView->file_exists($this->path);
if ($this->info && $exists && !$this->info->isUpdateable()) { if ($this->info && $exists && !$this->info->isUpdateable()) {
throw new Forbidden(); throw new Forbidden();
} }
} catch (StorageNotAvailableException $e) { } catch (StorageNotAvailableException $e) {
throw new ServiceUnavailable("File is not updatable: ".$e->getMessage()); throw new ServiceUnavailable("File is not updatable: " . $e->getMessage());
} }
// verify path of the target // verify path of the target
@ -155,7 +160,7 @@ class File extends Node implements IFile {
// returning 503 will allow retry of the operation at a later point in time // returning 503 will allow retry of the operation at a later point in time
throw new ServiceUnavailable("Encryption not ready: " . $e->getMessage()); throw new ServiceUnavailable("Encryption not ready: " . $e->getMessage());
} catch (StorageNotAvailableException $e) { } catch (StorageNotAvailableException $e) {
throw new ServiceUnavailable("Failed to write file contents: ".$e->getMessage()); throw new ServiceUnavailable("Failed to write file contents: " . $e->getMessage());
} }
try { try {
@ -178,30 +183,33 @@ class File extends Node implements IFile {
// since we skipped the view we need to scan and emit the hooks ourselves // since we skipped the view we need to scan and emit the hooks ourselves
$storage->getScanner()->scanFile($internalPath); $storage->getScanner()->scanFile($internalPath);
$hookPath = \OC\Files\Filesystem::getView()->getRelativePath($this->fileView->getAbsolutePath($this->path)); $view = \OC\Files\Filesystem::getView();
if (!$exists) { if ($view) {
\OC_Hook::emit(\OC\Files\Filesystem::CLASSNAME, \OC\Files\Filesystem::signal_post_create, array( $hookPath = $view->getRelativePath($this->fileView->getAbsolutePath($this->path));
\OC\Files\Filesystem::signal_param_path => $hookPath if (!$exists) {
)); \OC_Hook::emit(\OC\Files\Filesystem::CLASSNAME, \OC\Files\Filesystem::signal_post_create, array(
} else { \OC\Files\Filesystem::signal_param_path => $hookPath
\OC_Hook::emit(\OC\Files\Filesystem::CLASSNAME, \OC\Files\Filesystem::signal_post_update, array( ));
} else {
\OC_Hook::emit(\OC\Files\Filesystem::CLASSNAME, \OC\Files\Filesystem::signal_post_update, array(
\OC\Files\Filesystem::signal_param_path => $hookPath
));
}
\OC_Hook::emit(\OC\Files\Filesystem::CLASSNAME, \OC\Files\Filesystem::signal_post_write, array(
\OC\Files\Filesystem::signal_param_path => $hookPath \OC\Files\Filesystem::signal_param_path => $hookPath
)); ));
} }
\OC_Hook::emit(\OC\Files\Filesystem::CLASSNAME, \OC\Files\Filesystem::signal_post_write, array(
\OC\Files\Filesystem::signal_param_path => $hookPath
));
// allow sync clients to send the mtime along in a header // allow sync clients to send the mtime along in a header
$request = \OC::$server->getRequest(); $request = \OC::$server->getRequest();
if (isset($request->server['HTTP_X_OC_MTIME'])) { if (isset($request->server['HTTP_X_OC_MTIME'])) {
if($this->fileView->touch($this->path, $request->server['HTTP_X_OC_MTIME'])) { if ($this->fileView->touch($this->path, $request->server['HTTP_X_OC_MTIME'])) {
header('X-OC-MTime: accepted'); header('X-OC-MTime: accepted');
} }
} }
$this->refreshInfo(); $this->refreshInfo();
} catch (StorageNotAvailableException $e) { } catch (StorageNotAvailableException $e) {
throw new ServiceUnavailable("Failed to check file size: ".$e->getMessage()); throw new ServiceUnavailable("Failed to check file size: " . $e->getMessage());
} }
return '"' . $this->info->getEtag() . '"'; return '"' . $this->info->getEtag() . '"';
@ -209,6 +217,7 @@ class File extends Node implements IFile {
/** /**
* Returns the data * Returns the data
*
* @return string|resource * @return string|resource
* @throws Forbidden * @throws Forbidden
* @throws ServiceUnavailable * @throws ServiceUnavailable
@ -222,12 +231,13 @@ class File extends Node implements IFile {
// returning 503 will allow retry of the operation at a later point in time // returning 503 will allow retry of the operation at a later point in time
throw new ServiceUnavailable("Encryption not ready: " . $e->getMessage()); throw new ServiceUnavailable("Encryption not ready: " . $e->getMessage());
} catch (StorageNotAvailableException $e) { } catch (StorageNotAvailableException $e) {
throw new ServiceUnavailable("Failed to open file: ".$e->getMessage()); throw new ServiceUnavailable("Failed to open file: " . $e->getMessage());
} }
} }
/** /**
* Delete the current file * Delete the current file
*
* @throws Forbidden * @throws Forbidden
* @throws ServiceUnavailable * @throws ServiceUnavailable
*/ */
@ -242,7 +252,7 @@ class File extends Node implements IFile {
throw new Forbidden(); throw new Forbidden();
} }
} catch (StorageNotAvailableException $e) { } catch (StorageNotAvailableException $e) {
throw new ServiceUnavailable("Failed to unlink: ".$e->getMessage()); throw new ServiceUnavailable("Failed to unlink: " . $e->getMessage());
} }
} }
@ -257,7 +267,7 @@ class File extends Node implements IFile {
$mimeType = $this->info->getMimetype(); $mimeType = $this->info->getMimetype();
// PROPFIND needs to return the correct mime type, for consistency with the web UI // PROPFIND needs to return the correct mime type, for consistency with the web UI
if (isset($_SERVER['REQUEST_METHOD']) && $_SERVER['REQUEST_METHOD'] === 'PROPFIND' ) { if (isset($_SERVER['REQUEST_METHOD']) && $_SERVER['REQUEST_METHOD'] === 'PROPFIND') {
return $mimeType; return $mimeType;
} }
return \OC_Helper::getSecureMimeType($mimeType); return \OC_Helper::getSecureMimeType($mimeType);
@ -310,7 +320,7 @@ class File extends Node implements IFile {
} }
if ($chunk_handler->isComplete()) { if ($chunk_handler->isComplete()) {
list($storage, ) = $this->fileView->resolvePath($path); list($storage,) = $this->fileView->resolvePath($path);
$needsPartFile = $this->needsPartFile($storage); $needsPartFile = $this->needsPartFile($storage);
try { try {
@ -339,7 +349,7 @@ class File extends Node implements IFile {
// allow sync clients to send the mtime along in a header // allow sync clients to send the mtime along in a header
$request = \OC::$server->getRequest(); $request = \OC::$server->getRequest();
if (isset($request->server['HTTP_X_OC_MTIME'])) { if (isset($request->server['HTTP_X_OC_MTIME'])) {
if($this->fileView->touch($targetPath, $request->server['HTTP_X_OC_MTIME'])) { if ($this->fileView->touch($targetPath, $request->server['HTTP_X_OC_MTIME'])) {
header('X-OC-MTime: accepted'); header('X-OC-MTime: accepted');
} }
} }
@ -347,7 +357,7 @@ class File extends Node implements IFile {
$info = $this->fileView->getFileInfo($targetPath); $info = $this->fileView->getFileInfo($targetPath);
return $info->getEtag(); return $info->getEtag();
} catch (StorageNotAvailableException $e) { } catch (StorageNotAvailableException $e) {
throw new ServiceUnavailable("Failed to put file: ".$e->getMessage()); throw new ServiceUnavailable("Failed to put file: " . $e->getMessage());
} }
} }
@ -358,6 +368,7 @@ class File extends Node implements IFile {
* Returns whether a part file is needed for the given storage * Returns whether a part file is needed for the given storage
* or whether the file can be assembled/uploaded directly on the * or whether the file can be assembled/uploaded directly on the
* target storage. * target storage.
*
* @param \OCP\Files\Storage $storage * @param \OCP\Files\Storage $storage
* @return bool true if the storage needs part file handling * @return bool true if the storage needs part file handling
*/ */
@ -365,6 +376,6 @@ class File extends Node implements IFile {
// TODO: in the future use ChunkHandler provided by storage // TODO: in the future use ChunkHandler provided by storage
// and/or add method on Storage called "needsPartFile()" // and/or add method on Storage called "needsPartFile()"
return !$storage->instanceOfStorage('OCA\Files_Sharing\External\Storage') && return !$storage->instanceOfStorage('OCA\Files_Sharing\External\Storage') &&
!$storage->instanceOfStorage('OC\Files\Storage\OwnCloud'); !$storage->instanceOfStorage('OC\Files\Storage\OwnCloud');
} }
} }

View File

@ -15,9 +15,13 @@ class File extends \Test\TestCase {
*/ */
public function testSimplePutFails() { public function testSimplePutFails() {
// setup // setup
$view = $this->getMock('\OC\Files\View', array('file_put_contents', 'getRelativePath'), array()); $storage = $this->getMock('\OC\Files\Storage\Local', ['fopen'], [['datadir' => \OC::$server->getTempManager()->getTemporaryFolder()]]);
$view = $this->getMock('\OC\Files\View', array('file_put_contents', 'getRelativePath', 'resolvePath'), array());
$view->expects($this->any()) $view->expects($this->any())
->method('file_put_contents') ->method('resolvePath')
->will($this->returnValue(array($storage, '')));
$storage->expects($this->once())
->method('fopen')
->will($this->returnValue(false)); ->will($this->returnValue(false));
$view->expects($this->any()) $view->expects($this->any())
@ -36,8 +40,9 @@ class File extends \Test\TestCase {
public function testPutSingleFileShare() { public function testPutSingleFileShare() {
// setup // setup
$storage = $this->getMock('\OCP\Files\Storage'); $stream = fopen('php://temp', 'w+');
$view = $this->getMock('\OC\Files\View', array('file_put_contents', 'getRelativePath'), array()); $storage = $this->getMock('\OC\Files\Storage\Local', ['fopen'], [['datadir' => \OC::$server->getTempManager()->getTemporaryFolder()]]);
$view = $this->getMock('\OC\Files\View', array('file_put_contents', 'getRelativePath', 'resolvePath'), array());
$view->expects($this->any()) $view->expects($this->any())
->method('resolvePath') ->method('resolvePath')
->with('') ->with('')
@ -49,6 +54,9 @@ class File extends \Test\TestCase {
->method('file_put_contents') ->method('file_put_contents')
->with('') ->with('')
->will($this->returnValue(true)); ->will($this->returnValue(true));
$storage->expects($this->once())
->method('fopen')
->will($this->returnValue($stream));
$info = new \OC\Files\FileInfo('/foo.txt', null, null, array( $info = new \OC\Files\FileInfo('/foo.txt', null, null, array(
'permissions' => \OCP\Constants::PERMISSION_ALL 'permissions' => \OCP\Constants::PERMISSION_ALL