wipfix/strict-types

7 files changed, 31 insertions, 29 deletions

diff --git a/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php b/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php
index a3151d0b040..e04de52490f 100644
--- a/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php
+++ b/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php
@@ -77,10 +77,10 @@ class ContactsStoreTest extends TestCase {
 			->with($this->equalTo(''), $this->equalTo(['FN', 'EMAIL']))
 			->willReturn([
 				[
-					'UID' => 123,
+					'UID' => '123',
 				],
 				[
-					'UID' => 567,
+					'UID' => '567',
 					'FN' => 'Darren Roner',
 					'EMAIL' => [
 						'darren@roner.au'
@@ -110,7 +110,7 @@ class ContactsStoreTest extends TestCase {
 					'UID' => 'user123',
 				],
 				[
-					'UID' => 567,
+					'UID' => '567',
 					'FN' => 'Darren Roner',
 					'EMAIL' => [
 						'darren@roner.au'
@@ -138,10 +138,10 @@ class ContactsStoreTest extends TestCase {
 			->with($this->equalTo(''), $this->equalTo(['FN', 'EMAIL']))
 			->willReturn([
 				[
-					'UID' => 123,
+					'UID' => '123',
 				],
 				[
-					'UID' => 567,
+					'UID' => '567',
 					'FN' => 'Darren Roner',
 					'EMAIL' => [
 						'darren@roner.au'
@@ -167,10 +167,10 @@ class ContactsStoreTest extends TestCase {
 			->with($this->equalTo(''), $this->equalTo(['FN', 'EMAIL']))
 			->willReturn([
 				[
-					'UID' => 123,
+					'UID' => '123',
 				],
 				[
-					'UID' => 567,
+					'UID' => '567',
 					'FN' => 'Darren Roner',
 					'EMAIL' => [
 						'darren@roner.au'
@@ -868,7 +868,7 @@ class ContactsStoreTest extends TestCase {
 			->with($this->equalTo('a567'), $this->equalTo(['UID']))
 			->willReturn([
 				[
-					'UID' => 123,
+					'UID' => '123',
 					'isLocalSystemBook' => false
 				],
 				[
@@ -899,7 +899,7 @@ class ContactsStoreTest extends TestCase {
 			->with($this->equalTo('darren@roner.au'), $this->equalTo(['EMAIL']))
 			->willReturn([
 				[
-					'UID' => 123,
+					'UID' => '123',
 					'isLocalSystemBook' => false
 				],
 				[
@@ -939,7 +939,7 @@ class ContactsStoreTest extends TestCase {
 			->with($this->equalTo('a567'), $this->equalTo(['UID']))
 			->willReturn([
 				[
-					'UID' => 123,
+					'UID' => '123',
 					'isLocalSystemBook' => false
 				],
 				[
diff --git a/tests/lib/Files/Mount/MountPointTest.php b/tests/lib/Files/Mount/MountPointTest.php
index eda61feb249..db04b1114fc 100644
--- a/tests/lib/Files/Mount/MountPointTest.php
+++ b/tests/lib/Files/Mount/MountPointTest.php
@@ -18,7 +18,7 @@ class MountPointTest extends \Test\TestCase {
 		$storage = $this->createMock(Storage::class);
 		$storage->expects($this->once())
 			->method('getId')
-			->willReturn(123);
+			->willReturn('123');
 
 		$loader = $this->createMock(StorageFactory::class);
 		$loader->expects($this->once())
@@ -34,7 +34,7 @@ class MountPointTest extends \Test\TestCase {
 		);
 
 		$this->assertEquals($storage, $mountPoint->getStorage());
-		$this->assertEquals(123, $mountPoint->getStorageId());
+		$this->assertEquals('123', $mountPoint->getStorageId());
 		$this->assertEquals('/mountpoint/', $mountPoint->getMountPoint());
 
 		$mountPoint->setMountPoint('another');
diff --git a/tests/lib/Files/ViewTest.php b/tests/lib/Files/ViewTest.php
index 0a029889fb1..4a47f081b90 100644
--- a/tests/lib/Files/ViewTest.php
+++ b/tests/lib/Files/ViewTest.php
@@ -1750,6 +1750,8 @@ class ViewTest extends \Test\TestCase {
 				ILockingProvider::LOCK_SHARED,
 				ILockingProvider::LOCK_EXCLUSIVE,
 				ILockingProvider::LOCK_SHARED,
+				null,
+				0,
 			],
 
 			// ---- delete hook ----
@@ -2515,7 +2517,7 @@ class ViewTest extends \Test\TestCase {
 			->willReturn($mountPoint);
 		$mount->expects($this->once())
 			->method('removeMount')
-			->willReturn('foo');
+			->willReturn(true);
 		$mount->expects($this->any())
 			->method('getInternalPath')
 			->willReturn('');
@@ -2548,7 +2550,7 @@ class ViewTest extends \Test\TestCase {
 
 		//Delete the mountpoint
 		$view = new View('/' . $this->user . '/files');
-		$this->assertEquals('foo', $view->rmdir('mount'));
+		$this->assertEquals(true, $view->rmdir('mount'));
 	}
 
 	public function mimeFilterProvider() {
diff --git a/tests/lib/Template/ResourceLocatorTest.php b/tests/lib/Template/ResourceLocatorTest.php
index 65d4c7938f9..3de1298ab3a 100644
--- a/tests/lib/Template/ResourceLocatorTest.php
+++ b/tests/lib/Template/ResourceLocatorTest.php
@@ -41,12 +41,12 @@ class ResourceLocatorTest extends \Test\TestCase {
 		$locator = $this->getResourceLocator('theme');
 		$locator->expects($this->once())
 			->method('doFind')
-			->with('foo');
+			->with('app/foo');
 		$locator->expects($this->once())
 			->method('doFindTheme')
-			->with('foo');
+			->with('app/foo');
 		/** @var \OC\Template\ResourceLocator $locator */
-		$locator->find(['foo']);
+		$locator->find(['app/foo']);
 	}
 
 	public function testFindNotFound(): void {
@@ -59,17 +59,17 @@ class ResourceLocatorTest extends \Test\TestCase {
 			['core' => 'map'], ['3rd' => 'party'], ['foo' => 'bar']);
 		$locator->expects($this->once())
 			->method('doFind')
-			->with('foo')
-			->will($this->throwException(new ResourceNotFoundException('foo', 'map')));
+			->with('app/foo')
+			->will($this->throwException(new ResourceNotFoundException('app/foo', 'map')));
 		$locator->expects($this->once())
 			->method('doFindTheme')
-			->with('foo')
-			->will($this->throwException(new ResourceNotFoundException('foo', 'map')));
+			->with('app/foo')
+			->will($this->throwException(new ResourceNotFoundException('app/foo', 'map')));
 		$this->logger->expects($this->exactly(2))
 			->method('debug')
-			->with($this->stringContains('map/foo'));
+			->with($this->stringContains('map/app/foo'));
 		/** @var \OC\Template\ResourceLocator $locator */
-		$locator->find(['foo']);
+		$locator->find(['app/foo']);
 	}
 
 	public function testAppendIfExist(): void {
diff --git a/tests/lib/TemplateFunctionsTest.php b/tests/lib/TemplateFunctionsTest.php
index a693a39c580..a3b2cfe1bb0 100644
--- a/tests/lib/TemplateFunctionsTest.php
+++ b/tests/lib/TemplateFunctionsTest.php
@@ -170,12 +170,12 @@ class TemplateFunctionsTest extends \Test\TestCase {
 	public function testRelativeDateYearsAgo(): void {
 		$currentTime = 1380703592;
 		$elementTime = $currentTime - 86400 * 365.25 * 2;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+		$result = (string)relative_modified_date((int)$elementTime, $currentTime, true);
 
 		$this->assertEquals('2 years ago', $result);
 
 		$elementTime = $currentTime - 86400 * 365.25 * 3;
-		$result = (string)relative_modified_date($elementTime, $currentTime, true);
+		$result = (string)relative_modified_date((int)$elementTime, $currentTime, true);
 
 		$this->assertEquals('3 years ago', $result);
 	}
@@ -264,12 +264,12 @@ class TemplateFunctionsTest extends \Test\TestCase {
 	public function testRelativeTimeYearsAgo(): void {
 		$currentTime = 1380703592;
 		$elementTime = $currentTime - 86400 * 365.25 * 2;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+		$result = (string)relative_modified_date((int)$elementTime, $currentTime, false);
 
 		$this->assertEquals('2 years ago', $result);
 
 		$elementTime = $currentTime - 86400 * 365.25 * 3;
-		$result = (string)relative_modified_date($elementTime, $currentTime, false);
+		$result = (string)relative_modified_date((int)$elementTime, $currentTime, false);
 
 		$this->assertEquals('3 years ago', $result);
 	}
diff --git a/tests/lib/User/ManagerTest.php b/tests/lib/User/ManagerTest.php
index 302cba4ea2b..bc6a41fb07c 100644
--- a/tests/lib/User/ManagerTest.php
+++ b/tests/lib/User/ManagerTest.php
@@ -154,7 +154,7 @@ class ManagerTest extends TestCase {
 		$backend->expects($this->once())
 			->method('checkPassword')
 			->with($this->equalTo('foo'), $this->equalTo('bar'))
-			->willReturn(true);
+			->willReturn('foo');
 
 		$backend->expects($this->any())
 			->method('implementsActions')
diff --git a/tests/lib/User/SessionTest.php b/tests/lib/User/SessionTest.php
index fad5d11565c..1702ef1fbec 100644
--- a/tests/lib/User/SessionTest.php
+++ b/tests/lib/User/SessionTest.php
@@ -970,7 +970,7 @@ class SessionTest extends \Test\TestCase {
 			->method('generateToken')
 			->with($sessionId, $uid, $loginName, $password, 'Firefox', IToken::TEMPORARY_TOKEN, IToken::REMEMBER);
 
-		$this->assertTrue($userSession->createSessionToken($request, $uid, $loginName, $password, true));
+		$this->assertTrue($userSession->createSessionToken($request, $uid, $loginName, $password, IToken::REMEMBER));
 	}
 
 	public function testCreateSessionTokenWithTokenPassword(): void {