diff --git a/message_subscribe.test b/message_subscribe.test index 1fa088c..622c752 100644 --- a/message_subscribe.test +++ b/message_subscribe.test @@ -473,10 +473,9 @@ class MessageSubscribeQueueTest extends DrupalWebTestCase { } /** - * Test that if we get a list of user IDs directly from the implementing module, - * the messages are sent respecting the range value. - * - * @see https://drupal.org/node/2256899 + * Test that if we get a list of user IDs directly from + * the implementing module, the messages are sent respecting + * the range value. */ function testProvidedUserIdsAreSplitAccordingToRangeValue() { @@ -493,12 +492,12 @@ class MessageSubscribeQueueTest extends DrupalWebTestCase { $user3->uid => array('notifiers' => array('email')), ), 'skip context' => TRUE, - 'range' => 1 + 'range' => 1, ); message_subscribe_send_message('node', $this->node, $message, array(), $subscribe_options); - // Run cron + // Run cron. $this->cronRun(); $captured_emails = variable_get('drupal_test_email_collector', array()); @@ -509,9 +508,9 @@ class MessageSubscribeQueueTest extends DrupalWebTestCase { $recipients[] = $captured_email['to']; } - $this->assertTrue(in_array($user1->mail, $recipients), 'user1 mail is in the recipient list'); - $this->assertTrue(in_array($user2->mail, $recipients), 'user2 mail is in the recipient list'); - $this->assertTrue(in_array($user3->mail, $recipients), 'user3 mail is in the recipient list'); + $this->assertTrue(in_array($user1->mail, $recipients), 'User 1 mail is in the recipient list.'); + $this->assertTrue(in_array($user2->mail, $recipients), 'User 2 mail is in the recipient list.'); + $this->assertTrue(in_array($user3->mail, $recipients), 'User 3 mail is in the recipient list.'); } }