test: integrierte Freigabepruefungen stabilisieren

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(
+            substr_count($inhalt, '$this->geraetezugangBenoetigt()'),
-                $inhalt,
+            'Der Geraetezugang muss beim Anwenden, Erneuern und Anfordern beruecksichtigt werden.'
-                "|| \$this->ReadPropertyBoolean('StoerueberwachungAktiv'))"
-            ),
-            'Der Gerätezugang muss beim Anwenden, Erneuern und Anfordern berücksichtigt werden.'
         );
     }
 

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)
         ));