From fe966516b5a3ab1d6fda96afb8a94213f7069d5d Mon Sep 17 00:00:00 2001
From: dh_Agent <dh@belevo.ch>
Date: Tue, 29 Sep 2026 12:06:51 +0000
Subject: [PATCH] test: integrierte Freigabepruefungen stabilisieren

---
 tests/ManagerModulstrukturTest.php   | 14 ++++++++------
 tests/Symcon/modules/Waermepumpe.php |  9 ++++++---
 2 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/tests/ManagerModulstrukturTest.php b/tests/ManagerModulstrukturTest.php
index 7703d59..7d49efd 100644
--- a/tests/ManagerModulstrukturTest.php
+++ b/tests/ManagerModulstrukturTest.php
@@ -218,9 +218,14 @@ final class ManagerModulstrukturTest extends TestCase
         self::assertStringContainsString("'issueDeviceToken' =>", $inhalt);
         self::assertStringContainsString("unset(\$neueLease['deviceToken'])", $inhalt);
         self::assertStringContainsString(
-            "|| \$this->ReadPropertyBoolean('StoerueberwachungAktiv'))",
+            "|| \$this->ReadPropertyBoolean('StoerueberwachungAktiv')",
             $inhalt
         );
+        self::assertGreaterThanOrEqual(
+            3,
+            substr_count($inhalt, '$this->geraetezugangBenoetigt()'),
+            'Der Geraetezugang muss beim Anwenden, Erneuern und Anfordern beruecksichtigt werden.'
+        );
         self::assertStringContainsString(
             "\$this->ReadPropertyString('Lizenzcode'),\n            \$geraetezugangFehlt",
             $inhalt
@@ -317,11 +322,8 @@ final class ManagerModulstrukturTest extends TestCase
         );
         self::assertGreaterThanOrEqual(
             3,
-            substr_count(
-                $inhalt,
-                "|| \$this->ReadPropertyBoolean('StoerueberwachungAktiv'))"
-            ),
-            'Der Gerätezugang muss beim Anwenden, Erneuern und Anfordern berücksichtigt werden.'
+            substr_count($inhalt, '$this->geraetezugangBenoetigt()'),
+            'Der Geraetezugang muss beim Anwenden, Erneuern und Anfordern beruecksichtigt werden.'
         );
     }
 
diff --git a/tests/Symcon/modules/Waermepumpe.php b/tests/Symcon/modules/Waermepumpe.php
index f987e4b..243064e 100644
--- a/tests/Symcon/modules/Waermepumpe.php
+++ b/tests/Symcon/modules/Waermepumpe.php
@@ -22,7 +22,7 @@ return static function (TestContext $test): void {
             'Wiederholsperre' => 3,
             'Mindestlaufzeit' => 5,
             'Mindestsperrzeit' => 0,
-            'MaximaleSperrzeit' => 2,
+            'MaximaleSperrzeit' => 3,
             'MaximaleSperrzeit24h' => 60,
             'VorgabeTimeout' => 30,
             'DiagnosevariablenAnzeigen' => true,
@@ -136,13 +136,16 @@ return static function (TestContext $test): void {
     ) use ($waermepumpe, $kontakt1, $kontakt2, $senden): void {
         $senden(null, 'Peak');
         $senden(0, 'Peak');
-        $test->waitUntil(static fn (): bool => GetValue($kontakt1) === true);
+        $test->waitUntil(static fn (): bool => GetValue($kontakt1) === true, 2500);
         $test->assertSame(false, GetValue($kontakt2));
         $test->assertSame(true, GetValue(
             $test->objectByIdent('SperreAktiv', $waermepumpe)
         ));
 
-        $test->waitUntil(static fn (): bool => GetValue($kontakt1) === false, 2500);
+        $test->waitUntil(static fn (): bool => GetValue($kontakt1) === false, 3500);
+        $test->waitUntil(static fn (): bool => GetValue(
+            $test->objectByIdent('SperrerholungAktiv', $waermepumpe)
+        ) === true);
         $test->assertSame(true, GetValue(
             $test->objectByIdent('SperrerholungAktiv', $waermepumpe)
         ));