Skip to content

Commit 267a356

Browse files
committed
Fix declarations to match the coding style
1 parent b39f191 commit 267a356

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

tests/external/get_preview_test.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,14 +24,14 @@
2424
* @copyright 2016 David Mudrak <david@moodle.com>
2525
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
2626
*/
27-
class get_preview_test extends \advanced_testcase {
27+
final class get_preview_test extends \advanced_testcase {
2828

2929
/**
3030
* Test the editor_marklar_get_preview() external function.
3131
*
3232
* @runInSeparateProcess
3333
*/
34-
public function test_get_preview() {
34+
public function test_get_preview(): void {
3535
global $CFG;
3636
require_once($CFG->libdir.'/externallib.php');
3737

@@ -55,7 +55,7 @@ public function test_get_preview() {
5555
*
5656
* @runInSeparateProcess
5757
*/
58-
public function test_embeded_images_preview() {
58+
public function test_embeded_images_preview(): void {
5959
global $CFG;
6060
require_once($CFG->libdir.'/externallib.php');
6161

tests/privacy/provider_test.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,12 +29,12 @@
2929
* @copyright 2018 David Mudrak <david@moodle.com>
3030
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
3131
*/
32-
class provider_test extends \advanced_testcase {
32+
final class provider_test extends \advanced_testcase {
3333

3434
/**
3535
* Assert that provider::get_metadata() returns valid content.
3636
*/
37-
public function test_get_metadata() {
37+
public function test_get_metadata(): void {
3838

3939
$items = new collection('editor_marklar');
4040
$result = provider::get_metadata($items);
@@ -45,7 +45,7 @@ public function test_get_metadata() {
4545
/**
4646
* Assert that provider::export_user_preferences() gives no data if the user has no Marklar preferences.
4747
*/
48-
public function test_export_user_preferences_no_pref() {
48+
public function test_export_user_preferences_no_pref(): void {
4949

5050
$user = \core_user::get_user_by_username('admin');
5151
provider::export_user_preferences($user->id);
@@ -58,7 +58,7 @@ public function test_export_user_preferences_no_pref() {
5858
/**
5959
* Assert that provider::export_user_preferences() gives user's Marklar preferences if they exist.
6060
*/
61-
public function test_export_user_preferences_has_pref() {
61+
public function test_export_user_preferences_has_pref(): void {
6262
$this->resetAfterTest();
6363

6464
$user = \core_user::get_user_by_username('admin');

0 commit comments

Comments
 (0)