From c9a632f3a9ba67e89d42b72ac0d587350ba87e02 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien.fastre@champs-libres.coop>
Date: Thu, 2 Apr 2026 14:35:20 +0200
Subject: [PATCH] Add LockTimeoutAnalyzer utility and corresponding tests

- Implemented `LockTimeoutAnalyzer` to parse timeout values from RFC-compliant strings and return `DateInterval` objects.
- Added `LockTimeoutAnalyzerTest` with data providers to validate handling of various timeout cases, including "Second" and "Infinite".
---
 .../Dav/Utils/LockTimeoutAnalyzer.php         | 45 ++++++++++++++
 .../Dav/Utils/LockTimeoutAnalyzerTest.php     | 62 +++++++++++++++++++
 2 files changed, 107 insertions(+)
 create mode 100644 src/Bundle/ChillDocStoreBundle/Dav/Utils/LockTimeoutAnalyzer.php
 create mode 100644 src/Bundle/ChillDocStoreBundle/Tests/Dav/Utils/LockTimeoutAnalyzerTest.php

diff --git a/src/Bundle/ChillDocStoreBundle/Dav/Utils/LockTimeoutAnalyzer.php b/src/Bundle/ChillDocStoreBundle/Dav/Utils/LockTimeoutAnalyzer.php
new file mode 100644
index 000000000..2c191d923
--- /dev/null
+++ b/src/Bundle/ChillDocStoreBundle/Dav/Utils/LockTimeoutAnalyzer.php
@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\DocStoreBundle\Dav\Utils;
+
+use DateInterval;
+
+class LockTimeoutAnalyzer
+{
+    /**
+     * Analyzes the timeout value from the provided content string, RFC2068 string
+     * and return a DateInterval object representing the timeout duration.
+     *
+     * @param string $content The input string containing timeout information, as described by RFC2518, section 4.2
+     *
+     * @return \DateInterval the calculated timeout as a DateInterval object
+     *
+     * @throws \Exception if the DateInterval creation fails
+     */
+    public function analyzeTimeout(string $content): \DateInterval
+    {
+        $types = explode(',', $content);
+        $firstType = trim(reset($types));
+
+        if (str_starts_with($firstType, 'Second-')) {
+            $seconds = (int) substr($firstType, 7);
+
+            return new \DateInterval(sprintf('PT%dS', $seconds));
+        }
+
+        if ('Infinite' === $firstType) {
+            return new \DateInterval('PT24H');
+        }
+
+        return new \DateInterval('PT3600S');
+    }
+}
diff --git a/src/Bundle/ChillDocStoreBundle/Tests/Dav/Utils/LockTimeoutAnalyzerTest.php b/src/Bundle/ChillDocStoreBundle/Tests/Dav/Utils/LockTimeoutAnalyzerTest.php
new file mode 100644
index 000000000..9d4e049f9
--- /dev/null
+++ b/src/Bundle/ChillDocStoreBundle/Tests/Dav/Utils/LockTimeoutAnalyzerTest.php
@@ -0,0 +1,62 @@
+<?php
+
+declare(strict_types=1);
+
+/*
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace Chill\DocStoreBundle\Tests\Dav\Utils;
+
+use Chill\DocStoreBundle\Dav\Utils\LockTimeoutAnalyzer;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * @internal
+ *
+ * @coversNothing
+ */
+class LockTimeoutAnalyzerTest extends TestCase
+{
+    /**
+     * @dataProvider provideTimeoutData
+     */
+    public function testAnalyzeTimeout(string $content, \DateInterval $expected): void
+    {
+        $analyzer = new LockTimeoutAnalyzer();
+        $result = $analyzer->analyzeTimeout($content);
+
+        self::assertEquals($expected, $result);
+    }
+
+    public function provideTimeoutData(): iterable
+    {
+        yield 'Second-1800' => [
+            'Second-1800',
+            new \DateInterval('PT1800S'),
+        ];
+
+        yield 'Second-3600' => [
+            'Second-3600',
+            new \DateInterval('PT3600S'),
+        ];
+
+        yield 'Infinite' => [
+            'Infinite',
+            new \DateInterval('PT24H'), // Typically "Infinite" is represented by a long duration or special handling; RFC says it should be long. Let's assume a long one for now or check common practices.
+        ];
+
+        yield 'Multiple types, first is Second' => [
+            'Second-1800, Infinite',
+            new \DateInterval('PT1800S'),
+        ];
+
+        yield 'Multiple types, first is Infinite' => [
+            'Infinite, Second-1800',
+            new \DateInterval('PT24H'),
+        ];
+    }
+}