nextcloud / server

apps/dav/lib/CardDAV/AddressBookImpl.php

Indentation +318 added lines, -318 removed lines

@@ -18,322 +18,322 @@
 
 class AddressBookImpl implements IAddressBookEnabled {
 
-	/**
-	 * AddressBookImpl constructor.
-	 *
-	 * @param AddressBook $addressBook
-	 * @param array $addressBookInfo
-	 * @param CardDavBackend $backend
-	 * @param IUrlGenerator $urlGenerator
-	 */
-	public function __construct(
-		private AddressBook $addressBook,
-		private array $addressBookInfo,
-		private CardDavBackend $backend,
-		private IURLGenerator $urlGenerator,
-		private PropertyMapper $propertyMapper,
-		private ?string $userId,
-	) {
-	}
-
-	/**
-	 * @return string defining the technical unique key
-	 * @since 5.0.0
-	 */
-	public function getKey() {
-		return $this->addressBookInfo['id'];
-	}
-
-	/**
-	 * @return string defining the unique uri
-	 * @since 16.0.0
-	 */
-	public function getUri(): string {
-		return $this->addressBookInfo['uri'];
-	}
-
-	/**
-	 * In comparison to getKey() this function returns a human readable (maybe translated) name
-	 *
-	 * @return mixed
-	 * @since 5.0.0
-	 */
-	public function getDisplayName() {
-		return $this->addressBookInfo['{DAV:}displayname'];
-	}
-
-	/**
-	 * @param string $pattern which should match within the $searchProperties
-	 * @param array $searchProperties defines the properties within the query pattern should match
-	 * @param array $options Options to define the output format and search behavior
-	 *                       - 'types' boolean (since 15.0.0) If set to true, fields that come with a TYPE property will be an array
-	 *                       example: ['id' => 5, 'FN' => 'Thomas Tanghus', 'EMAIL' => ['type => 'HOME', 'value' => 'g@h.i']]
-	 *                       - 'escape_like_param' - If set to false wildcards _ and % are not escaped
-	 *                       - 'limit' - Set a numeric limit for the search results
-	 *                       - 'offset' - Set the offset for the limited search results
-	 *                       - 'wildcard' - Whether the search should use wildcards
-	 * @psalm-param array{types?: bool, escape_like_param?: bool, limit?: int, offset?: int, wildcard?: bool} $options
-	 * @return array an array of contacts which are arrays of key-value-pairs
-	 *               example result:
-	 *               [
-	 *               ['id' => 0, 'FN' => 'Thomas Müller', 'EMAIL' => 'a@b.c', 'GEO' => '37.386013;-122.082932'],
-	 *               ['id' => 5, 'FN' => 'Thomas Tanghus', 'EMAIL' => ['d@e.f', 'g@h.i']]
-	 *               ]
-	 * @since 5.0.0
-	 */
-	public function search($pattern, $searchProperties, $options) {
-		$results = $this->backend->search($this->getKey(), $pattern, $searchProperties, $options);
-
-		$withTypes = \array_key_exists('types', $options) && $options['types'] === true;
-
-		$vCards = [];
-		foreach ($results as $result) {
-			$vCards[] = $this->vCard2Array($result['uri'], $this->readCard($result['carddata']), $withTypes);
-		}
-
-		return $vCards;
-	}
-
-	/**
-	 * @param array $properties this array if key-value-pairs defines a contact
-	 * @return array an array representing the contact just created or updated
-	 * @since 5.0.0
-	 */
-	public function createOrUpdate($properties) {
-		$update = false;
-		if (!isset($properties['URI'])) { // create a new contact
-			$uid = $this->createUid();
-			$uri = $uid . '.vcf';
-			$vCard = $this->createEmptyVCard($uid);
-		} else { // update existing contact
-			$uri = $properties['URI'];
-			$vCardData = $this->backend->getCard($this->getKey(), $uri);
-			$vCard = $this->readCard($vCardData['carddata']);
-			$update = true;
-		}
-
-		foreach ($properties as $key => $value) {
-			if (is_array($value)) {
-				$vCard->remove($key);
-				foreach ($value as $entry) {
-					if (is_string($entry)) {
-						$property = $vCard->createProperty($key, $entry);
-					} else {
-						if (($key === 'ADR' || $key === 'PHOTO') && is_string($entry['value'])) {
-							$entry['value'] = stripslashes($entry['value']);
-							$entry['value'] = explode(';', $entry['value']);
-						}
-						$property = $vCard->createProperty($key, $entry['value']);
-						if (isset($entry['type'])) {
-							$property->add('TYPE', $entry['type']);
-						}
-					}
-					$vCard->add($property);
-				}
-			} elseif ($key !== 'URI') {
-				$vCard->$key = $vCard->createProperty($key, $value);
-			}
-		}
-
-		if ($update) {
-			$this->backend->updateCard($this->getKey(), $uri, $vCard->serialize());
-		} else {
-			$this->backend->createCard($this->getKey(), $uri, $vCard->serialize());
-		}
-
-		return $this->vCard2Array($uri, $vCard);
-	}
-
-	/**
-	 * @return mixed
-	 * @since 5.0.0
-	 */
-	public function getPermissions() {
-		$permissions = $this->addressBook->getACL();
-		$result = 0;
-		foreach ($permissions as $permission) {
-			if ($this->addressBookInfo['principaluri'] !== $permission['principal']) {
-				continue;
-			}
-
-			switch ($permission['privilege']) {
-				case '{DAV:}read':
-					$result |= Constants::PERMISSION_READ;
-					break;
-				case '{DAV:}write':
-					$result |= Constants::PERMISSION_CREATE;
-					$result |= Constants::PERMISSION_UPDATE;
-					break;
-				case '{DAV:}all':
-					$result |= Constants::PERMISSION_ALL;
-					break;
-			}
-		}
-
-		return $result;
-	}
-
-	/**
-	 * @param int $id the unique identifier to a contact
-	 * @return bool successful or not
-	 * @since 5.0.0
-	 */
-	public function delete($id) {
-		$uri = $this->backend->getCardUri($id);
-		return $this->backend->deleteCard($this->addressBookInfo['id'], $uri);
-	}
-
-	/**
-	 * read vCard data into a vCard object
-	 *
-	 * @param string $cardData
-	 * @return VCard
-	 */
-	protected function readCard($cardData) {
-		return  Reader::read($cardData);
-	}
-
-	/**
-	 * create UID for contact
-	 *
-	 * @return string
-	 */
-	protected function createUid() {
-		do {
-			$uid = $this->getUid();
-			$contact = $this->backend->getContact($this->getKey(), $uid . '.vcf');
-		} while (!empty($contact));
-
-		return $uid;
-	}
-
-	/**
-	 * getUid is only there for testing, use createUid instead
-	 */
-	protected function getUid() {
-		return UUIDUtil::getUUID();
-	}
-
-	/**
-	 * create empty vcard
-	 *
-	 * @param string $uid
-	 * @return VCard
-	 */
-	protected function createEmptyVCard($uid) {
-		$vCard = new VCard();
-		$vCard->UID = $uid;
-		return $vCard;
-	}
-
-	/**
-	 * create array with all vCard properties
-	 *
-	 * @param string $uri
-	 * @param VCard $vCard
-	 * @param boolean $withTypes (optional) return the values as arrays of value/type pairs
-	 * @return array
-	 */
-	protected function vCard2Array($uri, VCard $vCard, $withTypes = false) {
-		$result = [
-			'URI' => $uri,
-		];
-
-		foreach ($vCard->children() as $property) {
-			if ($property->name === 'PHOTO' && in_array($property->getValueType(), ['BINARY', 'URI'])) {
-				$url = $this->urlGenerator->getAbsoluteURL(
-					$this->urlGenerator->linkTo('', 'remote.php') . '/dav/');
-				$url .= implode('/', [
-					'addressbooks',
-					substr($this->addressBookInfo['principaluri'], 11), //cut off 'principals/'
-					$this->addressBookInfo['uri'],
-					$uri
-				]) . '?photo';
-
-				$result['PHOTO'] = 'VALUE=uri:' . $url;
-			} elseif (in_array($property->name, ['URL', 'GEO', 'CLOUD', 'ADR', 'EMAIL', 'IMPP', 'TEL', 'X-SOCIALPROFILE', 'RELATED', 'LANG', 'X-ADDRESSBOOKSERVER-MEMBER'])) {
-				if (!isset($result[$property->name])) {
-					$result[$property->name] = [];
-				}
-
-				$type = $this->getTypeFromProperty($property);
-				if ($withTypes) {
-					$result[$property->name][] = [
-						'type' => $type,
-						'value' => $property->getValue()
-					];
-				} else {
-					$result[$property->name][] = $property->getValue();
-				}
-			} else {
-				$result[$property->name] = $property->getValue();
-			}
-		}
-
-		if ($this->isSystemAddressBook()) {
-			$result['isLocalSystemBook'] = true;
-		}
-		return $result;
-	}
-
-	/**
-	 * Get the type of the current property
-	 *
-	 * @param Property $property
-	 * @return null|string
-	 */
-	protected function getTypeFromProperty(Property $property) {
-		$parameters = $property->parameters();
-		// Type is the social network, when it's empty we don't need this.
-		if (isset($parameters['TYPE'])) {
-			/** @var \Sabre\VObject\Parameter $type */
-			$type = $parameters['TYPE'];
-			return $type->getValue();
-		}
-
-		return null;
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isShared(): bool {
-		if (!isset($this->addressBookInfo['{http://owncloud.org/ns}owner-principal'])) {
-			return false;
-		}
-
-		return $this->addressBookInfo['principaluri']
-			!== $this->addressBookInfo['{http://owncloud.org/ns}owner-principal'];
-	}
-
-	/**
-	 * @inheritDoc
-	 */
-	public function isSystemAddressBook(): bool {
-		return $this->addressBookInfo['principaluri'] === 'principals/system/system' && (
-			$this->addressBookInfo['uri'] === 'system'
-			|| $this->addressBookInfo['{DAV:}displayname'] === $this->urlGenerator->getBaseUrl()
-		);
-	}
-
-	public function isEnabled(): bool {
-		if (!$this->userId) {
-			return true;
-		}
-
-		if ($this->isSystemAddressBook()) {
-			$user = $this->userId ;
-			$uri = 'z-server-generated--system';
-		} else {
-			$user = str_replace('principals/users/', '', $this->addressBookInfo['principaluri']);
-			$uri = $this->addressBookInfo['uri'];
-		}
-
-		$path = 'addressbooks/users/' . $user . '/' . $uri;
-		$properties = $this->propertyMapper->findPropertyByPathAndName($user, $path, '{http://owncloud.org/ns}enabled');
-		if (count($properties) > 0) {
-			return (bool)$properties[0]->getPropertyvalue();
-		}
-		return true;
-	}
+    /**
+     * AddressBookImpl constructor.
+     *
+     * @param AddressBook $addressBook
+     * @param array $addressBookInfo
+     * @param CardDavBackend $backend
+     * @param IUrlGenerator $urlGenerator
+     */
+    public function __construct(
+        private AddressBook $addressBook,
+        private array $addressBookInfo,
+        private CardDavBackend $backend,
+        private IURLGenerator $urlGenerator,
+        private PropertyMapper $propertyMapper,
+        private ?string $userId,
+    ) {
+    }
+
+    /**
+     * @return string defining the technical unique key
+     * @since 5.0.0
+     */
+    public function getKey() {
+        return $this->addressBookInfo['id'];
+    }
+
+    /**
+     * @return string defining the unique uri
+     * @since 16.0.0
+     */
+    public function getUri(): string {
+        return $this->addressBookInfo['uri'];
+    }
+
+    /**
+     * In comparison to getKey() this function returns a human readable (maybe translated) name
+     *
+     * @return mixed
+     * @since 5.0.0
+     */
+    public function getDisplayName() {
+        return $this->addressBookInfo['{DAV:}displayname'];
+    }
+
+    /**
+     * @param string $pattern which should match within the $searchProperties
+     * @param array $searchProperties defines the properties within the query pattern should match
+     * @param array $options Options to define the output format and search behavior
+     *                       - 'types' boolean (since 15.0.0) If set to true, fields that come with a TYPE property will be an array
+     *                       example: ['id' => 5, 'FN' => 'Thomas Tanghus', 'EMAIL' => ['type => 'HOME', 'value' => 'g@h.i']]
+     *                       - 'escape_like_param' - If set to false wildcards _ and % are not escaped
+     *                       - 'limit' - Set a numeric limit for the search results
+     *                       - 'offset' - Set the offset for the limited search results
+     *                       - 'wildcard' - Whether the search should use wildcards
+     * @psalm-param array{types?: bool, escape_like_param?: bool, limit?: int, offset?: int, wildcard?: bool} $options
+     * @return array an array of contacts which are arrays of key-value-pairs
+     *               example result:
+     *               [
+     *               ['id' => 0, 'FN' => 'Thomas Müller', 'EMAIL' => 'a@b.c', 'GEO' => '37.386013;-122.082932'],
+     *               ['id' => 5, 'FN' => 'Thomas Tanghus', 'EMAIL' => ['d@e.f', 'g@h.i']]
+     *               ]
+     * @since 5.0.0
+     */
+    public function search($pattern, $searchProperties, $options) {
+        $results = $this->backend->search($this->getKey(), $pattern, $searchProperties, $options);
+
+        $withTypes = \array_key_exists('types', $options) && $options['types'] === true;
+
+        $vCards = [];
+        foreach ($results as $result) {
+            $vCards[] = $this->vCard2Array($result['uri'], $this->readCard($result['carddata']), $withTypes);
+        }
+
+        return $vCards;
+    }
+
+    /**
+     * @param array $properties this array if key-value-pairs defines a contact
+     * @return array an array representing the contact just created or updated
+     * @since 5.0.0
+     */
+    public function createOrUpdate($properties) {
+        $update = false;
+        if (!isset($properties['URI'])) { // create a new contact
+            $uid = $this->createUid();
+            $uri = $uid . '.vcf';
+            $vCard = $this->createEmptyVCard($uid);
+        } else { // update existing contact
+            $uri = $properties['URI'];
+            $vCardData = $this->backend->getCard($this->getKey(), $uri);
+            $vCard = $this->readCard($vCardData['carddata']);
+            $update = true;
+        }
+
+        foreach ($properties as $key => $value) {
+            if (is_array($value)) {
+                $vCard->remove($key);
+                foreach ($value as $entry) {
+                    if (is_string($entry)) {
+                        $property = $vCard->createProperty($key, $entry);
+                    } else {
+                        if (($key === 'ADR' || $key === 'PHOTO') && is_string($entry['value'])) {
+                            $entry['value'] = stripslashes($entry['value']);
+                            $entry['value'] = explode(';', $entry['value']);
+                        }
+                        $property = $vCard->createProperty($key, $entry['value']);
+                        if (isset($entry['type'])) {
+                            $property->add('TYPE', $entry['type']);
+                        }
+                    }
+                    $vCard->add($property);
+                }
+            } elseif ($key !== 'URI') {
+                $vCard->$key = $vCard->createProperty($key, $value);
+            }
+        }
+
+        if ($update) {
+            $this->backend->updateCard($this->getKey(), $uri, $vCard->serialize());
+        } else {
+            $this->backend->createCard($this->getKey(), $uri, $vCard->serialize());
+        }
+
+        return $this->vCard2Array($uri, $vCard);
+    }
+
+    /**
+     * @return mixed
+     * @since 5.0.0
+     */
+    public function getPermissions() {
+        $permissions = $this->addressBook->getACL();
+        $result = 0;
+        foreach ($permissions as $permission) {
+            if ($this->addressBookInfo['principaluri'] !== $permission['principal']) {
+                continue;
+            }
+
+            switch ($permission['privilege']) {
+                case '{DAV:}read':
+                    $result |= Constants::PERMISSION_READ;
+                    break;
+                case '{DAV:}write':
+                    $result |= Constants::PERMISSION_CREATE;
+                    $result |= Constants::PERMISSION_UPDATE;
+                    break;
+                case '{DAV:}all':
+                    $result |= Constants::PERMISSION_ALL;
+                    break;
+            }
+        }
+
+        return $result;
+    }
+
+    /**
+     * @param int $id the unique identifier to a contact
+     * @return bool successful or not
+     * @since 5.0.0
+     */
+    public function delete($id) {
+        $uri = $this->backend->getCardUri($id);
+        return $this->backend->deleteCard($this->addressBookInfo['id'], $uri);
+    }
+
+    /**
+     * read vCard data into a vCard object
+     *
+     * @param string $cardData
+     * @return VCard
+     */
+    protected function readCard($cardData) {
+        return  Reader::read($cardData);
+    }
+
+    /**
+     * create UID for contact
+     *
+     * @return string
+     */
+    protected function createUid() {
+        do {
+            $uid = $this->getUid();
+            $contact = $this->backend->getContact($this->getKey(), $uid . '.vcf');
+        } while (!empty($contact));
+
+        return $uid;
+    }
+
+    /**
+     * getUid is only there for testing, use createUid instead
+     */
+    protected function getUid() {
+        return UUIDUtil::getUUID();
+    }
+
+    /**
+     * create empty vcard
+     *
+     * @param string $uid
+     * @return VCard
+     */
+    protected function createEmptyVCard($uid) {
+        $vCard = new VCard();
+        $vCard->UID = $uid;
+        return $vCard;
+    }
+
+    /**
+     * create array with all vCard properties
+     *
+     * @param string $uri
+     * @param VCard $vCard
+     * @param boolean $withTypes (optional) return the values as arrays of value/type pairs
+     * @return array
+     */
+    protected function vCard2Array($uri, VCard $vCard, $withTypes = false) {
+        $result = [
+            'URI' => $uri,
+        ];
+
+        foreach ($vCard->children() as $property) {
+            if ($property->name === 'PHOTO' && in_array($property->getValueType(), ['BINARY', 'URI'])) {
+                $url = $this->urlGenerator->getAbsoluteURL(
+                    $this->urlGenerator->linkTo('', 'remote.php') . '/dav/');
+                $url .= implode('/', [
+                    'addressbooks',
+                    substr($this->addressBookInfo['principaluri'], 11), //cut off 'principals/'
+                    $this->addressBookInfo['uri'],
+                    $uri
+                ]) . '?photo';
+
+                $result['PHOTO'] = 'VALUE=uri:' . $url;
+            } elseif (in_array($property->name, ['URL', 'GEO', 'CLOUD', 'ADR', 'EMAIL', 'IMPP', 'TEL', 'X-SOCIALPROFILE', 'RELATED', 'LANG', 'X-ADDRESSBOOKSERVER-MEMBER'])) {
+                if (!isset($result[$property->name])) {
+                    $result[$property->name] = [];
+                }
+
+                $type = $this->getTypeFromProperty($property);
+                if ($withTypes) {
+                    $result[$property->name][] = [
+                        'type' => $type,
+                        'value' => $property->getValue()
+                    ];
+                } else {
+                    $result[$property->name][] = $property->getValue();
+                }
+            } else {
+                $result[$property->name] = $property->getValue();
+            }
+        }
+
+        if ($this->isSystemAddressBook()) {
+            $result['isLocalSystemBook'] = true;
+        }
+        return $result;
+    }
+
+    /**
+     * Get the type of the current property
+     *
+     * @param Property $property
+     * @return null|string
+     */
+    protected function getTypeFromProperty(Property $property) {
+        $parameters = $property->parameters();
+        // Type is the social network, when it's empty we don't need this.
+        if (isset($parameters['TYPE'])) {
+            /** @var \Sabre\VObject\Parameter $type */
+            $type = $parameters['TYPE'];
+            return $type->getValue();
+        }
+
+        return null;
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isShared(): bool {
+        if (!isset($this->addressBookInfo['{http://owncloud.org/ns}owner-principal'])) {
+            return false;
+        }
+
+        return $this->addressBookInfo['principaluri']
+            !== $this->addressBookInfo['{http://owncloud.org/ns}owner-principal'];
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function isSystemAddressBook(): bool {
+        return $this->addressBookInfo['principaluri'] === 'principals/system/system' && (
+            $this->addressBookInfo['uri'] === 'system'
+            || $this->addressBookInfo['{DAV:}displayname'] === $this->urlGenerator->getBaseUrl()
+        );
+    }
+
+    public function isEnabled(): bool {
+        if (!$this->userId) {
+            return true;
+        }
+
+        if ($this->isSystemAddressBook()) {
+            $user = $this->userId ;
+            $uri = 'z-server-generated--system';
+        } else {
+            $user = str_replace('principals/users/', '', $this->addressBookInfo['principaluri']);
+            $uri = $this->addressBookInfo['uri'];
+        }
+
+        $path = 'addressbooks/users/' . $user . '/' . $uri;
+        $properties = $this->propertyMapper->findPropertyByPathAndName($user, $path, '{http://owncloud.org/ns}enabled');
+        if (count($properties) > 0) {
+            return (bool)$properties[0]->getPropertyvalue();
+        }
+        return true;
+    }
 }

apps/dav/tests/unit/CardDAV/AddressBookImplTest.php

Indentation +515 added lines, -515 removed lines

@@ -19,519 +19,519 @@
 use Test\TestCase;
 
 class AddressBookImplTest extends TestCase {
-	private array $addressBookInfo;
-	private AddressBook&MockObject $addressBook;
-	private IURLGenerator&MockObject $urlGenerator;
-	private CardDavBackend&MockObject $backend;
-	private PropertyMapper&MockObject $propertyMapper;
-	private VCard&MockObject $vCard;
-	private AddressBookImpl $addressBookImpl;
-
-	protected function setUp(): void {
-		parent::setUp();
-
-		$this->addressBookInfo = [
-			'id' => 42,
-			'uri' => 'system',
-			'principaluri' => 'principals/system/system',
-			'{DAV:}displayname' => 'display name',
-		];
-		$this->addressBook = $this->createMock(AddressBook::class);
-		$this->backend = $this->createMock(CardDavBackend::class);
-		$this->vCard = $this->createMock(VCard::class);
-		$this->urlGenerator = $this->createMock(IURLGenerator::class);
-		$this->propertyMapper = $this->createMock(PropertyMapper::class);
-
-		$this->addressBookImpl = new AddressBookImpl(
-			$this->addressBook,
-			$this->addressBookInfo,
-			$this->backend,
-			$this->urlGenerator,
-			$this->propertyMapper,
-			null
-		);
-	}
-
-	public function testGetKey(): void {
-		$this->assertSame($this->addressBookInfo['id'],
-			$this->addressBookImpl->getKey());
-	}
-
-	public function testGetDisplayName(): void {
-		$this->assertSame($this->addressBookInfo['{DAV:}displayname'],
-			$this->addressBookImpl->getDisplayName());
-	}
-
-	public function testSearch(): void {
-		/** @var MockObject&AddressBookImpl $addressBookImpl */
-		$addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
-			->setConstructorArgs(
-				[
-					$this->addressBook,
-					$this->addressBookInfo,
-					$this->backend,
-					$this->urlGenerator,
-					$this->propertyMapper,
-					null
-				]
-			)
-			->onlyMethods(['vCard2Array', 'readCard'])
-			->getMock();
-
-		$pattern = 'pattern';
-		$searchProperties = 'properties';
-
-		$this->backend->expects($this->once())->method('search')
-			->with($this->addressBookInfo['id'], $pattern, $searchProperties)
-			->willReturn(
-				[
-					['uri' => 'foo.vcf', 'carddata' => 'cardData1'],
-					['uri' => 'bar.vcf', 'carddata' => 'cardData2']
-				]
-			);
-
-		$addressBookImpl->expects($this->exactly(2))->method('readCard')
-			->willReturn($this->vCard);
-		$addressBookImpl->expects($this->exactly(2))->method('vCard2Array')
-			->willReturnMap([
-				['foo.vcf', $this->vCard, 'vCard'],
-				['bar.vcf', $this->vCard, 'vCard'],
-			]);
-
-		$result = $addressBookImpl->search($pattern, $searchProperties, []);
-		$this->assertTrue((is_array($result)));
-		$this->assertSame(2, count($result));
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataTestCreate')]
-	public function testCreate(array $properties): void {
-		$uid = 'uid';
-
-		/** @var MockObject&AddressBookImpl $addressBookImpl */
-		$addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
-			->setConstructorArgs(
-				[
-					$this->addressBook,
-					$this->addressBookInfo,
-					$this->backend,
-					$this->urlGenerator,
-					$this->propertyMapper,
-					null
-				]
-			)
-			->onlyMethods(['vCard2Array', 'createUid', 'createEmptyVCard'])
-			->getMock();
-
-		$expectedProperties = 0;
-		foreach ($properties as $data) {
-			if (is_string($data)) {
-				$expectedProperties++;
-			} else {
-				$expectedProperties += count($data);
-			}
-		}
-
-		$addressBookImpl->expects($this->once())->method('createUid')
-			->willReturn($uid);
-		$addressBookImpl->expects($this->once())->method('createEmptyVCard')
-			->with($uid)->willReturn($this->vCard);
-		$this->vCard->expects($this->exactly($expectedProperties))
-			->method('createProperty');
-		$this->backend->expects($this->once())->method('createCard');
-		$this->backend->expects($this->never())->method('updateCard');
-		$this->backend->expects($this->never())->method('getCard');
-		$addressBookImpl->expects($this->once())->method('vCard2Array')
-			->with('uid.vcf', $this->vCard)->willReturn(true);
-
-		$this->assertTrue($addressBookImpl->createOrUpdate($properties));
-	}
-
-	public static function dataTestCreate(): array {
-		return [
-			[[]],
-			[['FN' => 'John Doe']],
-			[['FN' => 'John Doe', 'EMAIL' => ['john@doe.cloud', 'john.doe@example.org']]],
-		];
-	}
-
-	public function testUpdate(): void {
-		$uid = 'uid';
-		$uri = 'bla.vcf';
-		$properties = ['URI' => $uri, 'UID' => $uid, 'FN' => 'John Doe'];
-
-		/** @var MockObject&AddressBookImpl $addressBookImpl */
-		$addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
-			->setConstructorArgs(
-				[
-					$this->addressBook,
-					$this->addressBookInfo,
-					$this->backend,
-					$this->urlGenerator,
-					$this->propertyMapper,
-					null
-				]
-			)
-			->onlyMethods(['vCard2Array', 'createUid', 'createEmptyVCard', 'readCard'])
-			->getMock();
-
-		$addressBookImpl->expects($this->never())->method('createUid');
-		$addressBookImpl->expects($this->never())->method('createEmptyVCard');
-		$this->backend->expects($this->once())->method('getCard')
-			->with($this->addressBookInfo['id'], $uri)
-			->willReturn(['carddata' => 'data']);
-		$addressBookImpl->expects($this->once())->method('readCard')
-			->with('data')->willReturn($this->vCard);
-		$this->vCard->expects($this->exactly(count($properties) - 1))
-			->method('createProperty');
-		$this->backend->expects($this->never())->method('createCard');
-		$this->backend->expects($this->once())->method('updateCard');
-		$addressBookImpl->expects($this->once())->method('vCard2Array')
-			->with($uri, $this->vCard)->willReturn(true);
-
-		$this->assertTrue($addressBookImpl->createOrUpdate($properties));
-	}
-
-	public function testUpdateWithTypes(): void {
-		$uid = 'uid';
-		$uri = 'bla.vcf';
-		$properties = ['URI' => $uri, 'UID' => $uid, 'FN' => 'John Doe', 'ADR' => [['type' => 'HOME', 'value' => ';;street;city;;;country']]];
-		$vCard = new vCard;
-		$textProperty = $vCard->createProperty('KEY', 'value');
-
-		/** @var MockObject&AddressBookImpl $addressBookImpl */
-		$addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
-			->setConstructorArgs(
-				[
-					$this->addressBook,
-					$this->addressBookInfo,
-					$this->backend,
-					$this->urlGenerator,
-					$this->propertyMapper,
-					null
-				]
-			)
-			->onlyMethods(['vCard2Array', 'createUid', 'createEmptyVCard', 'readCard'])
-			->getMock();
-
-		$this->backend->expects($this->once())->method('getCard')
-			->with($this->addressBookInfo['id'], $uri)
-			->willReturn(['carddata' => 'data']);
-		$addressBookImpl->expects($this->once())->method('readCard')
-			->with('data')->willReturn($this->vCard);
-		$this->vCard->method('createProperty')->willReturn($textProperty);
-		$this->vCard->expects($this->exactly(count($properties) - 1))
-			->method('createProperty');
-		$this->vCard->expects($this->once())->method('remove')
-			->with('ADR');
-		$this->vCard->expects($this->once())->method('add');
-
-		$addressBookImpl->createOrUpdate($properties);
-	}
-
-	#[\PHPUnit\Framework\Attributes\DataProvider('dataTestGetPermissions')]
-	public function testGetPermissions(array $permissions, int $expected): void {
-		$this->addressBook->expects($this->once())->method('getACL')
-			->willReturn($permissions);
-
-		$this->assertSame($expected,
-			$this->addressBookImpl->getPermissions()
-		);
-	}
-
-	public static function dataTestGetPermissions(): array {
-		return [
-			[[], 0],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system']], 1],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}write', 'principal' => 'principals/someone/else']], 1],
-			[[['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 6],
-			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 7],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
-		];
-	}
-
-	public function testDelete(): void {
-		$cardId = 1;
-		$cardUri = 'cardUri';
-		$this->backend->expects($this->once())->method('getCardUri')
-			->with($cardId)->willReturn($cardUri);
-		$this->backend->expects($this->once())->method('deleteCard')
-			->with($this->addressBookInfo['id'], $cardUri)
-			->willReturn(true);
-
-		$this->assertTrue($this->addressBookImpl->delete($cardId));
-	}
-
-	public function testReadCard(): void {
-		$vCard = new VCard();
-		$vCard->add(new Text($vCard, 'UID', 'uid'));
-		$vCardSerialized = $vCard->serialize();
-
-		$result = $this->invokePrivate($this->addressBookImpl, 'readCard', [$vCardSerialized]);
-		$resultSerialized = $result->serialize();
-
-		$this->assertSame($vCardSerialized, $resultSerialized);
-	}
-
-	public function testCreateUid(): void {
-		/** @var MockObject&AddressBookImpl $addressBookImpl */
-		$addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
-			->setConstructorArgs(
-				[
-					$this->addressBook,
-					$this->addressBookInfo,
-					$this->backend,
-					$this->urlGenerator,
-					$this->propertyMapper,
-					null
-				]
-			)
-			->onlyMethods(['getUid'])
-			->getMock();
-
-		$addressBookImpl->expects($this->exactly(2))
-			->method('getUid')
-			->willReturnOnConsecutiveCalls(
-				'uid0',
-				'uid1',
-			);
-
-		// simulate that 'uid0' already exists, so the second uid will be returned
-		$this->backend->expects($this->exactly(2))->method('getContact')
-			->willReturnCallback(
-				function ($id, $uid) {
-					return ($uid === 'uid0.vcf');
-				}
-			);
-
-		$this->assertSame('uid1',
-			$this->invokePrivate($addressBookImpl, 'createUid', [])
-		);
-	}
-
-	public function testCreateEmptyVCard(): void {
-		$uid = 'uid';
-		$expectedVCard = new VCard();
-		$expectedVCard->UID = $uid;
-		$expectedVCardSerialized = $expectedVCard->serialize();
-
-		$result = $this->invokePrivate($this->addressBookImpl, 'createEmptyVCard', [$uid]);
-		$resultSerialized = $result->serialize();
-
-		$this->assertSame($expectedVCardSerialized, $resultSerialized);
-	}
-
-	public function testVCard2Array(): void {
-		$vCard = new VCard();
-
-		$vCard->add($vCard->createProperty('FN', 'Full Name'));
-
-		// Multi-value properties
-		$vCard->add($vCard->createProperty('CLOUD', 'cloud-user1@localhost'));
-		$vCard->add($vCard->createProperty('CLOUD', 'cloud-user2@example.tld'));
-		$vCard->add($vCard->createProperty('EMAIL', 'email-user1@localhost'));
-		$vCard->add($vCard->createProperty('EMAIL', 'email-user2@example.tld'));
-		$vCard->add($vCard->createProperty('IMPP', 'impp-user1@localhost'));
-		$vCard->add($vCard->createProperty('IMPP', 'impp-user2@example.tld'));
-		$vCard->add($vCard->createProperty('TEL', '+49 123456789'));
-		$vCard->add($vCard->createProperty('TEL', '+1 555 123456789'));
-		$vCard->add($vCard->createProperty('URL', 'https://localhost'));
-		$vCard->add($vCard->createProperty('URL', 'https://example.tld'));
-
-		// Type depending properties
-		$property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example');
-		$property->add('TYPE', 'twitter');
-		$vCard->add($property);
-		$property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example-2');
-		$property->add('TYPE', 'twitter');
-		$vCard->add($property);
-		$property = $vCard->createProperty('X-SOCIALPROFILE', 'fb-example');
-		$property->add('TYPE', 'facebook');
-		$vCard->add($property);
-
-		$array = $this->invokePrivate($this->addressBookImpl, 'vCard2Array', ['uri', $vCard]);
-		unset($array['PRODID']);
-		unset($array['UID']);
-
-		$this->assertEquals([
-			'URI' => 'uri',
-			'VERSION' => '4.0',
-			'FN' => 'Full Name',
-			'CLOUD' => [
-				'cloud-user1@localhost',
-				'cloud-user2@example.tld',
-			],
-			'EMAIL' => [
-				'email-user1@localhost',
-				'email-user2@example.tld',
-			],
-			'IMPP' => [
-				'impp-user1@localhost',
-				'impp-user2@example.tld',
-			],
-			'TEL' => [
-				'+49 123456789',
-				'+1 555 123456789',
-			],
-			'URL' => [
-				'https://localhost',
-				'https://example.tld',
-			],
-
-			'X-SOCIALPROFILE' => [
-				'tw-example',
-				'tw-example-2',
-				'fb-example',
-			],
-
-			'isLocalSystemBook' => true,
-		], $array);
-	}
-
-	public function testVCard2ArrayWithTypes(): void {
-		$vCard = new VCard();
-
-		$vCard->add($vCard->createProperty('FN', 'Full Name'));
-
-		// Multi-value properties
-		$vCard->add($vCard->createProperty('CLOUD', 'cloud-user1@localhost'));
-		$vCard->add($vCard->createProperty('CLOUD', 'cloud-user2@example.tld'));
-
-		$property = $vCard->createProperty('EMAIL', 'email-user1@localhost');
-		$property->add('TYPE', 'HOME');
-		$vCard->add($property);
-		$property = $vCard->createProperty('EMAIL', 'email-user2@example.tld');
-		$property->add('TYPE', 'WORK');
-		$vCard->add($property);
-
-		$vCard->add($vCard->createProperty('IMPP', 'impp-user1@localhost'));
-		$vCard->add($vCard->createProperty('IMPP', 'impp-user2@example.tld'));
-
-		$property = $vCard->createProperty('TEL', '+49 123456789');
-		$property->add('TYPE', 'HOME,VOICE');
-		$vCard->add($property);
-		$property = $vCard->createProperty('TEL', '+1 555 123456789');
-		$property->add('TYPE', 'WORK');
-		$vCard->add($property);
-
-		$vCard->add($vCard->createProperty('URL', 'https://localhost'));
-		$vCard->add($vCard->createProperty('URL', 'https://example.tld'));
-
-		// Type depending properties
-		$property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example');
-		$property->add('TYPE', 'twitter');
-		$vCard->add($property);
-		$property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example-2');
-		$property->add('TYPE', 'twitter');
-		$vCard->add($property);
-		$property = $vCard->createProperty('X-SOCIALPROFILE', 'fb-example');
-		$property->add('TYPE', 'facebook');
-		$vCard->add($property);
-
-		$array = $this->invokePrivate($this->addressBookImpl, 'vCard2Array', ['uri', $vCard, true]);
-		unset($array['PRODID']);
-		unset($array['UID']);
-
-		$this->assertEquals([
-			'URI' => 'uri',
-			'VERSION' => '4.0',
-			'FN' => 'Full Name',
-			'CLOUD' => [
-				['type' => '', 'value' => 'cloud-user1@localhost'],
-				['type' => '', 'value' => 'cloud-user2@example.tld'],
-			],
-			'EMAIL' => [
-				['type' => 'HOME', 'value' => 'email-user1@localhost'],
-				['type' => 'WORK', 'value' => 'email-user2@example.tld'],
-			],
-			'IMPP' => [
-				['type' => '', 'value' => 'impp-user1@localhost'],
-				['type' => '', 'value' => 'impp-user2@example.tld'],
-			],
-			'TEL' => [
-				['type' => 'HOME,VOICE', 'value' => '+49 123456789'],
-				['type' => 'WORK', 'value' => '+1 555 123456789'],
-			],
-			'URL' => [
-				['type' => '', 'value' => 'https://localhost'],
-				['type' => '', 'value' => 'https://example.tld'],
-			],
-
-			'X-SOCIALPROFILE' => [
-				['type' => 'twitter', 'value' => 'tw-example'],
-				['type' => 'twitter', 'value' => 'tw-example-2'],
-				['type' => 'facebook', 'value' => 'fb-example'],
-			],
-
-			'isLocalSystemBook' => true,
-		], $array);
-	}
-
-	public function testIsSystemAddressBook(): void {
-		$addressBookInfo = [
-			'{http://owncloud.org/ns}owner-principal' => 'principals/system/system',
-			'principaluri' => 'principals/system/system',
-			'{DAV:}displayname' => 'display name',
-			'id' => 666,
-			'uri' => 'system',
-		];
-
-		$addressBookImpl = new AddressBookImpl(
-			$this->addressBook,
-			$addressBookInfo,
-			$this->backend,
-			$this->urlGenerator,
-			$this->propertyMapper,
-			null
-		);
-
-		$this->assertTrue($addressBookImpl->isSystemAddressBook());
-	}
-
-	public function testIsShared(): void {
-		$addressBookInfo = [
-			'{http://owncloud.org/ns}owner-principal' => 'user1',
-			'{DAV:}displayname' => 'Test address book',
-			'principaluri' => 'user2',
-			'id' => 666,
-			'uri' => 'default',
-		];
-
-		$addressBookImpl = new AddressBookImpl(
-			$this->addressBook,
-			$addressBookInfo,
-			$this->backend,
-			$this->urlGenerator,
-			$this->propertyMapper,
-			'user2'
-		);
-
-		$this->assertFalse($addressBookImpl->isSystemAddressBook());
-		$this->assertTrue($addressBookImpl->isShared());
-	}
-
-	public function testIsNotShared(): void {
-		$addressBookInfo = [
-			'{http://owncloud.org/ns}owner-principal' => 'user1',
-			'{DAV:}displayname' => 'Test address book',
-			'principaluri' => 'user1',
-			'id' => 666,
-			'uri' => 'default',
-		];
-
-		$addressBookImpl = new AddressBookImpl(
-			$this->addressBook,
-			$addressBookInfo,
-			$this->backend,
-			$this->urlGenerator,
-			$this->propertyMapper,
-			'user2'
-		);
-
-		$this->assertFalse($addressBookImpl->isSystemAddressBook());
-		$this->assertFalse($addressBookImpl->isShared());
-	}
+    private array $addressBookInfo;
+    private AddressBook&MockObject $addressBook;
+    private IURLGenerator&MockObject $urlGenerator;
+    private CardDavBackend&MockObject $backend;
+    private PropertyMapper&MockObject $propertyMapper;
+    private VCard&MockObject $vCard;
+    private AddressBookImpl $addressBookImpl;
+
+    protected function setUp(): void {
+        parent::setUp();
+
+        $this->addressBookInfo = [
+            'id' => 42,
+            'uri' => 'system',
+            'principaluri' => 'principals/system/system',
+            '{DAV:}displayname' => 'display name',
+        ];
+        $this->addressBook = $this->createMock(AddressBook::class);
+        $this->backend = $this->createMock(CardDavBackend::class);
+        $this->vCard = $this->createMock(VCard::class);
+        $this->urlGenerator = $this->createMock(IURLGenerator::class);
+        $this->propertyMapper = $this->createMock(PropertyMapper::class);
+
+        $this->addressBookImpl = new AddressBookImpl(
+            $this->addressBook,
+            $this->addressBookInfo,
+            $this->backend,
+            $this->urlGenerator,
+            $this->propertyMapper,
+            null
+        );
+    }
+
+    public function testGetKey(): void {
+        $this->assertSame($this->addressBookInfo['id'],
+            $this->addressBookImpl->getKey());
+    }
+
+    public function testGetDisplayName(): void {
+        $this->assertSame($this->addressBookInfo['{DAV:}displayname'],
+            $this->addressBookImpl->getDisplayName());
+    }
+
+    public function testSearch(): void {
+        /** @var MockObject&AddressBookImpl $addressBookImpl */
+        $addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
+            ->setConstructorArgs(
+                [
+                    $this->addressBook,
+                    $this->addressBookInfo,
+                    $this->backend,
+                    $this->urlGenerator,
+                    $this->propertyMapper,
+                    null
+                ]
+            )
+            ->onlyMethods(['vCard2Array', 'readCard'])
+            ->getMock();
+
+        $pattern = 'pattern';
+        $searchProperties = 'properties';
+
+        $this->backend->expects($this->once())->method('search')
+            ->with($this->addressBookInfo['id'], $pattern, $searchProperties)
+            ->willReturn(
+                [
+                    ['uri' => 'foo.vcf', 'carddata' => 'cardData1'],
+                    ['uri' => 'bar.vcf', 'carddata' => 'cardData2']
+                ]
+            );
+
+        $addressBookImpl->expects($this->exactly(2))->method('readCard')
+            ->willReturn($this->vCard);
+        $addressBookImpl->expects($this->exactly(2))->method('vCard2Array')
+            ->willReturnMap([
+                ['foo.vcf', $this->vCard, 'vCard'],
+                ['bar.vcf', $this->vCard, 'vCard'],
+            ]);
+
+        $result = $addressBookImpl->search($pattern, $searchProperties, []);
+        $this->assertTrue((is_array($result)));
+        $this->assertSame(2, count($result));
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('dataTestCreate')]
+    public function testCreate(array $properties): void {
+        $uid = 'uid';
+
+        /** @var MockObject&AddressBookImpl $addressBookImpl */
+        $addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
+            ->setConstructorArgs(
+                [
+                    $this->addressBook,
+                    $this->addressBookInfo,
+                    $this->backend,
+                    $this->urlGenerator,
+                    $this->propertyMapper,
+                    null
+                ]
+            )
+            ->onlyMethods(['vCard2Array', 'createUid', 'createEmptyVCard'])
+            ->getMock();
+
+        $expectedProperties = 0;
+        foreach ($properties as $data) {
+            if (is_string($data)) {
+                $expectedProperties++;
+            } else {
+                $expectedProperties += count($data);
+            }
+        }
+
+        $addressBookImpl->expects($this->once())->method('createUid')
+            ->willReturn($uid);
+        $addressBookImpl->expects($this->once())->method('createEmptyVCard')
+            ->with($uid)->willReturn($this->vCard);
+        $this->vCard->expects($this->exactly($expectedProperties))
+            ->method('createProperty');
+        $this->backend->expects($this->once())->method('createCard');
+        $this->backend->expects($this->never())->method('updateCard');
+        $this->backend->expects($this->never())->method('getCard');
+        $addressBookImpl->expects($this->once())->method('vCard2Array')
+            ->with('uid.vcf', $this->vCard)->willReturn(true);
+
+        $this->assertTrue($addressBookImpl->createOrUpdate($properties));
+    }
+
+    public static function dataTestCreate(): array {
+        return [
+            [[]],
+            [['FN' => 'John Doe']],
+            [['FN' => 'John Doe', 'EMAIL' => ['john@doe.cloud', 'john.doe@example.org']]],
+        ];
+    }
+
+    public function testUpdate(): void {
+        $uid = 'uid';
+        $uri = 'bla.vcf';
+        $properties = ['URI' => $uri, 'UID' => $uid, 'FN' => 'John Doe'];
+
+        /** @var MockObject&AddressBookImpl $addressBookImpl */
+        $addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
+            ->setConstructorArgs(
+                [
+                    $this->addressBook,
+                    $this->addressBookInfo,
+                    $this->backend,
+                    $this->urlGenerator,
+                    $this->propertyMapper,
+                    null
+                ]
+            )
+            ->onlyMethods(['vCard2Array', 'createUid', 'createEmptyVCard', 'readCard'])
+            ->getMock();
+
+        $addressBookImpl->expects($this->never())->method('createUid');
+        $addressBookImpl->expects($this->never())->method('createEmptyVCard');
+        $this->backend->expects($this->once())->method('getCard')
+            ->with($this->addressBookInfo['id'], $uri)
+            ->willReturn(['carddata' => 'data']);
+        $addressBookImpl->expects($this->once())->method('readCard')
+            ->with('data')->willReturn($this->vCard);
+        $this->vCard->expects($this->exactly(count($properties) - 1))
+            ->method('createProperty');
+        $this->backend->expects($this->never())->method('createCard');
+        $this->backend->expects($this->once())->method('updateCard');
+        $addressBookImpl->expects($this->once())->method('vCard2Array')
+            ->with($uri, $this->vCard)->willReturn(true);
+
+        $this->assertTrue($addressBookImpl->createOrUpdate($properties));
+    }
+
+    public function testUpdateWithTypes(): void {
+        $uid = 'uid';
+        $uri = 'bla.vcf';
+        $properties = ['URI' => $uri, 'UID' => $uid, 'FN' => 'John Doe', 'ADR' => [['type' => 'HOME', 'value' => ';;street;city;;;country']]];
+        $vCard = new vCard;
+        $textProperty = $vCard->createProperty('KEY', 'value');
+
+        /** @var MockObject&AddressBookImpl $addressBookImpl */
+        $addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
+            ->setConstructorArgs(
+                [
+                    $this->addressBook,
+                    $this->addressBookInfo,
+                    $this->backend,
+                    $this->urlGenerator,
+                    $this->propertyMapper,
+                    null
+                ]
+            )
+            ->onlyMethods(['vCard2Array', 'createUid', 'createEmptyVCard', 'readCard'])
+            ->getMock();
+
+        $this->backend->expects($this->once())->method('getCard')
+            ->with($this->addressBookInfo['id'], $uri)
+            ->willReturn(['carddata' => 'data']);
+        $addressBookImpl->expects($this->once())->method('readCard')
+            ->with('data')->willReturn($this->vCard);
+        $this->vCard->method('createProperty')->willReturn($textProperty);
+        $this->vCard->expects($this->exactly(count($properties) - 1))
+            ->method('createProperty');
+        $this->vCard->expects($this->once())->method('remove')
+            ->with('ADR');
+        $this->vCard->expects($this->once())->method('add');
+
+        $addressBookImpl->createOrUpdate($properties);
+    }
+
+    #[\PHPUnit\Framework\Attributes\DataProvider('dataTestGetPermissions')]
+    public function testGetPermissions(array $permissions, int $expected): void {
+        $this->addressBook->expects($this->once())->method('getACL')
+            ->willReturn($permissions);
+
+        $this->assertSame($expected,
+            $this->addressBookImpl->getPermissions()
+        );
+    }
+
+    public static function dataTestGetPermissions(): array {
+        return [
+            [[], 0],
+            [[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system']], 1],
+            [[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}write', 'principal' => 'principals/someone/else']], 1],
+            [[['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 6],
+            [[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
+            [[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 7],
+            [[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
+            [[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
+            [[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
+            [[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
+        ];
+    }
+
+    public function testDelete(): void {
+        $cardId = 1;
+        $cardUri = 'cardUri';
+        $this->backend->expects($this->once())->method('getCardUri')
+            ->with($cardId)->willReturn($cardUri);
+        $this->backend->expects($this->once())->method('deleteCard')
+            ->with($this->addressBookInfo['id'], $cardUri)
+            ->willReturn(true);
+
+        $this->assertTrue($this->addressBookImpl->delete($cardId));
+    }
+
+    public function testReadCard(): void {
+        $vCard = new VCard();
+        $vCard->add(new Text($vCard, 'UID', 'uid'));
+        $vCardSerialized = $vCard->serialize();
+
+        $result = $this->invokePrivate($this->addressBookImpl, 'readCard', [$vCardSerialized]);
+        $resultSerialized = $result->serialize();
+
+        $this->assertSame($vCardSerialized, $resultSerialized);
+    }
+
+    public function testCreateUid(): void {
+        /** @var MockObject&AddressBookImpl $addressBookImpl */
+        $addressBookImpl = $this->getMockBuilder(AddressBookImpl::class)
+            ->setConstructorArgs(
+                [
+                    $this->addressBook,
+                    $this->addressBookInfo,
+                    $this->backend,
+                    $this->urlGenerator,
+                    $this->propertyMapper,
+                    null
+                ]
+            )
+            ->onlyMethods(['getUid'])
+            ->getMock();
+
+        $addressBookImpl->expects($this->exactly(2))
+            ->method('getUid')
+            ->willReturnOnConsecutiveCalls(
+                'uid0',
+                'uid1',
+            );
+
+        // simulate that 'uid0' already exists, so the second uid will be returned
+        $this->backend->expects($this->exactly(2))->method('getContact')
+            ->willReturnCallback(
+                function ($id, $uid) {
+                    return ($uid === 'uid0.vcf');
+                }
+            );
+
+        $this->assertSame('uid1',
+            $this->invokePrivate($addressBookImpl, 'createUid', [])
+        );
+    }
+
+    public function testCreateEmptyVCard(): void {
+        $uid = 'uid';
+        $expectedVCard = new VCard();
+        $expectedVCard->UID = $uid;
+        $expectedVCardSerialized = $expectedVCard->serialize();
+
+        $result = $this->invokePrivate($this->addressBookImpl, 'createEmptyVCard', [$uid]);
+        $resultSerialized = $result->serialize();
+
+        $this->assertSame($expectedVCardSerialized, $resultSerialized);
+    }
+
+    public function testVCard2Array(): void {
+        $vCard = new VCard();
+
+        $vCard->add($vCard->createProperty('FN', 'Full Name'));
+
+        // Multi-value properties
+        $vCard->add($vCard->createProperty('CLOUD', 'cloud-user1@localhost'));
+        $vCard->add($vCard->createProperty('CLOUD', 'cloud-user2@example.tld'));
+        $vCard->add($vCard->createProperty('EMAIL', 'email-user1@localhost'));
+        $vCard->add($vCard->createProperty('EMAIL', 'email-user2@example.tld'));
+        $vCard->add($vCard->createProperty('IMPP', 'impp-user1@localhost'));
+        $vCard->add($vCard->createProperty('IMPP', 'impp-user2@example.tld'));
+        $vCard->add($vCard->createProperty('TEL', '+49 123456789'));
+        $vCard->add($vCard->createProperty('TEL', '+1 555 123456789'));
+        $vCard->add($vCard->createProperty('URL', 'https://localhost'));
+        $vCard->add($vCard->createProperty('URL', 'https://example.tld'));
+
+        // Type depending properties
+        $property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example');
+        $property->add('TYPE', 'twitter');
+        $vCard->add($property);
+        $property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example-2');
+        $property->add('TYPE', 'twitter');
+        $vCard->add($property);
+        $property = $vCard->createProperty('X-SOCIALPROFILE', 'fb-example');
+        $property->add('TYPE', 'facebook');
+        $vCard->add($property);
+
+        $array = $this->invokePrivate($this->addressBookImpl, 'vCard2Array', ['uri', $vCard]);
+        unset($array['PRODID']);
+        unset($array['UID']);
+
+        $this->assertEquals([
+            'URI' => 'uri',
+            'VERSION' => '4.0',
+            'FN' => 'Full Name',
+            'CLOUD' => [
+                'cloud-user1@localhost',
+                'cloud-user2@example.tld',
+            ],
+            'EMAIL' => [
+                'email-user1@localhost',
+                'email-user2@example.tld',
+            ],
+            'IMPP' => [
+                'impp-user1@localhost',
+                'impp-user2@example.tld',
+            ],
+            'TEL' => [
+                '+49 123456789',
+                '+1 555 123456789',
+            ],
+            'URL' => [
+                'https://localhost',
+                'https://example.tld',
+            ],
+
+            'X-SOCIALPROFILE' => [
+                'tw-example',
+                'tw-example-2',
+                'fb-example',
+            ],
+
+            'isLocalSystemBook' => true,
+        ], $array);
+    }
+
+    public function testVCard2ArrayWithTypes(): void {
+        $vCard = new VCard();
+
+        $vCard->add($vCard->createProperty('FN', 'Full Name'));
+
+        // Multi-value properties
+        $vCard->add($vCard->createProperty('CLOUD', 'cloud-user1@localhost'));
+        $vCard->add($vCard->createProperty('CLOUD', 'cloud-user2@example.tld'));
+
+        $property = $vCard->createProperty('EMAIL', 'email-user1@localhost');
+        $property->add('TYPE', 'HOME');
+        $vCard->add($property);
+        $property = $vCard->createProperty('EMAIL', 'email-user2@example.tld');
+        $property->add('TYPE', 'WORK');
+        $vCard->add($property);
+
+        $vCard->add($vCard->createProperty('IMPP', 'impp-user1@localhost'));
+        $vCard->add($vCard->createProperty('IMPP', 'impp-user2@example.tld'));
+
+        $property = $vCard->createProperty('TEL', '+49 123456789');
+        $property->add('TYPE', 'HOME,VOICE');
+        $vCard->add($property);
+        $property = $vCard->createProperty('TEL', '+1 555 123456789');
+        $property->add('TYPE', 'WORK');
+        $vCard->add($property);
+
+        $vCard->add($vCard->createProperty('URL', 'https://localhost'));
+        $vCard->add($vCard->createProperty('URL', 'https://example.tld'));
+
+        // Type depending properties
+        $property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example');
+        $property->add('TYPE', 'twitter');
+        $vCard->add($property);
+        $property = $vCard->createProperty('X-SOCIALPROFILE', 'tw-example-2');
+        $property->add('TYPE', 'twitter');
+        $vCard->add($property);
+        $property = $vCard->createProperty('X-SOCIALPROFILE', 'fb-example');
+        $property->add('TYPE', 'facebook');
+        $vCard->add($property);
+
+        $array = $this->invokePrivate($this->addressBookImpl, 'vCard2Array', ['uri', $vCard, true]);
+        unset($array['PRODID']);
+        unset($array['UID']);
+
+        $this->assertEquals([
+            'URI' => 'uri',
+            'VERSION' => '4.0',
+            'FN' => 'Full Name',
+            'CLOUD' => [
+                ['type' => '', 'value' => 'cloud-user1@localhost'],
+                ['type' => '', 'value' => 'cloud-user2@example.tld'],
+            ],
+            'EMAIL' => [
+                ['type' => 'HOME', 'value' => 'email-user1@localhost'],
+                ['type' => 'WORK', 'value' => 'email-user2@example.tld'],
+            ],
+            'IMPP' => [
+                ['type' => '', 'value' => 'impp-user1@localhost'],
+                ['type' => '', 'value' => 'impp-user2@example.tld'],
+            ],
+            'TEL' => [
+                ['type' => 'HOME,VOICE', 'value' => '+49 123456789'],
+                ['type' => 'WORK', 'value' => '+1 555 123456789'],
+            ],
+            'URL' => [
+                ['type' => '', 'value' => 'https://localhost'],
+                ['type' => '', 'value' => 'https://example.tld'],
+            ],
+
+            'X-SOCIALPROFILE' => [
+                ['type' => 'twitter', 'value' => 'tw-example'],
+                ['type' => 'twitter', 'value' => 'tw-example-2'],
+                ['type' => 'facebook', 'value' => 'fb-example'],
+            ],
+
+            'isLocalSystemBook' => true,
+        ], $array);
+    }
+
+    public function testIsSystemAddressBook(): void {
+        $addressBookInfo = [
+            '{http://owncloud.org/ns}owner-principal' => 'principals/system/system',
+            'principaluri' => 'principals/system/system',
+            '{DAV:}displayname' => 'display name',
+            'id' => 666,
+            'uri' => 'system',
+        ];
+
+        $addressBookImpl = new AddressBookImpl(
+            $this->addressBook,
+            $addressBookInfo,
+            $this->backend,
+            $this->urlGenerator,
+            $this->propertyMapper,
+            null
+        );
+
+        $this->assertTrue($addressBookImpl->isSystemAddressBook());
+    }
+
+    public function testIsShared(): void {
+        $addressBookInfo = [
+            '{http://owncloud.org/ns}owner-principal' => 'user1',
+            '{DAV:}displayname' => 'Test address book',
+            'principaluri' => 'user2',
+            'id' => 666,
+            'uri' => 'default',
+        ];
+
+        $addressBookImpl = new AddressBookImpl(
+            $this->addressBook,
+            $addressBookInfo,
+            $this->backend,
+            $this->urlGenerator,
+            $this->propertyMapper,
+            'user2'
+        );
+
+        $this->assertFalse($addressBookImpl->isSystemAddressBook());
+        $this->assertTrue($addressBookImpl->isShared());
+    }
+
+    public function testIsNotShared(): void {
+        $addressBookInfo = [
+            '{http://owncloud.org/ns}owner-principal' => 'user1',
+            '{DAV:}displayname' => 'Test address book',
+            'principaluri' => 'user1',
+            'id' => 666,
+            'uri' => 'default',
+        ];
+
+        $addressBookImpl = new AddressBookImpl(
+            $this->addressBook,
+            $addressBookInfo,
+            $this->backend,
+            $this->urlGenerator,
+            $this->propertyMapper,
+            'user2'
+        );
+
+        $this->assertFalse($addressBookImpl->isSystemAddressBook());
+        $this->assertFalse($addressBookImpl->isShared());
+    }
 }

Spacing +6 added lines, -6 removed lines

@@ -245,11 +245,11 @@ 
 			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}write', 'principal' => 'principals/someone/else']], 1],
 			[[['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 6],
 			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 7],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
-			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'],['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
+			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 7],
+			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
+			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
+			[[['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}write', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}all', 'principal' => 'principals/system/system']], 31],
+			[[['privilege' => '{DAV:}all', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}read', 'principal' => 'principals/system/system'], ['privilege' => '{DAV:}write', 'principal' => 'principals/system/system']], 31],
 		];
 	}
 
@@ -302,7 +302,7 @@ 
 		// simulate that 'uid0' already exists, so the second uid will be returned
 		$this->backend->expects($this->exactly(2))->method('getContact')
 			->willReturnCallback(
-				function ($id, $uid) {
+				function($id, $uid) {
 					return ($uid === 'uid0.vcf');
 				}
 			);