Skip to content

Fix deprecated expectErrorMessage call#152

Open
Fragonite wants to merge 1 commit intomoodleou:MOODLE_404_STABLEfrom
catalyst:deprecation-fix
Open

Fix deprecated expectErrorMessage call#152
Fragonite wants to merge 1 commit intomoodleou:MOODLE_404_STABLEfrom
catalyst:deprecation-fix

Conversation

@Fragonite
Copy link

Resolves issue #151

Before

php vendor/bin/phpunit --testsuite mod_oublog_testsuite --filter test_oublog_check_view_permissions_personal_restrict
Moodle 4.5.1+ (Build: 20250124), ref: refs/heads/wr-452085-add-submodules
Php: 8.1.29, pgsql: 16.0 (Debian 16.0-1.pgdg120+1), OS: Linux 6.8.0-52-generic x86_64
PHPUnit 9.6.18 by Sebastian Bergmann and contributors.

W                                                                   1 / 1 (100%)

Time: 00:00.688, Memory: 86.50 MB

There was 1 warning:

1) oublog_locallib_test::test_oublog_check_view_permissions_personal_restrict
Expecting E_ERROR and E_USER_ERROR is deprecated and will no longer be possible in PHPUnit 10.

/var/www/icms/lib/phpunit/classes/advanced_testcase.php:76

WARNINGS!
Tests: 1, Assertions: 2, Warnings: 1.

After

php vendor/bin/phpunit --testsuite mod_oublog_testsuite --filter test_oublog_check_view_permissions_personal_restrict
Moodle 4.5.1+ (Build: 20250124), ref: refs/heads/wr-452085-add-submodules
Php: 8.1.29, pgsql: 16.0 (Debian 16.0-1.pgdg120+1), OS: Linux 6.8.0-52-generic x86_64
PHPUnit 9.6.18 by Sebastian Bergmann and contributors.

.                                                                   1 / 1 (100%)

Time: 00:00.669, Memory: 84.50 MB

OK (1 test, 2 assertions)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant