From 2b65156d472d80ec38846ad1fea7d5447b3ec8f4 Mon Sep 17 00:00:00 2001 From: Marc Alexander Date: Mon, 22 Jun 2015 21:18:32 +0200 Subject: [PATCH 1/4] [ticket/634] Add more tests B3P-634 --- tests/unit/event/listener_test.php | 56 ++++++++++++++++++++++------- tests/unit/modules/search_test.php | 39 ++++++++++++++++++++ tests/unit/modules/welcome_test.php | 1 + 3 files changed, 83 insertions(+), 13 deletions(-) create mode 100644 tests/unit/modules/search_test.php diff --git a/tests/unit/event/listener_test.php b/tests/unit/event/listener_test.php index 21d87988..46eee061 100644 --- a/tests/unit/event/listener_test.php +++ b/tests/unit/event/listener_test.php @@ -68,12 +68,12 @@ class listener_test extends \phpbb_template_template_test_case $this->config = new \phpbb\config\config(array( 'enable_mod_rewrite' => '1', - 'board3_portal_enable' => '1', + 'board3_enable' => '1', )); $provider = new \phpbb\controller\provider(); $provider->find_routing_files($finder); $provider->find(dirname(__FILE__) . '/'); - $this->controller_helper = new \phpbb_mock_controller_helper($this->template, $this->user, $this->config, $provider, $manager, new \phpbb\symfony_request($request), $request, new \phpbb\filesystem(), '', 'php', dirname(__FILE__) . '/'); + $this->controller_helper = new mock_controller_helper($this->template, $this->user, $this->config, $provider, $manager, new \phpbb\symfony_request($request), $request, new \phpbb\filesystem(), '', 'php', dirname(__FILE__) . '/'); $this->path_helper = new \phpbb\path_helper( new \phpbb\symfony_request( @@ -85,14 +85,13 @@ class listener_test extends \phpbb_template_template_test_case $this->php_ext ); - $this->auth = new \phpbb\auth\auth(); - $userdata = array( - 'user_id' => 2, - ); - $this->auth->acl($userdata); - // Pretend to allow downloads - $this->auth->acl_options['global']['u_view_portal'] = 0; - $this->auth->acl[0][0] = true; + $this->auth = $this->getMockBuilder('\phpbb\auth\auth') + ->disableOriginalConstructor() + ->getMock(); + $this->auth->expects($this->any()) + ->method('acl_get') + ->with($this->anything()) + ->will($this->returnValue(true)); $this->controller = $this->getMockBuilder('\board3\portal\controller\main') ->disableOriginalConstructor() @@ -160,14 +159,30 @@ class listener_test extends \phpbb_template_template_test_case $this->assertEmpty($result); - $this->user->data['session_page'] = '/app.php/portal'; + $this->controller_helper->set_current_url('/app.php/portal'); $result = $this->phpbb_dispatcher->trigger_event('core.page_header', compact($vars)); $this->assertEmpty($result); // Make sure user shouldn't see link - unset($this->auth->cache[0]['u_view_portal']); - $this->auth->acl[0][0] = false; + $this->config->set('board3_enable', 0); + $this->controller_helper->set_current_url(''); + $result = $this->phpbb_dispatcher->trigger_event('core.page_header', compact($vars)); + + $this->assertEmpty($result); + $this->config->set('board3_enable', 1); + } + + public function test_check_portal_all() + { + $this->phpbb_dispatcher->addListener('core.page_header', array($this->listener, 'add_portal_link')); + + $this->controller_helper->set_current_url(''); + $result = $this->phpbb_dispatcher->trigger_event('core.page_header', compact($vars)); + + $this->assertEmpty($result); + + $this->config->set('board3_show_all_pages', true); $result = $this->phpbb_dispatcher->trigger_event('core.page_header', compact($vars)); $this->assertEmpty($result); @@ -196,3 +211,18 @@ class listener_test extends \phpbb_template_template_test_case )), $result['lang_set_ext']); } } + +class mock_controller_helper extends \phpbb_mock_controller_helper +{ + protected $current_url = ''; + + public function set_current_url($url) + { + $this->current_url = $url; + } + + public function get_current_url() + { + return $this->current_url; + } +} diff --git a/tests/unit/modules/search_test.php b/tests/unit/modules/search_test.php new file mode 100644 index 00000000..42715eeb --- /dev/null +++ b/tests/unit/modules/search_test.php @@ -0,0 +1,39 @@ +template = new \board3\portal\tests\mock\template($this); + $this->search = new \board3\portal\modules\search($this->template, '', ''); + } + + public function test_get_template_side() + { + $this->assertSame('search_side.html', $this->search->get_template_side(5)); + } + + public function test_get_template_acp() + { + $acp_template = $this->search->get_template_acp(5); + $this->assertNotEmpty($acp_template); + $this->assertEmpty($acp_template['vars']); + } +} diff --git a/tests/unit/modules/welcome_test.php b/tests/unit/modules/welcome_test.php index 55b228a7..70e2d272 100644 --- a/tests/unit/modules/welcome_test.php +++ b/tests/unit/modules/welcome_test.php @@ -166,6 +166,7 @@ class phpbb_unit_modules_welcome_test extends \board3\portal\tests\testframework $this->request->overwrite('submit', true, \phpbb\request\request_interface::POST); $this->request->overwrite('welcome_message', ''); $this->setExpectedTriggerError(E_USER_WARNING); + self::$form_key_valid = true; $this->welcome->update_welcome('foobar', 5); } } From 0cc4b4ea0a1be85e8461483e487078fa7d1e8540 Mon Sep 17 00:00:00 2001 From: Marc Alexander Date: Tue, 23 Jun 2015 00:48:52 +0200 Subject: [PATCH 2/4] [ticket/634] Add tests for clock module and improve tests for calendar B3P-634 --- tests/mock/check_form_key.php | 20 ++++++ tests/unit/modules/calendar_test.php | 100 ++++++++++++++++++++++++++- tests/unit/modules/clock_test.php | 57 +++++++++++++++ tests/unit/modules/welcome_test.php | 16 ++--- 4 files changed, 180 insertions(+), 13 deletions(-) create mode 100644 tests/mock/check_form_key.php create mode 100644 tests/unit/modules/clock_test.php diff --git a/tests/mock/check_form_key.php b/tests/mock/check_form_key.php new file mode 100644 index 00000000..7a1406cc --- /dev/null +++ b/tests/mock/check_form_key.php @@ -0,0 +1,20 @@ +createXMLDataSet(dirname(__FILE__) . '/fixtures/configs.xml'); @@ -26,7 +37,7 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor public function setUp() { parent::setUp(); - global $cache, $phpbb_root_path, $phpEx; + global $cache, $phpbb_root_path, $phpEx, $phpbb_dispatcher, $request; $this->path_helper = new \phpbb\path_helper( new \phpbb\symfony_request( @@ -37,11 +48,32 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor $phpbb_root_path, 'php' ); + $db = $this->new_dbal(); + $phpbb_dispatcher = $this->getMockBuilder('\phpbb\event\dispatcher') + ->disableOriginalConstructor() + ->getMock(); + $phpbb_dispatcher->expects($this->any()) + ->method('trigger_event') + ->with($this->anything()) + ->will($this->returnArgument(1)); self::$config = new \phpbb\config\config(array()); + $this->template = new \board3\portal\tests\mock\template($this); $controller_helper = new \board3\portal\tests\mock\controller_helper($phpbb_root_path, $phpEx); $controller_helper->add_route('board3_portal_controller', 'portal'); $modules_helper = new \board3\portal\includes\modules_helper(new \phpbb\auth\auth(), new \phpbb\config\config(array()), $controller_helper, new \phpbb_mock_request()); - $this->calendar = new \board3\portal\modules\calendar(self::$config, $modules_helper, null, null, null, dirname(__FILE__) . '/../../../', 'php', null, $this->path_helper, null); + $request = $this->request = new \phpbb_mock_request(); + $user = new \phpbb\user('\phpbb\datetime'); + $user->timezone = new \DateTimeZone('UTC'); + $user->add_lang('common'); + $log = $this->getMockBuilder('\phpbb\log') + ->setMethods(array('add')) + ->disableOriginalConstructor() + ->getMock(); + $log->expects($this->any()) + ->method('add') + ->with($this->anything()) + ->will($this->returnValue(true)); + $this->calendar = new \board3\portal\modules\calendar(self::$config, $modules_helper, $this->template, $db, $this->request, dirname(__FILE__) . '/../../../', 'php', $user, $this->path_helper, $log); define('PORTAL_MODULES_TABLE', 'phpbb_portal_modules'); define('PORTAL_CONFIG_TABLE', 'phpbb_portal_config'); $cache = $this->getMock('\phpbb\cache\cache', array('destroy', 'sql_exists', 'get', 'put')); @@ -114,6 +146,68 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor $this->assertFalse(isset($portal_config[$key])); } } + + public function test_get_template_side() + { + $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + self::$config->set('board3_sunday_first_5', true); + $this->request->overwrite('m5', 1); + $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + $this->request->overwrite('m5', -1); + $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + } + + public function test_update_events_no_error() + { + $this->calendar->update_events('foobar', 5); + } + + public function test_update_events_form_key_fail() + { + // Save event + check_form_key::$form_key_valid = false; + $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); + $this->setExpectedTriggerError(E_USER_WARNING); + $this->calendar->update_events('foobar', 5); + } + + public function test_update_events_wrong_start_time() + { + // Save event + check_form_key::$form_key_valid = true; + $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); + $this->setExpectedTriggerError(E_USER_WARNING); + $this->calendar->update_events('foobar', 5); + } + + public function test_update_events_wrong_end_time() + { + // Save event + check_form_key::$form_key_valid = true; + $this->request->overwrite('event_start_date', '15.06.2035 13:00'); + $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); + $this->setExpectedTriggerError(E_USER_WARNING); + $this->calendar->update_events('foobar', 5); + } + + public function test_update_events_all_day() + { + // Save event + check_form_key::$form_key_valid = true; + $this->request->overwrite('event_start_date', '15.06.2035 13:00'); + $this->request->overwrite('event_all_day', true); + $this->request->overwrite('event_title', 'foobar'); + $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); + $this->setExpectedTriggerError(E_USER_NOTICE, '

« '); + $this->calendar->update_events('foobar', 5); + } + + public function test_display_events() + { + set_portal_config('board3_calendar_events_5', '[{"title":"foobar","desc":" ","start_time":2065518000,"end_time":"","all_day":true,"permission":"","url":" "}]'); + check_form_key::$form_key_valid = false; + $this->calendar->manage_events('', 'foobar', 5); + } } function set_config($config_name, $config_value, $is_dynamic = false) diff --git a/tests/unit/modules/clock_test.php b/tests/unit/modules/clock_test.php new file mode 100644 index 00000000..192c072f --- /dev/null +++ b/tests/unit/modules/clock_test.php @@ -0,0 +1,57 @@ +template = new \board3\portal\tests\mock\template($this); + $this->config = new \phpbb\config\config(array()); + $this->clock = new \board3\portal\modules\clock($this->config, $this->template); + } + + public function test_get_template_side() + { + $this->assertSame('clock_side.html', $this->clock->get_template_side(5)); + $this->template->assert_same(null, 'B3P_CLOCK_SRC'); + $this->config->set('board3_clock_src_5', 'foobar'); + $this->assertSame('clock_side.html', $this->clock->get_template_side(5)); + $this->template->assert_same('foobar', 'B3P_CLOCK_SRC'); + } + + public function test_get_template_acp() + { + $acp_template = $this->clock->get_template_acp(5); + $this->assertNotEmpty($acp_template); + $this->assertArrayHasKey('board3_clock_src_5', $acp_template['vars']); + } + + public function test_install_uninstall() + { + $this->config->delete('board3_clock_src_5'); + $this->assertTrue($this->clock->install(5)); + $this->assertTrue(isset($this->config['board3_clock_src_5'])); + $this->assertSame('', $this->config['board3_clock_src_5']); + $this->assertTrue($this->clock->uninstall(5, '')); + $this->assertFalse(isset($this->config['board3_clock_src_5'])); + } +} diff --git a/tests/unit/modules/welcome_test.php b/tests/unit/modules/welcome_test.php index 70e2d272..1388f48d 100644 --- a/tests/unit/modules/welcome_test.php +++ b/tests/unit/modules/welcome_test.php @@ -9,6 +9,8 @@ namespace board3\portal\modules; +require_once dirname(__FILE__) . '/../../mock/check_form_key.php'; + class phpbb_unit_modules_welcome_test extends \board3\portal\tests\testframework\database_test_case { protected $path_helper; @@ -28,8 +30,6 @@ class phpbb_unit_modules_welcome_test extends \board3\portal\tests\testframework protected $expected_config = array(); protected $expected_portal_config = array(); - static public $form_key_valid = false; - public function getDataSet() { return $this->createXMLDataSet(dirname(__FILE__) . '/fixtures/configs.xml'); @@ -74,6 +74,7 @@ class phpbb_unit_modules_welcome_test extends \board3\portal\tests\testframework $this->welcome = new \board3\portal\modules\welcome($this->config, $this->request, $this->template, $this->user, $phpbb_root_path, $phpEx); \set_config('foobar', 0, false, $this->config); $this->config->delete('foobar'); + check_form_key::$form_key_valid = false; } public function test_get_template_center() @@ -143,10 +144,10 @@ class phpbb_unit_modules_welcome_test extends \board3\portal\tests\testframework $this->request = new \phpbb_mock_request(); $this->welcome = new \board3\portal\modules\welcome($this->config, $this->request, $this->template, $this->user, '', ''); $this->request->overwrite('submit', true, \phpbb\request\request_interface::POST); - self::$form_key_valid = true; + check_form_key::$form_key_valid = true; $this->request->overwrite('welcome_message', 'foobar101'); $this->welcome->update_welcome('foobar', 5); - self::$form_key_valid = false; + check_form_key::$form_key_valid = false; } public function test_update_welcome_invalid_form_key() @@ -166,12 +167,7 @@ class phpbb_unit_modules_welcome_test extends \board3\portal\tests\testframework $this->request->overwrite('submit', true, \phpbb\request\request_interface::POST); $this->request->overwrite('welcome_message', ''); $this->setExpectedTriggerError(E_USER_WARNING); - self::$form_key_valid = true; + check_form_key::$form_key_valid = true; $this->welcome->update_welcome('foobar', 5); } } - -function check_form_key($value) -{ - return phpbb_unit_modules_welcome_test::$form_key_valid; -} From 66a5d95ed967075a5d9b08503e59fa5c5dffa080 Mon Sep 17 00:00:00 2001 From: Marc Alexander Date: Tue, 23 Jun 2015 12:15:38 +0200 Subject: [PATCH 3/4] [ticket/634] Improve test coverage and fix coding issues B3P-634 --- includes/functions.php | 2 + modules/calendar.php | 11 +- tests/unit/modules/calendar_test.php | 223 +++++++++++++++++++----- tests/unit/modules/fixtures/configs.xml | 10 ++ 4 files changed, 199 insertions(+), 47 deletions(-) diff --git a/includes/functions.php b/includes/functions.php index 81001e34..d049ab01 100644 --- a/includes/functions.php +++ b/includes/functions.php @@ -465,6 +465,8 @@ function get_user_groups() if ($groups_ary === false) { + $groups_ary = array(); + // get user's groups $sql = 'SELECT group_id FROM ' . USER_GROUP_TABLE . ' diff --git a/modules/calendar.php b/modules/calendar.php index 3d9a526c..ebf0491d 100644 --- a/modules/calendar.php +++ b/modules/calendar.php @@ -437,12 +437,12 @@ class calendar extends module_base trigger_error($this->user->lang['FORM_INVALID']. adm_back_link($u_action), E_USER_WARNING); } - $event_title = $this->request->variable('event_title', ' ', true); - $event_desc = $this->request->variable('event_desc', ' ', true); + $event_title = $this->request->variable('event_title', '', true); + $event_desc = $this->request->variable('event_desc', '', true); $event_start_date = trim($this->request->variable('event_start_date', '')); $event_end_date = trim($this->request->variable('event_end_date', '')); $event_all_day = $this->request->variable('event_all_day', false); // default to false - $event_url = $this->request->variable('event_url', ' '); + $event_url = $this->request->variable('event_url', ''); $event_permission = $this->request->variable('permission-setting-calendar', array(0 => '')); $groups_ary = array(); @@ -488,11 +488,6 @@ class calendar extends module_base trigger_error($this->user->lang['NO_EVENT_TITLE'] . adm_back_link($u_action), E_USER_WARNING); } - if (!$start_time || $start_time == 0) - { - trigger_error($this->user->lang['NO_EVENT_START'] . adm_back_link($u_action), E_USER_WARNING); - } - // overwrite already existing events and make sure we don't try to save an event outside of the normal array size of $events if (isset($link_id) && $link_id < sizeof($events)) { diff --git a/tests/unit/modules/calendar_test.php b/tests/unit/modules/calendar_test.php index 6b872e36..1aee2e15 100644 --- a/tests/unit/modules/calendar_test.php +++ b/tests/unit/modules/calendar_test.php @@ -37,7 +37,7 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor public function setUp() { parent::setUp(); - global $cache, $phpbb_root_path, $phpEx, $phpbb_dispatcher, $request; + global $cache, $phpbb_root_path, $phpEx, $phpbb_dispatcher, $request, $user; $this->path_helper = new \phpbb\path_helper( new \phpbb\symfony_request( @@ -57,6 +57,7 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor ->with($this->anything()) ->will($this->returnArgument(1)); self::$config = new \phpbb\config\config(array()); + \set_config('foobar', false, false, self::$config); $this->template = new \board3\portal\tests\mock\template($this); $controller_helper = new \board3\portal\tests\mock\controller_helper($phpbb_root_path, $phpEx); $controller_helper->add_route('board3_portal_controller', 'portal'); @@ -149,12 +150,35 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor public function test_get_template_side() { + set_portal_config('board3_calendar_events_5', '[{"title":"foobar","desc":" ","start_time":' . (time() - 3600) . ',"end_time":"","all_day":true,"permission":"","url":" "},{"title":"foobar","desc":" ","start_time":' . (time() + 90000) . ',"end_time":"","all_day":true,"permission":"","url":" "}]'); $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + self::$config->set('board3_sunday_first_5', true); $this->request->overwrite('m5', 1); $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + $this->request->overwrite('m5', -1); $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + + self::$config->set('board3_display_events_5', true); + $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + $this->assertSame(1, sizeof($this->template->get_row('minical.cur_events'))); + $this->assertSame(1, sizeof($this->template->get_row('minical.upcoming_events'))); + + set_portal_config('board3_calendar_events_5', '[{"title":"foobar","desc":" ","start_time":' . (time() - 10800) . ',"end_time":"","all_day":true,"permission":"","url":"http://example.com"},{"title":"foobar","desc":" ","start_time":' . (time() + 108000) . ',"end_time":"","all_day":true,"permission":"","url":"' . generate_board_url() . '"},{"title":"foobar3","desc":" ","start_time":' . (time() - 90000) . ',"end_time":' . (time() + 90000) . ',"all_day":false,"permission":"","url":" "}]'); + $this->template->delete_var('minical.cur_events'); + $this->template->delete_var('minical.upcoming_events'); + $this->assertSame('calendar_side.html', $this->calendar->get_template_side(5)); + $this->assertSame(2, sizeof($this->template->get_row('minical.cur_events'))); + $this->assertSame(1, sizeof($this->template->get_row('minical.upcoming_events'))); + } + + public function test_get_template_acp() + { + $acp_template = $this->calendar->get_template_acp(5); + $this->assertArrayHasKey('title', $acp_template); + $this->assertArrayHasKey('vars', $acp_template); + $this->assertArrayHasKey('board3_display_events_5', $acp_template['vars']); } public function test_update_events_no_error() @@ -162,51 +186,172 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor $this->calendar->update_events('foobar', 5); } - public function test_update_events_form_key_fail() + public function data_update_events() { - // Save event - check_form_key::$form_key_valid = false; - $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); - $this->setExpectedTriggerError(E_USER_WARNING); - $this->calendar->update_events('foobar', 5); + return array( + array( + array( + 'event_start_date' => date('d.m.Y G:i', time() + 3600 * 3), + 'event_all_day' => true, + 'event_title' => 'foobar', + 'id' => 0, + ), + array( + 'save' => true, + ), + E_USER_NOTICE, + '

« Back to previous page', + '[{"title":"foobar","desc":" ","start_time":' . (time() + 3600) . ',"end_time":"","all_day":true,"permission":"","url":" "}]', + true, + ), + // Form key invalid + array( + array(), + array( + 'save' => true, + ), + E_USER_WARNING, + '', + '', + false, + ), + // Wrong start time + array( + array(), + array( + 'save' => true, + ), + E_USER_WARNING, + '', + '', + true, + ), + // Wrong end time + array( + array( + 'event_start_date' => '15.06.2035 13:00', + ), + array( + 'save' => true, + ), + E_USER_WARNING, + '', + '', + true, + ), + // End time in past + array( + array( + 'event_start_date' => '15.06.2035 13:00', + 'event_end_date' => '15.06.2005 19:00', + ), + array( + 'save' => true, + ), + E_USER_WARNING, + '', + '', + true, + ), + // End time before start + array( + array( + 'event_start_date' => '15.06.2035 13:00', + 'event_end_date' => '15.06.2035 12:00', + ), + array( + 'save' => true, + ), + E_USER_WARNING, + '', + '', + true, + ), + // No event title + array( + array( + 'event_start_date' => date('d.m.Y G:i', time() + 3600 * 3), + 'event_all_day' => true, + ), + array( + 'save' => true, + ), + E_USER_WARNING, + '', + '[{"title":"foobar","desc":" ","start_time":' . (time() + 3600) . ',"end_time":"","all_day":true,"permission":"","url":" "}]', + true, + ), + // Create valid event + array( + array( + 'event_start_date' => date('d.m.Y G:i', time() + 3600 * 3), + 'event_all_day' => true, + 'event_title' => 'foobar', + ), + array( + 'save' => true, + ), + E_USER_NOTICE, + '

« Back to previous page', + '[{"title":"foobar","desc":" ","start_time":' . (time() + 3600) . ',"end_time":"","all_day":true,"permission":"","url":" "}]', + true, + ), + // Display existing events + array( + array(), + array(), + false, + '', + 'board3_calendar_events_5' => '[{"title":"foobar","desc":" ","start_time":2065518000,"end_time":"","all_day":true,"permission":"","url":" "}]', + false, + ), + // Edit event + array( + array( + 'id' => 0, + 'action' => 'edit', + ), + array(), + false, + '', + '[{"title":"foobar","desc":" ","start_time":' . (time() + 3600) . ',"end_time":"","all_day":true,"permission":"","url":" "}]', + true, + ), + ); } - public function test_update_events_wrong_start_time() + /** + * @dataProvider data_update_events + */ + public function test_update_events($get_variables, $post_variables, $expected_error = E_USER_WARNING, $expected_error_message = '', $portal_config = array(), $form_key_valid = false) { - // Save event - check_form_key::$form_key_valid = true; - $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); - $this->setExpectedTriggerError(E_USER_WARNING); - $this->calendar->update_events('foobar', 5); - } + check_form_key::$form_key_valid = $form_key_valid; - public function test_update_events_wrong_end_time() - { - // Save event - check_form_key::$form_key_valid = true; - $this->request->overwrite('event_start_date', '15.06.2035 13:00'); - $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); - $this->setExpectedTriggerError(E_USER_WARNING); - $this->calendar->update_events('foobar', 5); - } + foreach ($get_variables as $key => $value) + { + $this->request->overwrite($key, $value); + } - public function test_update_events_all_day() - { - // Save event - check_form_key::$form_key_valid = true; - $this->request->overwrite('event_start_date', '15.06.2035 13:00'); - $this->request->overwrite('event_all_day', true); - $this->request->overwrite('event_title', 'foobar'); - $this->request->overwrite('save', true, \phpbb\request\request_interface::POST); - $this->setExpectedTriggerError(E_USER_NOTICE, '

« '); - $this->calendar->update_events('foobar', 5); - } + foreach ($post_variables as $key => $value) + { + $this->request->overwrite($key, $value, \phpbb\request\request_interface::POST); + } - public function test_display_events() - { - set_portal_config('board3_calendar_events_5', '[{"title":"foobar","desc":" ","start_time":2065518000,"end_time":"","all_day":true,"permission":"","url":" "}]'); - check_form_key::$form_key_valid = false; - $this->calendar->manage_events('', 'foobar', 5); + set_portal_config('board3_calendar_events_5', $portal_config); + + if ($expected_error !== false) + { + if (!empty($expected_error_message)) + { + $this->setExpectedTriggerError($expected_error, $expected_error_message); + } + else + { + $this->setExpectedTriggerError($expected_error); + } + } + + $this->calendar->update_events('foobar', 5); } } diff --git a/tests/unit/modules/fixtures/configs.xml b/tests/unit/modules/fixtures/configs.xml index 5173d042..e93552ba 100644 --- a/tests/unit/modules/fixtures/configs.xml +++ b/tests/unit/modules/fixtures/configs.xml @@ -8,4 +8,14 @@ config_name config_value + + group_id + group_name + group_desc + + 5 + admins + + +
From 1eb4dc0e8e2d5354584e0deaf53e3f3ce5e0d792 Mon Sep 17 00:00:00 2001 From: Marc Alexander Date: Tue, 23 Jun 2015 12:29:33 +0200 Subject: [PATCH 4/4] [ticket/634] Assert that events row is empty B3P-634 --- tests/unit/modules/calendar_test.php | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/unit/modules/calendar_test.php b/tests/unit/modules/calendar_test.php index 1aee2e15..d17efa9d 100644 --- a/tests/unit/modules/calendar_test.php +++ b/tests/unit/modules/calendar_test.php @@ -184,6 +184,7 @@ class phpbb_unit_modules_calendar_test extends \board3\portal\tests\testframewor public function test_update_events_no_error() { $this->calendar->update_events('foobar', 5); + $this->assertNull($this->template->get_row('events')); } public function data_update_events()