fix burn testing /wo leaks

php-lock · Dec 5, 2024 · 7151f9f · 7151f9f
1 parent 0c3cbab
commit 7151f9f
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 39 deletions.
diff --git a/tests/mutex/CASMutexTest.php b/tests/mutex/CASMutexTest.php
@@ -7,7 +7,6 @@
 use malkusch\lock\exception\LockAcquireException;
 use malkusch\lock\mutex\CASMutex;
 use phpmock\environment\SleepEnvironmentBuilder;
-use phpmock\MockEnabledException;
 use phpmock\phpunit\PHPMock;
 use PHPUnit\Framework\TestCase;
 
@@ -20,17 +19,14 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $sleepBuilder = new SleepEnvironmentBuilder();
-        $sleepBuilder->addNamespace(__NAMESPACE__);
-        $sleepBuilder->addNamespace('malkusch\lock\mutex');
-        $sleepBuilder->addNamespace('malkusch\lock\util');
-        $sleep = $sleepBuilder->build();
-        try {
+        if (!function_exists(__NAMESPACE__ . '\microtime')) { // workaround for burn testing
+            $sleepBuilder = new SleepEnvironmentBuilder();
+            $sleepBuilder->addNamespace(__NAMESPACE__);
+            $sleepBuilder->addNamespace('malkusch\lock\mutex');
+            $sleepBuilder->addNamespace('malkusch\lock\util');
+            $sleep = $sleepBuilder->build();
             $sleep->enable();
             $this->registerForTearDown($sleep);
-        } catch (MockEnabledException $e) {
-            // workaround for burn testing
-            self::assertSame('microtime is already enabled.Call disable() on the existing mock.', $e->getMessage());
         }
     }
 

diff --git a/tests/mutex/RedisMutexTest.php b/tests/mutex/RedisMutexTest.php
@@ -10,7 +10,6 @@
 use malkusch\lock\exception\TimeoutException;
 use malkusch\lock\mutex\RedisMutex;
 use phpmock\environment\SleepEnvironmentBuilder;
-use phpmock\MockEnabledException;
 use phpmock\phpunit\PHPMock;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\Attributes\Group;
@@ -30,17 +29,14 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $sleepBuilder = new SleepEnvironmentBuilder();
-        $sleepBuilder->addNamespace(__NAMESPACE__);
-        $sleepBuilder->addNamespace('malkusch\lock\mutex');
-        $sleepBuilder->addNamespace('malkusch\lock\util');
-        $sleep = $sleepBuilder->build();
-        try {
+        if (!function_exists(__NAMESPACE__ . '\microtime')) { // workaround for burn testing
+            $sleepBuilder = new SleepEnvironmentBuilder();
+            $sleepBuilder->addNamespace(__NAMESPACE__);
+            $sleepBuilder->addNamespace('malkusch\lock\mutex');
+            $sleepBuilder->addNamespace('malkusch\lock\util');
+            $sleep = $sleepBuilder->build();
             $sleep->enable();
             $this->registerForTearDown($sleep);
-        } catch (MockEnabledException $e) {
-            // workaround for burn testing
-            self::assertSame('microtime is already enabled.Call disable() on the existing mock.', $e->getMessage());
         }
     }
 

diff --git a/tests/mutex/SpinlockMutexTest.php b/tests/mutex/SpinlockMutexTest.php
@@ -10,7 +10,6 @@
 use malkusch\lock\exception\TimeoutException;
 use malkusch\lock\mutex\SpinlockMutex;
 use phpmock\environment\SleepEnvironmentBuilder;
-use phpmock\MockEnabledException;
 use phpmock\phpunit\PHPMock;
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
@@ -24,17 +23,14 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $sleepBuilder = new SleepEnvironmentBuilder();
-        $sleepBuilder->addNamespace(__NAMESPACE__);
-        $sleepBuilder->addNamespace('malkusch\lock\mutex');
-        $sleepBuilder->addNamespace('malkusch\lock\util');
-        $sleep = $sleepBuilder->build();
-        try {
+        if (!function_exists(__NAMESPACE__ . '\microtime')) { // workaround for burn testing
+            $sleepBuilder = new SleepEnvironmentBuilder();
+            $sleepBuilder->addNamespace(__NAMESPACE__);
+            $sleepBuilder->addNamespace('malkusch\lock\mutex');
+            $sleepBuilder->addNamespace('malkusch\lock\util');
+            $sleep = $sleepBuilder->build();
             $sleep->enable();
             $this->registerForTearDown($sleep);
-        } catch (MockEnabledException $e) {
-            // workaround for burn testing
-            self::assertSame('microtime is already enabled.Call disable() on the existing mock.', $e->getMessage());
         }
     }
 

diff --git a/tests/util/LoopTest.php b/tests/util/LoopTest.php
@@ -7,7 +7,6 @@
 use malkusch\lock\exception\TimeoutException;
 use malkusch\lock\util\Loop;
 use phpmock\environment\SleepEnvironmentBuilder;
-use phpmock\MockEnabledException;
 use phpmock\phpunit\PHPMock;
 use PHPUnit\Framework\TestCase;
 
@@ -20,16 +19,13 @@ protected function setUp(): void
     {
         parent::setUp();
 
-        $sleepBuilder = new SleepEnvironmentBuilder();
-        $sleepBuilder->addNamespace(__NAMESPACE__);
-        $sleepBuilder->addNamespace('malkusch\lock\util');
-        $sleep = $sleepBuilder->build();
-        try {
+        if (!function_exists(__NAMESPACE__ . '\microtime')) { // workaround for burn testing
+            $sleepBuilder = new SleepEnvironmentBuilder();
+            $sleepBuilder->addNamespace(__NAMESPACE__);
+            $sleepBuilder->addNamespace('malkusch\lock\util');
+            $sleep = $sleepBuilder->build();
             $sleep->enable();
             $this->registerForTearDown($sleep);
-        } catch (MockEnabledException $e) {
-            // workaround for burn testing
-            self::assertSame('microtime is already enabled.Call disable() on the existing mock.', $e->getMessage());
         }
     }