(REF) ActivityTest - Use "Invasive" helper

It's shorter/prettier.

diff --git a/tests/phpunit/CRM/Activity/Form/ActivityTest.php b/tests/phpunit/CRM/Activity/Form/ActivityTest.php
index dad354ae1794472a5fd1ab6b3b4a077da8e85047..8e603a500dc20ad1abe417d8ae9b9ae65579ab14 100644 (file)
--- a/tests/phpunit/CRM/Activity/Form/ActivityTest.php
+++ b/tests/phpunit/CRM/Activity/Form/ActivityTest.php
@@ -1,5 +1,7 @@
 <?php
 
+use Civi\Test\Invasive;
+
 /**
  *  Include dataProvider for tests
  * @group headless
@@ -39,10 +41,7 @@ class CRM_Activity_Form_ActivityTest extends CiviUnitTestCase {
       'activity_type_id' => $activityTypeId,
     ];
 
-    $activityRef = new ReflectionClass('CRM_Activity_Form_Activity');
-    $method = $activityRef->getMethod('processActivity');
-    $method->setAccessible(TRUE);
-    $method->invokeArgs($form, [&$params]);
+    Invasive::call([$form, 'processActivity'], [&$params]);
 
     $msg = $mut->getMostRecentEmail();
     $this->assertNotEmpty($msg);
@@ -51,7 +50,8 @@ class CRM_Activity_Form_ActivityTest extends CiviUnitTestCase {
     //Block Meeting notification.
     Civi::settings()->set('do_not_notify_assignees_for', [$activityTypeId]);
     $params['assignee_contact_id'] = [$this->assignee2];
-    $method->invokeArgs($form, [&$params]);
+    Invasive::call([$form, 'processActivity'], [&$params]);
+
     $msg = $mut->getMostRecentEmail();
     $this->assertEmpty($msg);
   }