makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-08 12:28:52 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ticket/14168] Add more test cases for attachment manager

Browse source 
PHPBB3-14168
This commit is contained in:
Marc Alexander 2015-10-09 14:33:12 +02:00
parent b90783a296
commit 52dccd3dba
1 changed files with 50 additions and 31 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

81
tests/attachment/manager_test.php
View file

@ -38,35 +38,6 @@ class phpbb_attachment_manager_test extends \phpbb_test_case
return new \phpbb\attachment\manager($this->delete, $this->resync, $this->upload); return new \phpbb\attachment\manager($this->delete, $this->resync, $this->upload);
} }
public function data_delete()
{
return array(
[
['foo', [1, 2, 3], false],
['foo', [1, 2, 3], false],
true,
],
[
['foo', [1, 2, 3], true],
['foo', [1, 2, 3]],
true,
],
);
}
/**
* @dataProvider data_delete
*/
public function test_delete($input, $input_manager, $output)
{
$mock = $this->delete->expects($this->atLeastOnce())
->method('delete');
$mock = call_user_func_array([$mock, 'with'], $input);
$mock->willReturn($output);
$manager = $this->get_manager();
$this->assertSame($output, call_user_func_array([$manager, 'delete'], $input_manager));
}
public function data_manager() public function data_manager()
{ {
return array( return array(
@ -77,6 +48,7 @@ class phpbb_attachment_manager_test extends \phpbb_test_case
['foo'], ['foo'],
['foo', 'file', false], ['foo', 'file', false],
true, true,
true,
), ),
array( array(
'delete', 'delete',
@ -85,6 +57,7 @@ class phpbb_attachment_manager_test extends \phpbb_test_case
['foo', 'bar'], ['foo', 'bar'],
['foo', 'bar', false], ['foo', 'bar', false],
true, true,
true,
), ),
array( array(
'delete', 'delete',
@ -93,6 +66,52 @@ class phpbb_attachment_manager_test extends \phpbb_test_case
['foo', 'bar', true], ['foo', 'bar', true],
['foo', 'bar', true], ['foo', 'bar', true],
true, true,
true,
),
array(
'delete',
'delete',
'delete',
['foo', [1, 2, 3]],
['foo', [1, 2, 3], true],
true,
true,
),
array(
'delete',
'delete',
'delete',
['foo', [1, 2, 3], false],
['foo', [1, 2, 3], false],
true,
true,
),
array(
'resync',
'resync',
'resync',
['foo', [1, 2, 3]],
['foo', [1, 2, 3]],
true,
null,
),
array(
'upload',
'upload',
'upload',
['foo', 1],
['foo', 1, false, '', false, []],
true,
true,
),
array(
'upload',
'upload',
'upload',
['foo', 1, true, 'bar', true, ['filename' => 'foobar']],
['foo', 1, true, 'bar', true, ['filename' => 'foobar']],
true,
true,
), ),
); );
} }
@ -100,12 +119,12 @@ class phpbb_attachment_manager_test extends \phpbb_test_case
/** /**
* @dataProvider data_manager * @dataProvider data_manager
*/ */
public function test_manager($class, $method_class, $method_manager, $input_manager, $input_method, $output) public function test_manager($class, $method_class, $method_manager, $input_manager, $input_method, $return, $output)
{ {
$mock = call_user_func_array([$this->{$class}, 'expects'], [$this->atLeastOnce()]); $mock = call_user_func_array([$this->{$class}, 'expects'], [$this->atLeastOnce()]);
$mock = $mock->method($method_class); $mock = $mock->method($method_class);
$mock = call_user_func_array([$mock, 'with'], $input_method); $mock = call_user_func_array([$mock, 'with'], $input_method);
$mock->willReturn($output); $mock->willReturn($return);
$manager = $this->get_manager(); $manager = $this->get_manager();
$this->assertSame($output, call_user_func_array([$manager, $method_manager], $input_manager)); $this->assertSame($output, call_user_func_array([$manager, $method_manager], $input_manager));
} }