diff --git a/lib/Router/Subscribers.php b/lib/Router/Subscribers.php index 2d0367acdf..44605b656d 100644 --- a/lib/Router/Subscribers.php +++ b/lib/Router/Subscribers.php @@ -160,7 +160,7 @@ class Subscribers { } } - switch ($form_settings['on_success']) { + switch($form_settings['on_success']) { case 'page': // response depending on context if($doing_ajax === true) { diff --git a/tests/unit/Router/SubscribersCest.php b/tests/unit/Router/SubscribersCest.php index 4fad8e7977..36d7ca0ed8 100644 --- a/tests/unit/Router/SubscribersCest.php +++ b/tests/unit/Router/SubscribersCest.php @@ -88,20 +88,54 @@ class SubscribersCest { expect($updated_subscriber->first_name)->equals('Jane'); } - function itCanSubscribeToSegments() { - - } - function itCanRestoreASubscriber() { + $subscriber = Subscriber::createOrUpdate(array( + 'email' => 'john.doe@mailpoet.com', + 'first_name' => 'John', + 'last_name' => 'Doe' + )); + expect($subscriber->id() > 0)->true(); + $subscriber->trash(); + + expect($subscriber->deleted_at)->notNull(); + + $router = new Subscribers(); + $router->restore($subscriber->id()); + + $restored_subscriber = Subscriber::findOne($subscriber->id()); + expect($restored_subscriber->deleted_at)->null(); } function itCanTrashASubscriber() { + $subscriber = Subscriber::createOrUpdate(array( + 'email' => 'john.doe@mailpoet.com', + 'first_name' => 'John', + 'last_name' => 'Doe' + )); + expect($subscriber->id() > 0)->true(); + $router = new Subscribers(); + $response = $router->trash($subscriber->id()); + expect($response)->true(); + + $trashed_subscriber = Subscriber::findOne($subscriber->id()); + expect($trashed_subscriber->deleted_at)->notNull(); } function itCanDeleteASubscriber() { + $subscriber = Subscriber::createOrUpdate(array( + 'email' => 'john.doe@mailpoet.com', + 'first_name' => 'John', + 'last_name' => 'Doe' + )); + expect($subscriber->id() > 0)->true(); + $router = new Subscribers(); + $response = $router->delete($subscriber->id()); + expect($response)->equals(1); + + expect(Subscriber::findOne($subscriber->id()))->false(); } function _after() {