Merge pull request #440 from clue-labs/tests-memory

WyriHaximus · web-flow · commit e27cfcd1cf60 · 2022-02-05T22:44:40.000+01:00
Improve test suite to skip memory tests when lowering memory limit does not work
diff --git a/tests/HttpServerTest.php b/tests/HttpServerTest.php
@@ -404,7 +404,9 @@ public function testConstructServerWithUnlimitedMemoryLimitDoesNotLimitConcurren
     public function testConstructServerWithMemoryLimitDoesLimitConcurrency()
     {
         $old = ini_get('memory_limit');
-        ini_set('memory_limit', '100M');
+        if (@ini_set('memory_limit', '128M') === false) {
+            $this->markTestSkipped('Unable to change memory limit');
+        }
 
         $http = new HttpServer(function () { });
 

Original file line number	Diff line number	Diff line change
`@@ -404,7 +404,9 @@ public function testConstructServerWithUnlimitedMemoryLimitDoesNotLimitConcurren`
`404`	`404`	`public function testConstructServerWithMemoryLimitDoesLimitConcurrency()`
`405`	`405`	`{`
`406`	`406`	`$old = ini_get('memory_limit');`
`407`		`- ini_set('memory_limit', '100M');`
	`407`	`+ if (@ini_set('memory_limit', '128M') === false) {`
	`408`	`+ $this->markTestSkipped('Unable to change memory limit');`
	`409`	`+ }`
`408`	`410`
`409`	`411`	`$http = new HttpServer(function () { });`
`410`	`412`