Skip to content

Commit 881f685

Browse files
committed
test cleanup
1 parent 6b7e471 commit 881f685

File tree

1 file changed

+18
-13
lines changed

1 file changed

+18
-13
lines changed

tests/Plugin/AddPathPluginTest.php

Lines changed: 18 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -27,17 +27,22 @@ protected function setUp()
2727
$this->plugin = new AddPathPlugin(new Uri('/api'));
2828
}
2929

30-
public function testRewriteSameObject()
30+
public function testRewriteSameUrl()
3131
{
32-
$request = new Request('GET', 'https://example.com/foo');
33-
$this->plugin->handleRequest($request, function (RequestInterface $request) {
32+
$verify = function (RequestInterface $request) {
3433
$this->assertEquals('https://example.com/api/foo', $request->getUri()->__toString());
35-
}, $this->callable);
34+
};
35+
36+
37+
$request = new Request('GET', 'https://example.com/foo');
38+
$this->plugin->handleRequest($request, $verify, $this->callable);
3639

3740
// Make a second call with the same $request object
38-
$this->plugin->handleRequest($request, function (RequestInterface $request) {
39-
$this->assertEquals('https://example.com/api/foo', $request->getUri()->__toString());
40-
}, $this->callable);
41+
$this->plugin->handleRequest($request, $verify, $this->callable);
42+
43+
// Make a new call with a new object but same URL
44+
$request = new Request('GET', 'https://example.com/foo');
45+
$this->plugin->handleRequest($request, $verify, $this->callable);
4146
}
4247

4348
public function testRewriteOnRestart()
@@ -68,14 +73,14 @@ public function testRewriteWithDifferentUrl()
6873

6974
public function testRewriteWithDifferentUrlWhenSecondUrlIncludesAddedPath()
7075
{
71-
$request = new Request('GET', 'https://example.com/foo');
72-
$this->plugin->handleRequest($request, function (RequestInterface $request) {
76+
$verify = function (RequestInterface $request) {
7377
$this->assertEquals('https://example.com/api/foo', $request->getUri()->__toString());
74-
}, $this->callable);
78+
};
79+
80+
$request = new Request('GET', 'https://example.com/foo');
81+
$this->plugin->handleRequest($request, $verify, $this->callable);
7582

7683
$request = new Request('GET', 'https://example.com/api/foo');
77-
$this->plugin->handleRequest($request, function (RequestInterface $request) {
78-
$this->assertEquals('https://example.com/api/foo', $request->getUri()->__toString());
79-
}, $this->callable);
84+
$this->plugin->handleRequest($request, $verify, $this->callable);
8085
}
8186
}

0 commit comments

Comments
 (0)