mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-04-14 13:02:07 +02:00
Code Issues Projects Releases Wiki Activity

MDL-4188 mod_forum: Modify PHPUnit to use the new sink method

Browse Source
This commit is contained in:
Huong Nguyen 2024-01-30 10:56:37 +07:00
parent 097a6d46e0
commit b68a89a4da
No known key found for this signature in database
GPG Key ID: 40D88AB693A3E72A
2 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
mod/forum/tests/mail_test.php
View File

@ -927,7 +927,7 @@ class mail_test extends \advanced_testcase {
$this->queue_tasks_and_assert($expect);
$this->send_notifications_and_assert($author, [$post]);
$messages = $this->messagesink->get_messages();
$messages = $this->messagesink->get_messages_by_component('mod_forum');
$message = reset($messages);
$this->assertEquals($author->id, $message->useridfrom);
$this->assertEquals($expectedsubject, $message->subject);
@ -1600,8 +1600,9 @@ class mail_test extends \advanced_testcase {
$this->send_notifications_and_assert($author, [$post]);
$this->send_notifications_and_assert($commenter, [$post]);
$messages = $this->messagesink->get_messages();
$customdata = json_decode($messages[0]->customdata);
$messages = $this->messagesink->get_messages_by_component('mod_forum');
$messages = reset($messages);
$customdata = json_decode($messages->customdata);
$this->assertEquals($forum->id, $customdata->instance);
$this->assertEquals($forum->cmid, $customdata->cmid);
$this->assertEquals($post->id, $customdata->postid);

10
mod/forum/tests/maildigest_test.php
View File

@ -62,7 +62,7 @@ class maildigest_test extends \advanced_testcase {
$this->mailsink = $this->redirectEmails();
// Confirm that we have an empty message sink so far.
$messages = $this->messagesink->get_messages();
$messages = $this->messagesink->get_messages_by_component('mod_forum');
$this->assertEquals(0, count($messages));
$messages = $this->mailsink->get_messages();
@ -405,7 +405,9 @@ class maildigest_test extends \advanced_testcase {
$this->send_digests_and_assert($user, $posts);
// The user does not, by default, have permission to view the fullname.
$messagecontent = $this->messagesink->get_messages()[0]->fullmessage;
$messages = $this->messagesink->get_messages_by_component('mod_forum');
$messages = reset($messages);
$messagecontent = $messages->fullmessage;
// Assert that the expected name is present (lastname only).
$this->assertStringContainsString(fullname($user, false), $messagecontent);
@ -456,7 +458,9 @@ class maildigest_test extends \advanced_testcase {
// The user does not, by default, have permission to view the fullname.
// However we have given the user that capability so we expect to see both firstname and lastname.
$messagecontent = $this->messagesink->get_messages()[0]->fullmessage;
$messages = $this->messagesink->get_messages_by_component('mod_forum');
$messages = reset($messages);
$messagecontent = $messages->fullmessage;
// Assert that the expected name is present (lastname only).
$this->assertStringContainsString(fullname($user, false), $messagecontent);