Clear the in_progress flag if WC sync task fails [MAILPOET-2202]

lib/Cron/Workers/WooCommerceSync.php

@@ -49,7 +49,13 @@ class WooCommerceSync extends SimpleWorker {
     $task->meta = ['in_progress' => true];
     $task->save();
 
-    $this->woocommerce_segment->synchronizeCustomers();
+    try {
+      $this->woocommerce_segment->synchronizeCustomers();
+    } catch (\Exception $e) {
+      $task->meta = null;
+      $task->save();
+      throw $e;
+    }
 
     return true;
   }

tests/integration/Cron/Workers/WooCommerceSyncTest.php

@@ -42,6 +42,20 @@ class WooCommerceSyncTest extends \MailPoetTest {
     expect($task->getMeta())->notEmpty();
   }
 
+  function testItWillResetTheInProgressFlagOnFail() {
+    $task = $this->createScheduledTask();
+    $this->woocommerce_segment->expects($this->once())
+      ->method('synchronizeCustomers')
+      ->willThrowException(new \Exception('test error'));
+    try {
+      $this->worker->processTaskStrategy($task);
+      $this->fail('An exception should be thrown');
+    } catch (\Exception $e) {
+      expect($e->getMessage())->equals('test error');
+      expect($task->getMeta())->isEmpty();
+    }
+  }
+
   function testItWillRescheduleTaskIfItIsRunningForTooLong() {
     $this->woocommerce_segment->expects($this->once())
       ->method('synchronizeCustomers');