make mailing failure tests less specific. We try not to make failure tests too specif...
authorEileen McNaughton <eileen@fuzion.co.nz>
Mon, 16 Mar 2015 08:38:56 +0000 (21:38 +1300)
committerEileen McNaughton <eileen@fuzion.co.nz>
Mon, 16 Mar 2015 08:38:56 +0000 (21:38 +1300)
to spec will change the message & we may as well not check it rather than changing the test

tests/phpunit/api/v3/MailingGroupTest.php

index d076c0bcca88e4d48379e7781a0cd3d353ac1a98..a4a3169c3e23e18c45940c7ff7a8983cbcb0e090 100644 (file)
@@ -58,13 +58,7 @@ class api_v3_MailingGroupTest extends CiviUnitTestCase {
       'time_stamp' => '20111111010101',
       'hash' => 'sasa',
     );
-    $result = $this->callAPIFailure('mailing_event_subscribe', 'create', $params);
-    if ($result['error_message'] != 'Subscription failed') {
-      $this->assertEquals($result['error_message'], 'Invalid Group id', 'In line ' . __LINE__);
-    }
-    else {
-      $this->assertEquals($result['error_message'], 'Subscription failed', 'In line ' . __LINE__);
-    }
+    $this->callAPIFailure('mailing_event_subscribe', 'create', $params);
   }
 
   /**
@@ -105,8 +99,7 @@ class api_v3_MailingGroupTest extends CiviUnitTestCase {
       'time_stamp' => '20101212121212',
     );
 
-    $result = $this->callAPIFailure('mailing_event_unsubscribe', 'create', $params);
-    $this->assertEquals($result['error_message'], 'Queue event could not be found', 'In line ' . __LINE__);
+    $this->callAPIFailure('mailing_event_unsubscribe', 'create', $params);
   }
 
   //--------- civicrm_mailing_group_event_domain_unsubscribe methods -------
@@ -123,8 +116,7 @@ class api_v3_MailingGroupTest extends CiviUnitTestCase {
       'time_stamp' => '20101212121212',
     );
 
-    $result = $this->callAPIFailure('mailing_event_unsubscribe', 'create', $params);
-    $this->assertEquals($result['error_message'], 'Domain Queue event could not be found', 'In line ' . __LINE__);
+    $this->callAPIFailure('mailing_event_unsubscribe', 'create', $params);
   }
 
   //----------- civicrm_mailing_group_event_resubscribe methods--------
@@ -144,8 +136,7 @@ class api_v3_MailingGroupTest extends CiviUnitTestCase {
       'org_unsubscribe' => 'test',
       'time_stamp' => '20101212121212',
     );
-    $result = $this->callAPIFailure('mailing_event_resubscribe', 'create', $params);
-    $this->assertEquals($result['error_message'], 'Queue event could not be found', 'In line ' . __LINE__);
+    $this->callAPIFailure('mailing_event_resubscribe', 'create', $params);
   }
 
   //------------------------ success case ---------------------