@@ -36,6 +36,8 @@ class status_test extends \phpbb_test_case
3636 /** @var \phpbb\notification\manager|\PHPUnit\Framework\MockObject\MockObject */
3737 protected $ notification_manager ;
3838
39+ protected int $ forum_id ;
40+
3941 protected function setUp (): void
4042 {
4143 parent ::setUp ();
@@ -65,7 +67,6 @@ protected function setUp(): void
6567 // Set protected properties using reflection
6668 $ reflection = new \ReflectionClass ($ this ->notification_type );
6769 $ notification_manager_property = $ reflection ->getProperty ('notification_manager ' );
68- $ notification_manager_property ->setAccessible (true );
6970 $ notification_manager_property ->setValue ($ this ->notification_type , $ this ->notification_manager );
7071 }
7172
@@ -76,7 +77,6 @@ protected function setNotificationData(array $data)
7677 {
7778 $ reflection = new \ReflectionClass ($ this ->notification_type );
7879 $ method = $ reflection ->getMethod ('set_data ' );
79- $ method ->setAccessible (true );
8080
8181 foreach ($ data as $ key => $ value )
8282 {
@@ -322,7 +322,6 @@ public function test_create_insert_array()
322322 // Verify data was set by checking get_data
323323 $ reflection = new \ReflectionClass ($ this ->notification_type );
324324 $ get_data_method = $ reflection ->getMethod ('get_data ' );
325- $ get_data_method ->setAccessible (true );
326325
327326 $ this ->assertEquals (7 , $ get_data_method ->invoke ($ this ->notification_type , 'idea_id ' ));
328327 $ this ->assertEquals (4 , $ get_data_method ->invoke ($ this ->notification_type , 'status ' ));
0 commit comments