Merge pull request #86 from jongotlin/external-calls

External calls
dizda · Nov 9, 2015 · 14f6849 · 14f6849
2 parents b105806 + 23fc0ff
commit 14f6849
Show file tree

Hide file tree

Showing 4 changed files with 77 additions and 5 deletions.
diff --git a/Event/BackupEvent.php b/Event/BackupEvent.php
@@ -0,0 +1,10 @@
+<?php
+
+namespace Dizda\CloudBackupBundle\Event;
+
+use Symfony\Component\EventDispatcher\Event;
+
+class BackupEvent extends Event
+{
+    const BACKUP_COMPLETED = 'dizda.cloudbackup.backup_completed';
+}
diff --git a/Manager/BackupManager.php b/Manager/BackupManager.php
@@ -2,7 +2,9 @@
 
 namespace Dizda\CloudBackupBundle\Manager;
 
+use Dizda\CloudBackupBundle\Event\BackupEvent;
 use Psr\Log\LoggerInterface;
+use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Symfony\Component\Filesystem\Exception\IOException;
 
 class BackupManager
@@ -28,21 +30,29 @@ class BackupManager
     private $processor;
 
     /**
-     * @param LoggerInterface  $logger
-     * @param DatabaseManager  $database
-     * @param ClientManager    $client
-     * @param ProcessorManager $processor
+     * @var EventDispatcherInterface
+     */
+    private $eventDispatcher;
+
+    /**
+     * @param LoggerInterface          $logger
+     * @param DatabaseManager          $database
+     * @param ClientManager            $client
+     * @param ProcessorManager         $processor
+     * @param EventDispatcherInterface $eventDispatcher
      */
     public function __construct(
         LoggerInterface $logger,
         DatabaseManager $database,
         ClientManager $client,
-        ProcessorManager $processor
+        ProcessorManager $processor,
+        EventDispatcherInterface $eventDispatcher
     ) {
         $this->logger = $logger;
         $this->dbm = $database;
         $this->cm = $client;
         $this->processor = $processor;
+        $this->eventDispatcher = $eventDispatcher;
     }
 
     /**
@@ -84,6 +94,10 @@ public function execute()
             return false;
         }
 
+        if ($successful) {
+            $this->eventDispatcher->dispatch(BackupEvent::BACKUP_COMPLETED, new BackupEvent());
+        }
+
         return $successful;
     }
 

diff --git a/Resources/config/services.yml b/Resources/config/services.yml
@@ -109,6 +109,7 @@ services:
             - @dizda.cloudbackup.manager.database
             - @dizda.cloudbackup.manager.client
             - @dizda.cloudbackup.manager.processor
+            - @event_dispatcher
 
     dizda.cloudbackup.manager.processor:
         class: Dizda\CloudBackupBundle\Manager\ProcessorManager

diff --git a/Tests/Manager/BackupManagerTest.php b/Tests/Manager/BackupManagerTest.php
@@ -0,0 +1,47 @@
+<?php
+
+namespace Dizda\CloudBackupBundle\Tests\Manager;
+
+use Dizda\CloudBackupBundle\Event\BackupEvent;
+use Dizda\CloudBackupBundle\Manager\BackupManager;
+
+class BackupManagerTest extends \PHPUnit_Framework_TestCase
+{
+    public function testBackupCompletedEventIsCalledOnSuccess()
+    {
+        $loggerMock = $this->getMock('Psr\Log\LoggerInterface');
+        $databaseManagerMock = $this->getMockBuilder('Dizda\CloudBackupBundle\Manager\DatabaseManager')
+            ->disableOriginalConstructor()->getMock();
+        $clientManagerMock = $this->getMockBuilder('Dizda\CloudBackupBundle\Manager\ClientManager')
+            ->disableOriginalConstructor()->getMock();
+        $processorManagerMock = $this->getMockBuilder('Dizda\CloudBackupBundle\Manager\ProcessorManager')
+            ->disableOriginalConstructor()->getMock();
+        $eventDispatcherMock = $this->getMock('Symfony\Component\EventDispatcher\EventDispatcherInterface');
+        $eventDispatcherMock->expects($this->once())->method('dispatch')->with(BackupEvent::BACKUP_COMPLETED);
+
+        $backupManager = new BackupManager(
+            $loggerMock, $databaseManagerMock, $clientManagerMock, $processorManagerMock, $eventDispatcherMock
+        );
+        $backupManager->execute();
+    }
+
+    public function testBackupCompletedEventIsNotCalledWhenFailed()
+    {
+        $loggerMock = $this->getMock('Psr\Log\LoggerInterface');
+        $databaseManagerMock = $this->getMockBuilder('Dizda\CloudBackupBundle\Manager\DatabaseManager')
+            ->disableOriginalConstructor()->getMock();
+        $clientManagerMock = $this->getMockBuilder('Dizda\CloudBackupBundle\Manager\ClientManager')
+            ->disableOriginalConstructor()->getMock();
+        $processorManagerMock = $this->getMockBuilder('Dizda\CloudBackupBundle\Manager\ProcessorManager')
+            ->disableOriginalConstructor()->getMock();
+        $processorManagerMock->expects($this->once())->method('copyFolders')
+            ->will($this->throwException(new \Exception()));
+        $eventDispatcherMock = $this->getMock('Symfony\Component\EventDispatcher\EventDispatcherInterface');
+        $eventDispatcherMock->expects($this->never())->method('dispatch')->with(BackupEvent::BACKUP_COMPLETED);
+
+        $backupManager = new BackupManager(
+            $loggerMock, $databaseManagerMock, $clientManagerMock, $processorManagerMock, $eventDispatcherMock
+        );
+        $backupManager->execute();
+    }
+}