Fix previous refactor of unit tests

Author: djarran

tests/email_action_step_test.php

@@ -119,7 +119,7 @@ public function test_execute_external_email_address(): void {
     }
 
     public function test_execute_multiple_email_addresses(): void {
-        $emailstosend = [$this->user1->email . ', ' . $this->user2->email . ', ' . 'testusernotinmoodle@example.com'];
+        $emailstosend = [$this->user1->email, $this->user2->email, 'testusernotinmoodle@example.com'];
         $settings = [
             'emailto' => implode(",", $emailstosend),
             'emailsubject' => 'Subject of the email',
@@ -141,6 +141,7 @@ public function test_execute_multiple_email_addresses(): void {
     }
 
     public function test_execute_multiple_email_addresses_with_datafields(): void {
+        $emailstosend = [$this->user1->email, $this->user2->email, 'testusernotinmoodle@example.com'];
         $settings = [
             'emailto' => '{user_emails}',
             'emailsubject' => 'Subject of the email',
@@ -151,8 +152,9 @@ public function test_execute_multiple_email_addresses_with_datafields(): void {
 
         // Ensure that this step can accept a string of emails from a previous step.
         // Not currently possible with existing steps.
+        $emailstosend = [$this->user1->email, $this->user2->email];
         $prevstepresults = [
-            'user_emails' => $this->user1->email . ',' . $this->user2->email,
+            'user_emails' => implode(',', $emailstosend),
         ];
 
         // Execute the step.
@@ -161,7 +163,7 @@ public function test_execute_multiple_email_addresses_with_datafields(): void {
 
         // Retrieve the messages.
         $messages = $this->sink->get_messages();
-        $this->assertEquals(2, count($messages));
+        $this->assertEquals(count($emailstosend), count($messages));
     }
 
     public function test_execute_with_datafields(): void {