Skip to content

Commit a988128

Browse files
committed
test: replace Services::request() with Services::incomingrequest()
These tests are for web tests.
1 parent 9cbf480 commit a988128

1 file changed

Lines changed: 21 additions & 21 deletions

File tree

tests/system/CodeIgniterTest.php

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public function testRunClosureRoute()
7878
$routes->add('pages/(:segment)', static function ($segment) {
7979
echo 'You want to see "' . esc($segment) . '" page.';
8080
});
81-
$router = Services::router($routes, Services::request());
81+
$router = Services::router($routes, Services::incomingrequest());
8282
Services::injectMock('router', $router);
8383

8484
ob_start();
@@ -97,7 +97,7 @@ public function testRun404Override()
9797
$routes = Services::routes();
9898
$routes->setAutoRoute(false);
9999
$routes->set404Override('Tests\Support\Controllers\Hello::index');
100-
$router = Services::router($routes, Services::request());
100+
$router = Services::router($routes, Services::incomingrequest());
101101
Services::injectMock('router', $router);
102102

103103
ob_start();
@@ -116,7 +116,7 @@ public function testRun404OverrideControllerReturnsResponse()
116116
$routes = Services::routes();
117117
$routes->setAutoRoute(false);
118118
$routes->set404Override('Tests\Support\Controllers\Popcorn::pop');
119-
$router = Services::router($routes, Services::request());
119+
$router = Services::router($routes, Services::incomingrequest());
120120
Services::injectMock('router', $router);
121121

122122
ob_start();
@@ -137,7 +137,7 @@ public function testRun404OverrideByClosure()
137137
$routes->set404Override(static function () {
138138
echo '404 Override by Closure.';
139139
});
140-
$router = Services::router($routes, Services::request());
140+
$router = Services::router($routes, Services::incomingrequest());
141141
Services::injectMock('router', $router);
142142

143143
ob_start();
@@ -157,7 +157,7 @@ public function testControllersCanReturnString()
157157
// Inject mock router.
158158
$routes = Services::routes();
159159
$routes->add('pages/(:segment)', static fn ($segment) => 'You want to see "' . esc($segment) . '" page.');
160-
$router = Services::router($routes, Services::request());
160+
$router = Services::router($routes, Services::incomingrequest());
161161
Services::injectMock('router', $router);
162162

163163
ob_start();
@@ -182,7 +182,7 @@ public function testControllersCanReturnResponseObject()
182182

183183
return $response->setBody($string);
184184
});
185-
$router = Services::router($routes, Services::request());
185+
$router = Services::router($routes, Services::incomingrequest());
186186
Services::injectMock('router', $router);
187187

188188
ob_start();
@@ -209,7 +209,7 @@ public function testControllersCanReturnDownloadResponseObject()
209209

210210
return $response->download('some.txt', 'some text', true);
211211
});
212-
$router = Services::router($routes, Services::request());
212+
$router = Services::router($routes, Services::incomingrequest());
213213
Services::injectMock('router', $router);
214214

215215
ob_start();
@@ -228,9 +228,9 @@ public function testControllersRunFilterByClassName()
228228

229229
// Inject mock router.
230230
$routes = Services::routes();
231-
$routes->add('pages/about', static fn () => Services::request()->getBody(), ['filter' => Customfilter::class]);
231+
$routes->add('pages/about', static fn () => Services::incomingrequest()->getBody(), ['filter' => Customfilter::class]);
232232

233-
$router = Services::router($routes, Services::request());
233+
$router = Services::router($routes, Services::incomingrequest());
234234
Services::injectMock('router', $router);
235235

236236
ob_start();
@@ -258,7 +258,7 @@ public function testRoutesIsEmpty()
258258
$_SERVER['argc'] = 2;
259259

260260
// Inject mock router.
261-
$router = Services::router(null, Services::request(), false);
261+
$router = Services::router(null, Services::incomingrequest(), false);
262262
Services::injectMock('router', $router);
263263

264264
ob_start();
@@ -335,7 +335,7 @@ public function testRunRedirectionWithNamed()
335335
}, ['as' => 'name']);
336336
$routes->addRedirect('example', 'name');
337337

338-
$router = Services::router($routes, Services::request());
338+
$router = Services::router($routes, Services::incomingrequest());
339339
Services::injectMock('router', $router);
340340

341341
ob_start();
@@ -358,7 +358,7 @@ public function testRunRedirectionWithURI()
358358
});
359359
$routes->addRedirect('example', 'pages/uri');
360360

361-
$router = Services::router($routes, Services::request());
361+
$router = Services::router($routes, Services::incomingrequest());
362362
Services::injectMock('router', $router);
363363

364364
ob_start();
@@ -382,7 +382,7 @@ public function testRunRedirectionWithURINotSet()
382382
$routes = Services::routes();
383383
$routes->addRedirect('example', 'pages/notset');
384384

385-
$router = Services::router($routes, Services::request());
385+
$router = Services::router($routes, Services::incomingrequest());
386386
Services::injectMock('router', $router);
387387

388388
ob_start();
@@ -405,7 +405,7 @@ public function testRunRedirectionWithHTTPCode303()
405405
$routes = Services::routes();
406406
$routes->addRedirect('example', 'pages/notset', 301);
407407

408-
$router = Services::router($routes, Services::request());
408+
$router = Services::router($routes, Services::incomingrequest());
409409
Services::injectMock('router', $router);
410410

411411
ob_start();
@@ -422,7 +422,7 @@ public function testStoresPreviousURL()
422422
$_SERVER['argc'] = 2;
423423

424424
// Inject mock router.
425-
$router = Services::router(null, Services::request(), false);
425+
$router = Services::router(null, Services::incomingrequest(), false);
426426
Services::injectMock('router', $router);
427427

428428
ob_start();
@@ -446,7 +446,7 @@ public function testNotStoresPreviousURL()
446446
$routes = Services::routes();
447447
$routes->addRedirect('example', 'pages/notset', 301);
448448

449-
$router = Services::router($routes, Services::request());
449+
$router = Services::router($routes, Services::incomingrequest());
450450
Services::injectMock('router', $router);
451451

452452
ob_start();
@@ -470,7 +470,7 @@ public function testNotStoresPreviousURLByCheckingContentType()
470470

471471
return $response->setContentType('image/jpeg', '');
472472
});
473-
$router = Services::router($routes, Services::request());
473+
$router = Services::router($routes, Services::incomingrequest());
474474
Services::injectMock('router', $router);
475475

476476
ob_start();
@@ -537,7 +537,7 @@ public function testSpoofRequestMethodCanUsePUT()
537537
$this->codeigniter->useSafeOutput(true)->run();
538538
ob_get_clean();
539539

540-
$this->assertSame('put', Services::request()->getMethod());
540+
$this->assertSame('put', Services::incomingrequest()->getMethod());
541541
}
542542

543543
public function testSpoofRequestMethodCannotUseGET()
@@ -561,7 +561,7 @@ public function testSpoofRequestMethodCannotUseGET()
561561
$this->codeigniter->useSafeOutput(true)->run();
562562
ob_get_clean();
563563

564-
$this->assertSame('post', Services::request()->getMethod());
564+
$this->assertSame('post', Services::incomingrequest()->getMethod());
565565
}
566566

567567
/**
@@ -588,7 +588,7 @@ public function testPageCacheSendSecureHeaders()
588588

589589
return $response->setBody($string);
590590
});
591-
$router = Services::router($routes, Services::request());
591+
$router = Services::router($routes, Services::incomingrequest());
592592
Services::injectMock('router', $router);
593593

594594
/** @var Filters $filterConfig */
@@ -662,7 +662,7 @@ public function testPageCacheWithCacheQueryString($cacheQueryStringValue, int $e
662662
});
663663

664664
// Inject router
665-
$router = Services::router($routes, Services::request(null, false));
665+
$router = Services::router($routes, Services::incomingrequest(null, false));
666666
Services::injectMock('router', $router);
667667

668668
// Cache the page output using default caching function and $cacheConfig with value from the data provider

0 commit comments

Comments
 (0)