mirror of
https://github.com/phpbb/phpbb.git
synced 2025-02-22 19:07:27 +01:00
[feature/avatars] Differentiate tests for get drivers functions
Tests for get_all_drivers() and get_enabled_drivers() should be different. PHPBB3-10018
This commit is contained in:
parent
cb08bf3c0c
commit
caa3516d13
19
tests/avatar/driver/barfoo.php
Normal file
19
tests/avatar/driver/barfoo.php
Normal file
@ -0,0 +1,19 @@
|
||||
<?php
|
||||
|
||||
class phpbb_avatar_driver_barfoo extends phpbb_avatar_driver
|
||||
{
|
||||
public function get_data($row)
|
||||
{
|
||||
return array();
|
||||
}
|
||||
|
||||
public function prepare_form($request, $template, $row, &$error)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
|
||||
public function process_form($request, $template, $row, &$error)
|
||||
{
|
||||
return false;
|
||||
}
|
||||
}
|
@ -26,12 +26,20 @@ class phpbb_avatar_manager_test extends PHPUnit_Framework_TestCase
|
||||
$config = new phpbb_config(array());
|
||||
$request = $this->getMock('phpbb_request');
|
||||
$cache = $this->getMock('phpbb_cache_driver_interface');
|
||||
|
||||
$this->avatar_foobar = $this->getMock('phpbb_avatar_driver_foobar', array('get_name'), array($config, $phpbb_root_path, $phpEx, $cache));
|
||||
$this->avatar_foobar->expects($this->any())
|
||||
->method('get_name')
|
||||
->will($this->returnValue('avatar.driver.foobar'));
|
||||
$avatar_drivers = array($this->avatar_foobar);
|
||||
$this->avatar_barfoo = $this->getMock('phpbb_avatar_driver_barfoo', array('get_name'));
|
||||
$this->avatar_barfoo->expects($this->any())
|
||||
->method('get_name')
|
||||
->will($this->returnValue('avatar.driver.barfoo'));
|
||||
|
||||
$avatar_drivers = array($this->avatar_foobar, $this->avatar_barfoo);
|
||||
|
||||
$config['allow_avatar_' . get_class($this->avatar_foobar)] = true;
|
||||
$config['allow_avatar_' . get_class($this->avatar_barfoo)] = false;
|
||||
|
||||
// Set up avatar manager
|
||||
$this->manager = new phpbb_avatar_manager($config, $avatar_drivers, $this->phpbb_container);
|
||||
@ -56,13 +64,16 @@ class phpbb_avatar_manager_test extends PHPUnit_Framework_TestCase
|
||||
{
|
||||
$drivers = $this->manager->get_all_drivers();
|
||||
$this->assertArrayHasKey('avatar.driver.foobar', $drivers);
|
||||
$this->assertArrayHasKey('avatar.driver.barfoo', $drivers);
|
||||
$this->assertEquals('avatar.driver.foobar', $drivers['avatar.driver.foobar']);
|
||||
$this->assertEquals('avatar.driver.barfoo', $drivers['avatar.driver.barfoo']);
|
||||
}
|
||||
|
||||
public function test_get_enabled_drivers()
|
||||
{
|
||||
$drivers = $this->manager->get_enabled_drivers();
|
||||
$this->assertArrayHasKey('avatar.driver.foobar', $drivers);
|
||||
$this->assertArrayNotHasKey('avatar.driver.barfoo', $drivers);
|
||||
$this->assertEquals('avatar.driver.foobar', $drivers['avatar.driver.foobar']);
|
||||
}
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user