Rename initApi() to init() in workers [PREMIUM-4]

lib/Cron/Workers/Bounce.php

@@ -21,7 +21,7 @@ class Bounce extends SimpleWorker {
 
   public $api;
 
-  function initApi() {
+  function init() {
     if(!$this->api) {
       $mailer_config = Mailer::getMailerConfig();
       $this->api = new API($mailer_config['mailpoet_api_key']);

lib/Cron/Workers/KeyCheck/KeyCheckWorker.php

@@ -13,7 +13,7 @@ abstract class KeyCheckWorker extends SimpleWorker {
 
   public $bridge;
 
-  function initApi() {
+  function init() {
     if(!$this->bridge) {
       $this->bridge = new Bridge();
     }

lib/Cron/Workers/SimpleWorker.php

@@ -19,12 +19,6 @@ abstract class SimpleWorker {
     CronHelper::enforceExecutionLimit($this->timer);
   }
 
-  function init() {
-    if(is_callable(array($this, 'initApi'))) {
-      $this->initApi();
-    }
-  }
-
   function checkProcessingRequirements() {
     return true;
   }
@@ -34,7 +28,9 @@ abstract class SimpleWorker {
       return false;
     }
 
-    $this->init();
+    if(is_callable(array($this, 'init'))) {
+      $this->init();
+    }
 
     $scheduled_queues = self::getScheduledQueues();
     $running_queues = self::getRunningQueues();

tests/unit/Cron/Workers/BounceTest.php

@@ -38,7 +38,7 @@ class BounceTest extends MailPoetTest {
   function testItCanInitializeBridgeAPI() {
     $this->setMailPoetSendingMethod();
     $worker = new Bounce(microtime(true));
-    $worker->initApi();
+    $worker->init();
     expect($worker->api instanceof API)->true();
   }
 

tests/unit/Cron/Workers/KeyCheck/KeyCheckWorkerTest.php

@@ -15,7 +15,7 @@ class KeyCheckWorkerTest extends MailPoetTest {
   }
 
   function testItCanInitializeBridgeAPI() {
-    $this->worker->initApi();
+    $this->worker->init();
     expect($this->worker->bridge instanceof Bridge)->true();
   }
 

tests/unit/Cron/Workers/SimpleWorkerMockImplementation.php

@@ -6,7 +6,7 @@ if(!defined('ABSPATH')) exit;
 class MockSimpleWorker extends SimpleWorker {
   const TASK_TYPE = 'mock_simple_worker';
 
-  function initApi() {
+  function init() {
     // to be mocked
   }
 }

tests/unit/Cron/Workers/SimpleWorkerTest.php

@@ -116,10 +116,10 @@ class SimpleWorkerTest extends MailPoetTest {
     expect($worker->process())->false();
   }
 
-  function testItCanInitAPIBeforeProcessing() {
+  function testItCanInitBeforeProcessing() {
     $worker = Stub::make(
       $this->worker,
-      array('initApi' => Stub::once()),
+      array('init' => Stub::once()),
       $this
     );
     $worker->process();