Skip to content

Commit f73ac82

Browse files
committed
Update tests
1 parent d4f11c2 commit f73ac82

5 files changed

Lines changed: 15 additions & 22 deletions

File tree

tests/controller/page_main_controller_test.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,19 +12,19 @@
1212

1313
class page_main_controller_test extends \phpbb_database_test_case
1414
{
15-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\auth\auth */
15+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\auth\auth */
1616
protected $auth;
1717

18-
/** @var \PHPUnit_Framework_MockObject_MockObject|\Symfony\Component\DependencyInjection\ContainerInterface */
18+
/** @var \PHPUnit\Framework\MockObject\MockObject|\Symfony\Component\DependencyInjection\ContainerInterface */
1919
protected $container;
2020

21-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\controller\helper */
21+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\controller\helper */
2222
protected $controller_helper;
2323

2424
/** @var \phpbb\language\language */
2525
protected $lang;
2626

27-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\template\template */
27+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\template\template */
2828
protected $template;
2929

3030
/** @var \phpbb\user */

tests/event/event_listener_base.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,19 +12,19 @@
1212

1313
class event_listener_base extends \phpbb_test_case
1414
{
15-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\auth\auth */
15+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\auth\auth */
1616
protected $auth;
1717

18-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\controller\helper */
18+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\controller\helper */
1919
protected $controller_helper;
2020

2121
/** @var \phpbb\language\language */
2222
protected $lang;
2323

24-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\pages\operators\page */
24+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\pages\operators\page */
2525
protected $page_operator;
2626

27-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\template\template */
27+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\template\template */
2828
protected $template;
2929

3030
/** @var \phpbb\user */

tests/event/event_listener_viewonline_test.php

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -119,19 +119,12 @@ public function test_viewonline_page($on_page, $row, $location_url, $location, $
119119

120120
$listener = $this->get_listener();
121121

122-
$dispatcher = new \Symfony\Component\EventDispatcher\EventDispatcher();
122+
$dispatcher = new \phpbb\event\dispatcher();
123123
$dispatcher->addListener('core.viewonline_overwrite_location', array($listener, 'viewonline_page'));
124124

125125
$event_data = array('on_page', 'row', 'location_url', 'location');
126-
$event = new \phpbb\event\data(compact($event_data));
127-
$dispatcher->dispatch('core.viewonline_overwrite_location', $event);
128-
129-
$event_data_after = $event->get_data_filtered($event_data);
130-
foreach ($event_data as $expected)
131-
{
132-
self::assertArrayHasKey($expected, $event_data_after);
133-
}
134-
extract($event_data_after);
126+
$event_data_after = $dispatcher->trigger_event('core.viewonline_overwrite_location', compact($event_data));
127+
extract($event_data_after, EXTR_OVERWRITE);
135128

136129
self::assertEquals($expected_location_url, $location_url);
137130
self::assertEquals($expected_location, $location);

tests/event/show_page_links_test.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,8 +106,8 @@ public function test_show_page_links()
106106
array('S_OVERALL_HEADER_NAVIGATION_APPEND', true)
107107
);
108108

109-
$dispatcher = new \Symfony\Component\EventDispatcher\EventDispatcher();
109+
$dispatcher = new \phpbb\event\dispatcher();
110110
$dispatcher->addListener('core.page_header', array($listener, 'show_page_links'));
111-
$dispatcher->dispatch('core.page_header');
111+
$dispatcher->trigger_event('core.page_header');
112112
}
113113
}

tests/operators/page_operator_base.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ protected static function setup_extensions()
3131
/** @var \phpbb\config\config */
3232
protected $config;
3333

34-
/** @var \PHPUnit_Framework_MockObject_MockObject|\Symfony\Component\DependencyInjection\ContainerInterface */
34+
/** @var \PHPUnit\Framework\MockObject\MockObject|\Symfony\Component\DependencyInjection\ContainerInterface */
3535
protected $container;
3636

3737
/** @var \phpbb\db\driver\driver_interface */
@@ -46,7 +46,7 @@ protected static function setup_extensions()
4646
/** @var \phpbb\textformatter\s9e\utils */
4747
protected $text_formatter_utils;
4848

49-
/** @var \PHPUnit_Framework_MockObject_MockObject|\phpbb\user */
49+
/** @var \PHPUnit\Framework\MockObject\MockObject|\phpbb\user */
5050
protected $user;
5151

5252
public function getDataSet()

0 commit comments

Comments
 (0)