mirror of
https://github.com/phpbb/phpbb.git
synced 2025-02-24 03:54:10 +01:00
[ticket/11465] Add tests for optional arguments of get_module_infos()
The possibilities of the first argument have already been covered previously. The second argument will be covered with an entry that should exist, an incorrect entry, and the default false entry that should use the previously set module class. Unfortunately, the third argument doesn't have an effect in the tests, as the mocked extension manager will not properly handle enabled/disabled extensions. PHPBB3-11465
This commit is contained in:
parent
6a1fa3e0f4
commit
7327f9326f
@ -97,6 +97,20 @@ class phpbb_extension_modules_test extends phpbb_test_case
|
||||
),
|
||||
), $acp_modules);
|
||||
|
||||
// Find a specific module info file (mcp_a_module) with passing the module_class
|
||||
$this->acp_modules->module_class = '';
|
||||
$acp_modules = $this->acp_modules->get_module_infos('mcp_a_module', 'mcp');
|
||||
$this->assertEquals(array(
|
||||
'phpbb_ext_foo_mcp_a_module' => array(
|
||||
'filename' => 'phpbb_ext_foo_mcp_a_module',
|
||||
'title' => 'Foobar',
|
||||
'version' => '3.1.0-dev',
|
||||
'modes' => array(
|
||||
'config' => array('title' => 'Config', 'auth' => '', 'cat' => array('MCP_MAIN')),
|
||||
),
|
||||
),
|
||||
), $acp_modules);
|
||||
|
||||
// The mcp module info file we're looking for shouldn't exist
|
||||
$this->acp_modules->module_class = 'mcp';
|
||||
$acp_modules = $this->acp_modules->get_module_infos('mcp_a_fail');
|
||||
@ -106,5 +120,51 @@ class phpbb_extension_modules_test extends phpbb_test_case
|
||||
$this->acp_modules->module_class = 'ucp';
|
||||
$acp_modules = $this->acp_modules->get_module_infos();
|
||||
$this->assertEquals(array(), $acp_modules);
|
||||
|
||||
// Get module info of specified extension module
|
||||
$this->acp_modules->module_class = 'acp';
|
||||
$acp_modules = $this->acp_modules->get_module_infos('phpbb_ext_foo_acp_a_module');
|
||||
$this->assertEquals(array(
|
||||
'phpbb_ext_foo_acp_a_module' => array (
|
||||
'filename' => 'phpbb_ext_foo_acp_a_module',
|
||||
'title' => 'Foobar',
|
||||
'version' => '3.1.0-dev',
|
||||
'modes' => array (
|
||||
'config' => array ('title' => 'Config', 'auth' => '', 'cat' => array ('ACP_MODS')),
|
||||
),
|
||||
),
|
||||
), $acp_modules);
|
||||
|
||||
// No specific module and module class set to an incorrect name
|
||||
$acp_modules = $this->acp_modules->get_module_infos('', 'wcp', true);
|
||||
$this->assertEquals(array(), $acp_modules);
|
||||
|
||||
// No specific module, no module_class set in the function parameter, and an incorrect module class
|
||||
$this->acp_modules->module_class = 'wcp';
|
||||
$acp_modules = $this->acp_modules->get_module_infos();
|
||||
$this->assertEquals(array(), $acp_modules);
|
||||
|
||||
// No specific module, module class set to false (will default to the above acp)
|
||||
// Setting $use_all_available will have no effect here as the ext manager is just mocked
|
||||
$this->acp_modules->module_class = 'acp';
|
||||
$acp_modules = $this->acp_modules->get_module_infos('', false, true);
|
||||
$this->assertEquals(array(
|
||||
'phpbb_ext_foo_acp_a_module' => array(
|
||||
'filename' => 'phpbb_ext_foo_acp_a_module',
|
||||
'title' => 'Foobar',
|
||||
'version' => '3.1.0-dev',
|
||||
'modes' => array(
|
||||
'config' => array('title' => 'Config', 'auth' => '', 'cat' => array('ACP_MODS')),
|
||||
),
|
||||
),
|
||||
'acp_foobar' => array(
|
||||
'filename' => 'acp_foobar',
|
||||
'title' => 'ACP Foobar',
|
||||
'version' => '3.1.0-dev',
|
||||
'modes' => array(
|
||||
'test' => array('title' => 'Test', 'auth' => '', 'cat' => array('ACP_GENERAL')),
|
||||
),
|
||||
),
|
||||
), $acp_modules);
|
||||
}
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user