aboutsummaryrefslogtreecommitdiffstats
path: root/tests/lib
diff options
context:
space:
mode:
Diffstat (limited to 'tests/lib')
-rw-r--r--tests/lib/archive.php30
-rw-r--r--tests/lib/archive/tar.php18
-rw-r--r--tests/lib/archive/zip.php18
-rw-r--r--tests/lib/cache.php16
-rw-r--r--tests/lib/cache/apc.php13
-rw-r--r--tests/lib/cache/xcache.php12
-rw-r--r--tests/lib/files/storage/storage.php32
-rw-r--r--tests/lib/filesystem.php9
-rw-r--r--tests/lib/share/share.php4
-rw-r--r--tests/lib/streamwrappers.php4
10 files changed, 69 insertions, 87 deletions
diff --git a/tests/lib/archive.php b/tests/lib/archive.php
index 565c314cb8c..04ae722aea7 100644
--- a/tests/lib/archive.php
+++ b/tests/lib/archive.php
@@ -22,46 +22,46 @@ abstract class Test_Archive extends UnitTestCase {
* @return OC_Archive
*/
abstract protected function getNew();
-
+
public function testGetFiles() {
$this->instance=$this->getExisting();
$allFiles=$this->instance->getFiles();
$expected=array('lorem.txt','logo-wide.png','dir/','dir/lorem.txt');
$this->assertEqual(4,count($allFiles),'only found '.count($allFiles).' out of 4 expected files');
foreach($expected as $file) {
- $this->assertNotIdentical(false,array_search($file,$allFiles),'cant find '.$file.' in archive');
+ $this->assertContains($file, $allFiles, 'cant find '. $file . ' in archive');
$this->assertTrue($this->instance->fileExists($file),'file '.$file.' does not exist in archive');
}
$this->assertFalse($this->instance->fileExists('non/existing/file'));
-
+
$rootContent=$this->instance->getFolder('');
$expected=array('lorem.txt','logo-wide.png','dir/');
$this->assertEqual(3,count($rootContent));
foreach($expected as $file) {
- $this->assertNotIdentical(false,array_search($file,$rootContent),'cant find '.$file.' in archive');
+ $this->assertContains($file, $rootContent, 'cant find '. $file . ' in archive');
}
$dirContent=$this->instance->getFolder('dir/');
$expected=array('lorem.txt');
$this->assertEqual(1,count($dirContent));
foreach($expected as $file) {
- $this->assertNotIdentical(false,array_search($file,$dirContent),'cant find '.$file.' in archive');
+ $this->assertContains($file, $dirContent, 'cant find '. $file . ' in archive');
}
}
-
+
public function testContent() {
$this->instance=$this->getExisting();
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
+ $dir=OC::$SERVERROOT.'/tests/data';
$textFile=$dir.'/lorem.txt';
$this->assertEqual(file_get_contents($textFile),$this->instance->getFile('lorem.txt'));
-
+
$tmpFile=OCP\Files::tmpFile('.txt');
$this->instance->extractFile('lorem.txt',$tmpFile);
$this->assertEqual(file_get_contents($textFile),file_get_contents($tmpFile));
}
public function testWrite() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
+ $dir=OC::$SERVERROOT.'/tests/data';
$textFile=$dir.'/lorem.txt';
$this->instance=$this->getNew();
$this->assertEqual(0,count($this->instance->getFiles()));
@@ -69,14 +69,14 @@ abstract class Test_Archive extends UnitTestCase {
$this->assertEqual(1,count($this->instance->getFiles()));
$this->assertTrue($this->instance->fileExists('lorem.txt'));
$this->assertFalse($this->instance->fileExists('lorem.txt/'));
-
+
$this->assertEqual(file_get_contents($textFile),$this->instance->getFile('lorem.txt'));
$this->instance->addFile('lorem.txt','foobar');
$this->assertEqual('foobar',$this->instance->getFile('lorem.txt'));
}
public function testReadStream() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
+ $dir=OC::$SERVERROOT.'/tests/data';
$this->instance=$this->getExisting();
$fh=$this->instance->getStream('lorem.txt','r');
$this->assertTrue($fh);
@@ -85,7 +85,7 @@ abstract class Test_Archive extends UnitTestCase {
$this->assertEqual(file_get_contents($dir.'/lorem.txt'),$content);
}
public function testWriteStream() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
+ $dir=OC::$SERVERROOT.'/tests/data';
$this->instance=$this->getNew();
$fh=$this->instance->getStream('lorem.txt','w');
$source=fopen($dir.'/lorem.txt','r');
@@ -107,7 +107,7 @@ abstract class Test_Archive extends UnitTestCase {
$this->assertFalse($this->instance->fileExists('/test/'));
}
public function testExtract() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
+ $dir=OC::$SERVERROOT.'/tests/data';
$this->instance=$this->getExisting();
$tmpDir=OCP\Files::tmpFolder();
$this->instance->extract($tmpDir);
@@ -118,7 +118,7 @@ abstract class Test_Archive extends UnitTestCase {
OCP\Files::rmdirr($tmpDir);
}
public function testMoveRemove() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
+ $dir=OC::$SERVERROOT.'/tests/data';
$textFile=$dir.'/lorem.txt';
$this->instance=$this->getNew();
$this->instance->addFile('lorem.txt',$textFile);
@@ -131,7 +131,7 @@ abstract class Test_Archive extends UnitTestCase {
$this->assertFalse($this->instance->fileExists('target.txt'));
}
public function testRecursive() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
+ $dir=OC::$SERVERROOT.'/tests/data';
$this->instance=$this->getNew();
$this->instance->addRecursive('/dir',$dir);
$this->assertTrue($this->instance->fileExists('/dir/lorem.txt'));
diff --git a/tests/lib/archive/tar.php b/tests/lib/archive/tar.php
index 2595b7cb195..51de004813a 100644
--- a/tests/lib/archive/tar.php
+++ b/tests/lib/archive/tar.php
@@ -8,17 +8,13 @@
require_once 'archive.php';
-if(is_dir(OC::$SERVERROOT.'/apps/files_archive/tests/data')) {
- class Test_Archive_TAR extends Test_Archive{
- protected function getExisting() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
- return new OC_Archive_TAR($dir.'/data.tar.gz');
- }
+class Test_Archive_TAR extends Test_Archive {
+ protected function getExisting() {
+ $dir = OC::$SERVERROOT . '/tests/data';
+ return new OC_Archive_TAR($dir . '/data.tar.gz');
+ }
- protected function getNew() {
- return new OC_Archive_TAR(OCP\Files::tmpFile('.tar.gz'));
- }
+ protected function getNew() {
+ return new OC_Archive_TAR(OCP\Files::tmpFile('.tar.gz'));
}
-}else{
- abstract class Test_Archive_TAR extends Test_Archive{}
}
diff --git a/tests/lib/archive/zip.php b/tests/lib/archive/zip.php
index a7682e34180..adddf81ee1b 100644
--- a/tests/lib/archive/zip.php
+++ b/tests/lib/archive/zip.php
@@ -8,17 +8,13 @@
require_once 'archive.php';
-if(is_dir(OC::$SERVERROOT.'/apps/files_archive/tests/data')) {
- class Test_Archive_ZIP extends Test_Archive{
- protected function getExisting() {
- $dir=OC::$SERVERROOT.'/apps/files_archive/tests/data';
- return new OC_Archive_ZIP($dir.'/data.zip');
- }
+class Test_Archive_ZIP extends Test_Archive {
+ protected function getExisting() {
+ $dir = OC::$SERVERROOT . '/tests/data';
+ return new OC_Archive_ZIP($dir . '/data.zip');
+ }
- protected function getNew() {
- return new OC_Archive_ZIP(OCP\Files::tmpFile('.zip'));
- }
+ protected function getNew() {
+ return new OC_Archive_ZIP(OCP\Files::tmpFile('.zip'));
}
-}else{
- abstract class Test_Archive_ZIP extends Test_Archive{}
}
diff --git a/tests/lib/cache.php b/tests/lib/cache.php
index 9ada0accc21..08653d4a310 100644
--- a/tests/lib/cache.php
+++ b/tests/lib/cache.php
@@ -13,7 +13,9 @@ abstract class Test_Cache extends UnitTestCase {
protected $instance;
public function tearDown() {
- $this->instance->clear();
+ if($this->instance){
+ $this->instance->clear();
+ }
}
function testSimple() {
@@ -64,16 +66,4 @@ abstract class Test_Cache extends UnitTestCase {
$this->assertFalse($this->instance->hasKey('2_value1'));
$this->assertFalse($this->instance->hasKey('3_value1'));
}
-
- function testTTL() {
- $value='foobar';
- $this->instance->set('value1',$value,1);
- $value2='foobar';
- $this->instance->set('value2',$value2);
- sleep(2);
- $this->assertFalse($this->instance->hasKey('value1'));
- $this->assertNull($this->instance->get('value1'));
- $this->assertTrue($this->instance->hasKey('value2'));
- $this->assertEqual($value2,$this->instance->get('value2'));
- }
}
diff --git a/tests/lib/cache/apc.php b/tests/lib/cache/apc.php
index 34ea968cd54..0e0edcf58f3 100644
--- a/tests/lib/cache/apc.php
+++ b/tests/lib/cache/apc.php
@@ -21,16 +21,11 @@
*/
class Test_Cache_APC extends Test_Cache {
- function skip() {
- $this->skipUnless(function_exists('apc_store'));
- }
-
public function setUp() {
+ if(!function_exists('apc_store')){
+ $this->markTestSkipped('The apc extension is not available.');
+ return;
+ }
$this->instance=new OC_Cache_APC();
}
-
- function testTTL() {
- // ttl doesn't work correctly in the same request
- // see https://bugs.php.net/bug.php?id=58084
- }
}
diff --git a/tests/lib/cache/xcache.php b/tests/lib/cache/xcache.php
index 85cc2d8b3c6..c081036a31f 100644
--- a/tests/lib/cache/xcache.php
+++ b/tests/lib/cache/xcache.php
@@ -21,15 +21,11 @@
*/
class Test_Cache_XCache extends Test_Cache {
- function skip() {
- $this->skipUnless(function_exists('xcache_get'));
- }
-
public function setUp() {
+ if(!function_exists('xcache_get')){
+ $this->markTestSkipped('The xcache extension is not available.');
+ return;
+ }
$this->instance=new OC_Cache_XCache();
}
-
- function testTTL() {
- // ttl doesn't work correctly in the same request
- }
}
diff --git a/tests/lib/files/storage/storage.php b/tests/lib/files/storage/storage.php
index 9c8581b259a..8ee08fe88ce 100644
--- a/tests/lib/files/storage/storage.php
+++ b/tests/lib/files/storage/storage.php
@@ -178,22 +178,24 @@ abstract class Storage extends \UnitTestCase {
$this->assertEqual($stat['ctime'],$cTime);
$mtimeStart=time();
- $this->instance->touch('/lorem.txt');
+ $supportsTouch = $this->instance->touch('/lorem.txt');
$mtimeEnd=time();
- $originalCTime=$cTime;
- $cTime=$this->instance->filectime('/lorem.txt');
- $mTime=$this->instance->filemtime('/lorem.txt');
- $this->assertTrue(($mtimeStart-1)<=$mTime);
- $this->assertTrue($mTime<=($mtimeEnd+1));
- $this->assertEqual($cTime,$originalCTime);
-
- $this->assertTrue($this->instance->hasUpdated('/lorem.txt',$mtimeStart-1));
-
- if($this->instance->touch('/lorem.txt',100)!==false) {
+ if($supportsTouch !== false){
+ $originalCTime=$cTime;
+ $cTime=$this->instance->filectime('/lorem.txt');
$mTime=$this->instance->filemtime('/lorem.txt');
- $this->assertEqual($mTime,100);
+ $this->assertTrue(($mtimeStart-1)<=$mTime);
+ $this->assertTrue($mTime<=($mtimeEnd+1));
+ $this->assertEqual($cTime,$originalCTime);
+
+ $this->assertTrue($this->instance->hasUpdated('/lorem.txt',$mtimeStart-1));
+
+ if($this->instance->touch('/lorem.txt',100)!==false) {
+ $mTime=$this->instance->filemtime('/lorem.txt');
+ $this->assertEqual($mTime,100);
+ }
}
-
+
$mtimeStart=time();
$fh=$this->instance->fopen('/lorem.txt','a');
fwrite($fh,' ');
@@ -217,7 +219,7 @@ abstract class Storage extends \UnitTestCase {
$this->instance->file_put_contents('/logo-wide.svg',file_get_contents($svgFile,'r'));
$result=$this->instance->search('logo');
$this->assertEqual(2,count($result));
- $this->assertNotIdentical(false,array_search('/logo-wide.svg',$result));
- $this->assertNotIdentical(false,array_search('/logo-wide.png',$result));
+ $this->assertContains('/logo-wide.svg',$result);
+ $this->assertContains('/logo-wide.png',$result);
}
}
diff --git a/tests/lib/filesystem.php b/tests/lib/filesystem.php
index b936608b1cf..8fc3ce641da 100644
--- a/tests/lib/filesystem.php
+++ b/tests/lib/filesystem.php
@@ -24,7 +24,7 @@ class Test_Filesystem extends UnitTestCase {
/**
* @var array tmpDirs
*/
- private $tmpDirs;
+ private $tmpDirs=array();
/**
* @return array
@@ -73,7 +73,12 @@ class Test_Filesystem extends UnitTestCase {
}
public function testHooks() {
- $user = OC_User::getUser();
+ if(OC_Filesystem::getView()){
+ $user = OC_User::getUser();
+ }else{
+ $user=uniqid();
+ OC_Filesystem::init('/'.$user.'/files');
+ }
OC_Hook::clear('OC_Filesystem');
OC_Hook::connect('OC_Filesystem', 'post_write', $this, 'dummyHook');
diff --git a/tests/lib/share/share.php b/tests/lib/share/share.php
index b2fecdc8bf7..df91a27c6e1 100644
--- a/tests/lib/share/share.php
+++ b/tests/lib/share/share.php
@@ -61,8 +61,10 @@ class Test_Share extends UnitTestCase {
$query->execute(array('test'));
}
+ /**
+ * @expectedException Exception
+ */
public function testShareInvalidShareType() {
- $this->expectException(new Exception('Share type foobar is not valid for test.txt'));
OCP\Share::shareItem('test', 'test.txt', 'foobar', $this->user2, OCP\Share::PERMISSION_READ);
}
diff --git a/tests/lib/streamwrappers.php b/tests/lib/streamwrappers.php
index 5d6fe8da826..46838ff9754 100644
--- a/tests/lib/streamwrappers.php
+++ b/tests/lib/streamwrappers.php
@@ -28,7 +28,7 @@ class Test_StreamWrappers extends UnitTestCase {
$result=array();
while($file=readdir($dh)) {
$result[]=$file;
- $this->assertNotIdentical(false,array_search($file,$items));
+ $this->assertContains($file, $items);
}
$this->assertEqual(count($items),count($result));
}
@@ -75,4 +75,4 @@ class Test_StreamWrappers extends UnitTestCase {
public static function closeCallBack($path) {
throw new Exception($path);
}
-} \ No newline at end of file
+}