malhuda/CodeIgniter4
1
0
Fork 0
mirror of https://github.com/codeigniter4/CodeIgniter4.git synced 2025-02-20 11:44:28 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

style: break long lines

Browse Source
This commit is contained in:
kenjis 2023-01-31 17:43:32 +09:00
parent 472562d984
commit 46727ea30b
No known key found for this signature in database
GPG Key ID: BD254878922AF198
1 changed files with 47 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

62
tests/system/CodeIgniterTest.php
View File

@ -189,7 +189,10 @@ final class CodeIgniterTest extends CIUnitTestCase
// Inject mock router.
$routes = Services::routes();
$routes->add('pages/(:segment)', static fn ($segment) => 'You want to see "' . esc($segment) . '" page.');
$routes->add(
'pages/(:segment)',
static fn ($segment) => 'You want to see "' . esc($segment) . '" page.'
);
$router = Services::router($routes, Services::incomingrequest());
Services::injectMock('router', $router);
@ -261,7 +264,11 @@ final class CodeIgniterTest extends CIUnitTestCase
// Inject mock router.
$routes = Services::routes();
$routes->add('pages/about', static fn () => Services::incomingrequest()->getBody(), ['filter' => Customfilter::class]);
$routes->add(
'pages/about',
static fn () => Services::incomingrequest()->getBody(),
['filter' => Customfilter::class]
);
$router = Services::router($routes, Services::incomingrequest());
Services::injectMock('router', $router);
@ -743,8 +750,11 @@ final class CodeIgniterTest extends CIUnitTestCase
*
* @see https://github.com/codeigniter4/CodeIgniter4/pull/6410
*/
public function testPageCacheWithCacheQueryString($cacheQueryStringValue, int $expectedPagesInCache, array $testingUrls)
{
public function testPageCacheWithCacheQueryString(
$cacheQueryStringValue,
int $expectedPagesInCache,
array $testingUrls
) {
// Suppress command() output
CITestStreamFilter::$buffer = '';
$outputStreamFilter = stream_filter_append(STDOUT, 'CITestStreamFilter');
@ -766,7 +776,10 @@ final class CodeIgniterTest extends CIUnitTestCase
$_SERVER['REQUEST_URI'] = '/' . $testingUrl;
$routes = Services::routes(true);
$routes->add($testingUrl, static function () {
CodeIgniter::cache(0); // Don't cache the page in the run() function because CodeIgniter class will create default $cacheConfig and overwrite settings from the dataProvider
// Don't cache the page in the run() function because CodeIgniter
// class will create default $cacheConfig and overwrite settings
// from the dataProvider
CodeIgniter::cache(0);
$response = Services::response();
$string = 'This is a test page, to check cache configuration';
@ -777,9 +790,11 @@ final class CodeIgniterTest extends CIUnitTestCase
$router = Services::router($routes, Services::incomingrequest(null, false));
Services::injectMock('router', $router);
// Cache the page output using default caching function and $cacheConfig with value from the data provider
// Cache the page output using default caching function and $cacheConfig
// with value from the data provider
$this->codeigniter->run();
$this->codeigniter->cachePage($cacheConfig); // Cache the page using our own $cacheConfig confugration
// Cache the page using our own $cacheConfig confugration
$this->codeigniter->cachePage($cacheConfig);
}
// Calculate how much cached items exist in the cache after the test requests
@ -800,17 +815,34 @@ final class CodeIgniterTest extends CIUnitTestCase
public function cacheQueryStringProvider(): array
{
$testingUrls = [
'test', // URL #1
'test?important_parameter=1', // URL #2
'test?important_parameter=2', // URL #3
'test?important_parameter=1&not_important_parameter=2', // URL #4
'test?important_parameter=1&not_important_parameter=2&another_not_important_parameter=3', // URL #5
// URL #1
'test',
// URL #2
'test?important_parameter=1',
// URL #3
'test?important_parameter=2',
// URL #4
'test?important_parameter=1&not_important_parameter=2',
// URL #5
'test?important_parameter=1&not_important_parameter=2&another_not_important_parameter=3',
];
return [
'$cacheQueryString=false' => [false, 1, $testingUrls], // We expect only 1 page in the cache, because when cacheQueryString is set to false, all GET parameter should be ignored, and page URI will be absolutely same "/test" string for all 5 requests
'$cacheQueryString=true' => [true, 5, $testingUrls], // We expect all 5 pages in the cache, because when cacheQueryString is set to true, all GET parameter should be processed as unique requests
'$cacheQueryString=array' => [['important_parameter'], 3, $testingUrls], // We expect only 3 pages in the cache, because when cacheQueryString is set to array with important parameters, we should ignore all parameters thats not in the array. Only URL #1, URL #2 and URL #3 should be cached. URL #4 and URL #5 is duplication of URL #2 (with value ?important_parameter=1), so they should not be processed as new unique requests and application should return already cached page for URL #2
// We expect only 1 page in the cache, because when cacheQueryString
// is set to false, all GET parameter should be ignored, and page URI
// will be absolutely same "/test" string for all 5 requests
'$cacheQueryString=false' => [false, 1, $testingUrls],
// We expect all 5 pages in the cache, because when cacheQueryString
// is set to true, all GET parameter should be processed as unique requests
'$cacheQueryString=true' => [true, 5, $testingUrls],
// We expect only 3 pages in the cache, because when cacheQueryString
// is set to array with important parameters, we should ignore all
// parameters thats not in the array. Only URL #1, URL #2 and URL #3
// should be cached. URL #4 and URL #5 is duplication of URL #2
// (with value ?important_parameter=1), so they should not be processed
// as new unique requests and application should return already cached
// page for URL #2
'$cacheQueryString=array' => [['important_parameter'], 3, $testingUrls],
];
}
}