Merge pull request #49596 from nextcloud/backport/49581/stable29

[stable29] chore(absence): Add capability

Author: provokateurin

apps/dav/lib/Capabilities.php

@@ -26,16 +26,19 @@
 
 use OCP\Capabilities\ICapability;
 use OCP\IConfig;
+use OCP\User\IAvailabilityCoordinator;
 
 class Capabilities implements ICapability {
 	private IConfig $config;
+	private IAvailabilityCoordinator $coordinator;
 
-	public function __construct(IConfig $config) {
+	public function __construct(IConfig $config, IAvailabilityCoordinator $coordinator) {
 		$this->config = $config;
+		$this->coordinator = $coordinator;
 	}
 
 	/**
-	 * @return array{dav: array{chunking: string, bulkupload?: string}}
+	 * @return array{dav: array{chunking: string, bulkupload?: string, absence-supported?: bool}}
 	 */
 	public function getCapabilities() {
 		$capabilities = [
@@ -46,6 +49,9 @@ public function getCapabilities() {
 		if ($this->config->getSystemValueBool('bulkupload.enabled', true)) {
 			$capabilities['dav']['bulkupload'] = '1.0';
 		}
+		if ($this->coordinator->isEnabled()) {
+			$capabilities['dav']['absence-supported'] = true;
+		}
 		return $capabilities;
 	}
 }

apps/dav/openapi.json

@@ -37,6 +37,9 @@
                             },
                             "bulkupload": {
                                 "type": "string"
+                            },
+                            "absence-supported": {
+                                "type": "boolean"
                             }
                         }
                     }

apps/dav/tests/unit/CalDAV/Reminder/NotificationProviderManagerTest.php

@@ -37,6 +37,9 @@
 use OCA\DAV\Capabilities;
 use Test\TestCase;
 
+/**
+ * @group DB
+ */
 class NotificationProviderManagerTest extends TestCase {
 
 	/** @var NotificationProviderManager|\PHPUnit\Framework\MockObject\MockObject */

apps/dav/tests/unit/CapabilitiesTest.php

@@ -25,6 +25,7 @@
 
 use OCA\DAV\Capabilities;
 use OCP\IConfig;
+use OCP\User\IAvailabilityCoordinator;
 use Test\TestCase;
 
 /**
@@ -37,7 +38,11 @@ public function testGetCapabilities(): void {
 			->method('getSystemValueBool')
 			->with('bulkupload.enabled', $this->isType('bool'))
 			->willReturn(false);
-		$capabilities = new Capabilities($config);
+		$coordinator = $this->createMock(IAvailabilityCoordinator::class);
+		$coordinator->expects($this->once())
+			->method('isEnabled')
+			->willReturn(false);
+		$capabilities = new Capabilities($config, $coordinator);
 		$expected = [
 			'dav' => [
 				'chunking' => '1.0',
@@ -52,7 +57,11 @@ public function testGetCapabilitiesWithBulkUpload(): void {
 			->method('getSystemValueBool')
 			->with('bulkupload.enabled', $this->isType('bool'))
 			->willReturn(true);
-		$capabilities = new Capabilities($config);
+		$coordinator = $this->createMock(IAvailabilityCoordinator::class);
+		$coordinator->expects($this->once())
+			->method('isEnabled')
+			->willReturn(false);
+		$capabilities = new Capabilities($config, $coordinator);
 		$expected = [
 			'dav' => [
 				'chunking' => '1.0',
@@ -61,4 +70,24 @@ public function testGetCapabilitiesWithBulkUpload(): void {
 		];
 		$this->assertSame($expected, $capabilities->getCapabilities());
 	}
+
+	public function testGetCapabilitiesWithAbsence(): void {
+		$config = $this->createMock(IConfig::class);
+		$config->expects($this->once())
+			->method('getSystemValueBool')
+			->with('bulkupload.enabled', $this->isType('bool'))
+			->willReturn(false);
+		$coordinator = $this->createMock(IAvailabilityCoordinator::class);
+		$coordinator->expects($this->once())
+			->method('isEnabled')
+			->willReturn(true);
+		$capabilities = new Capabilities($config, $coordinator);
+		$expected = [
+			'dav' => [
+				'chunking' => '1.0',
+				'absence-supported' => true,
+			],
+		];
+		$this->assertSame($expected, $capabilities->getCapabilities());
+	}
 }