Skip to content

Commit 7a5a728

Browse files
committed
Common test helpers moved to base TestCase
1 parent 6170cff commit 7a5a728

6 files changed

Lines changed: 68 additions & 48 deletions

File tree

composer.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,9 @@
1212
"autoload": {
1313
"psr-4": { "React\\Dns\\": "src" }
1414
},
15+
"autoload-dev": {
16+
"psr-4": { "React\\Tests\\Dns\\": "tests" }
17+
},
1518
"extra": {
1619
"branch-alias": {
1720
"dev-master": "0.4-dev"

tests/FunctionalResolverTest.php

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,12 @@
22

33
namespace React\Tests\Dns;
44

5+
use React\Tests\Dns\TestCase;
56
use React\EventLoop\Factory as LoopFactory;
67
use React\Dns\Resolver\Resolver;
78
use React\Dns\Resolver\Factory;
89

9-
class FunctionalTest extends \PHPUnit_Framework_TestCase
10+
class FunctionalTest extends TestCase
1011
{
1112
public function setUp()
1213
{
@@ -31,29 +32,4 @@ public function testResolveInvalidRejects()
3132

3233
$this->loop->run();
3334
}
34-
35-
protected function expectCallableOnce()
36-
{
37-
$mock = $this->createCallableMock();
38-
$mock
39-
->expects($this->once())
40-
->method('__invoke');
41-
42-
return $mock;
43-
}
44-
45-
protected function expectCallableNever()
46-
{
47-
$mock = $this->createCallableMock();
48-
$mock
49-
->expects($this->never())
50-
->method('__invoke');
51-
52-
return $mock;
53-
}
54-
55-
protected function createCallableMock()
56-
{
57-
return $this->getMock('React\Tests\Dns\CallableStub');
58-
}
5935
}

tests/Query/CachedExecutorTest.php

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,14 @@
22

33
namespace React\Tests\Dns\Query;
44

5+
use React\Tests\Dns\TestCase;
56
use React\Dns\Query\CachedExecutor;
67
use React\Dns\Query\Query;
78
use React\Dns\Model\Message;
89
use React\Dns\Model\Record;
910
use React\Promise;
1011

11-
class CachedExecutorTest extends \PHPUnit_Framework_TestCase
12+
class CachedExecutorTest extends TestCase
1213
{
1314
/**
1415
* @covers React\Dns\Query\CachedExecutor
@@ -87,21 +88,6 @@ private function callQueryCallbackWithAddress($address)
8788
});
8889
}
8990

90-
protected function expectCallableOnce()
91-
{
92-
$mock = $this->createCallableMock();
93-
$mock
94-
->expects($this->once())
95-
->method('__invoke');
96-
97-
return $mock;
98-
}
99-
100-
protected function createCallableMock()
101-
{
102-
return $this->getMock('React\Tests\Socket\Stub\CallableStub');
103-
}
104-
10591
private function createExecutorMock()
10692
{
10793
return $this->getMock('React\Dns\Query\ExecutorInterface');

tests/Query/RetryExecutorTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,15 @@
22

33
namespace React\Tests\Dns\Query;
44

5+
use React\Tests\Dns\TestCase;
56
use React\Dns\Query\RetryExecutor;
67
use React\Dns\Query\Query;
78
use React\Dns\Model\Message;
89
use React\Dns\Query\TimeoutException;
910
use React\Dns\Model\Record;
1011
use React\Promise;
1112

12-
class RetryExecutorTest extends \PHPUnit_Framework_TestCase
13+
class RetryExecutorTest extends TestCase
1314
{
1415
/**
1516
* @covers React\Dns\Query\RetryExecutor

tests/Resolver/FactoryTest.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,9 @@
33
namespace React\Tests\Dns\Resolver;
44

55
use React\Dns\Resolver\Factory;
6+
use React\Tests\Dns\TestCase;
67

7-
class FactoryTest extends \PHPUnit_Framework_TestCase
8+
class FactoryTest extends TestCase
89
{
910
/** @test */
1011
public function createShouldCreateResolver()

tests/bootstrap.php

Lines changed: 57 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,60 @@
11
<?php
22

3-
$loader = @include __DIR__ . '/../vendor/autoload.php';
4-
if (!$loader) {
5-
$loader = require __DIR__ . '/../../../../vendor/autoload.php';
3+
namespace React\Tests\Dns;
4+
5+
use React\Promise\Deferred;
6+
7+
require __DIR__ . '/../vendor/autoload.php';
8+
9+
class TestCase extends \PHPUnit_Framework_TestCase
10+
{
11+
protected function expectCallableOnce()
12+
{
13+
$mock = $this->createCallableMock();
14+
$mock
15+
->expects($this->once())
16+
->method('__invoke');
17+
18+
return $mock;
19+
}
20+
21+
protected function expectCallableOnceWith($value)
22+
{
23+
$mock = $this->createCallableMock();
24+
$mock
25+
->expects($this->once())
26+
->method('__invoke')
27+
->with($this->equalTo($value));
28+
29+
return $mock;
30+
}
31+
32+
protected function expectCallableNever()
33+
{
34+
$mock = $this->createCallableMock();
35+
$mock
36+
->expects($this->never())
37+
->method('__invoke');
38+
39+
return $mock;
40+
}
41+
42+
protected function createCallableMock()
43+
{
44+
return $this->getMock('React\Tests\Dns\CallableStub');
45+
}
46+
47+
protected function createPromiseResolved($value = null)
48+
{
49+
$deferred = new Deferred();
50+
$deferred->resolve($value);
51+
return $deferred->promise();
52+
}
53+
54+
protected function createPromiseRejected($value = null)
55+
{
56+
$deferred = new Deferred();
57+
$deferred->reject($value);
58+
return $deferred->promise();
59+
}
660
}
7-
$loader->addPsr4('React\\Tests\\Dns\\', __DIR__);

0 commit comments

Comments
 (0)