Skip to content

Commit 146edb6

Browse files
authored
Merge pull request #7288 from kenjis/fix-CodeIgniterTest-CITestStreamFilter
test: fix out-of-dated test code in CodeIgniterTest
2 parents 3618fb8 + dc0225e commit 146edb6

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

tests/system/CodeIgniterTest.php

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -709,9 +709,9 @@ public function testPageCacheSendSecureHeaders()
709709
public function testPageCacheWithCacheQueryString($cacheQueryStringValue, int $expectedPagesInCache, array $testingUrls)
710710
{
711711
// Suppress command() output
712-
CITestStreamFilter::$buffer = '';
713-
$outputStreamFilter = stream_filter_append(STDOUT, 'CITestStreamFilter');
714-
$errorStreamFilter = stream_filter_append(STDERR, 'CITestStreamFilter');
712+
CITestStreamFilter::registration();
713+
CITestStreamFilter::addOutputFilter();
714+
CITestStreamFilter::addErrorFilter();
715715

716716
// Create cache config with cacheQueryString value from the dataProvider
717717
$cacheConfig = new Cache();
@@ -756,8 +756,8 @@ public function testPageCacheWithCacheQueryString($cacheQueryStringValue, int $e
756756
$this->assertSame($expectedPagesInCache, $newPagesCached);
757757

758758
// Remove stream filters
759-
stream_filter_remove($outputStreamFilter);
760-
stream_filter_remove($errorStreamFilter);
759+
CITestStreamFilter::removeOutputFilter();
760+
CITestStreamFilter::removeErrorFilter();
761761
}
762762

763763
public function cacheQueryStringProvider(): array

0 commit comments

Comments
 (0)