Add GraveyardInterface and BufferedGraveyard

src/BufferedGraveyard.php

@@ -0,0 +1,34 @@
+<?php
+
+namespace Scheb\Tombstone;
+
+class BufferedGraveyard implements GraveyardInterface
+{
+    /**
+     * @var GraveyardInterface
+     */
+    private $graveyard;
+
+    /**
+     * @var array
+     */
+    private $tombstoneCalls = [];
+
+    public function __construct(GraveyardInterface $graveyard)
+    {
+        $this->graveyard = $graveyard;
+    }
+
+    public function tombstone(string $date, ?string $author, ?string $label, array $trace): void
+    {
+        $this->tombstoneCalls[] = func_get_args();
+    }
+
+    public function flush(): void
+    {
+        foreach ($this->tombstoneCalls as $args) {
+            $this->graveyard->tombstone(... $args);
+        }
+        $this->graveyard->flush();
+    }
+}

src/Graveyard.php

@@ -6,7 +6,7 @@
 use Scheb\Tombstone\Tracing\PathNormalizer;
 use Scheb\Tombstone\Tracing\TraceProvider;
 
-class Graveyard
+class Graveyard implements GraveyardInterface
 {
     /**
      * @var HandlerInterface[]

src/GraveyardInterface.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace Scheb\Tombstone;
+
+interface GraveyardInterface
+{
+    public function tombstone(string $date, ?string $author, ?string $label, array $trace): void;
+
+    public function flush(): void;
+}

src/GraveyardProvider.php

@@ -5,11 +5,11 @@
 class GraveyardProvider
 {
     /**
-     * @var Graveyard
+     * @var GraveyardInterface
      */
     private static $graveyard;
 
-    public static function getGraveyard(): Graveyard
+    public static function getGraveyard(): GraveyardInterface
     {
         if (null === self::$graveyard) {
             self::$graveyard = new Graveyard();
@@ -18,7 +18,7 @@ public static function getGraveyard(): Graveyard
         return self::$graveyard;
     }
 
-    public static function setGraveyard(Graveyard $graveyard): void
+    public static function setGraveyard(GraveyardInterface $graveyard): void
     {
         self::$graveyard = $graveyard;
     }

tests/BufferedGraveyardTest.php

@@ -0,0 +1,62 @@
+<?php
+
+namespace Scheb\Tombstone\Test;
+
+use Scheb\Tombstone\BufferedGraveyard;
+use Scheb\Tombstone\Graveyard;
+use Scheb\Tombstone\GraveyardInterface;
+
+class BufferedGraveyardTest extends TestCase
+{
+    /**
+     * @var \PHPUnit_Framework_MockObject_MockObject
+     */
+    private $innerGraveyard;
+
+    /**
+     * @var Graveyard
+     */
+    private $graveyard;
+
+    protected function setUp()
+    {
+        $this->innerGraveyard = $this->createMock(GraveyardInterface::class);
+        $this->graveyard = new BufferedGraveyard($this->innerGraveyard);
+    }
+
+    /**
+     * @test
+     */
+    public function tombstone_tombstoneInvoked_notAddToInnerGraveyard(): void
+    {
+        $this->innerGraveyard
+            ->expects($this->never())
+            ->method($this->anything());
+
+        $this->graveyard->tombstone('2018-01-01', 'author1', 'label1', ['trace1']);
+    }
+
+    /**
+     * @test
+     */
+    public function flush_tombstonesBuffered_addTombstonesAndFlush(): void
+    {
+        $this->graveyard->tombstone('2018-01-01', 'author1', 'label1', ['trace1']);
+        $this->graveyard->tombstone('2018-02-01', 'author2', 'label2', ['trace2']);
+
+        $this->innerGraveyard
+            ->expects($this->exactly(2))
+            ->method('tombstone')
+            ->withConsecutive(
+                ['2018-01-01', 'author1', 'label1', ['trace1']],
+                ['2018-02-01', 'author2', 'label2', ['trace2']]
+            );
+
+        $this->innerGraveyard
+            ->expects($this->once())
+            ->method('flush');
+
+        $this->graveyard->flush();
+
+    }
+}

tests/GraveyardProviderTest.php

@@ -3,6 +3,7 @@
 namespace Scheb\Tombstone\Test;
 
 use Scheb\Tombstone\Graveyard;
+use Scheb\Tombstone\GraveyardInterface;
 use Scheb\Tombstone\GraveyardProvider;
 
 class GraveyardProviderTest extends TestCase
@@ -21,7 +22,7 @@ public function getGraveyard_notSet_returnDefaultGraveyard(): void
      */
     public function setGraveyard_newGraveyard_exchangeGraveyard(): void
     {
-        $graveyard = $this->createMock(Graveyard::class);
+        $graveyard = $this->createMock(GraveyardInterface::class);
         GraveyardProvider::setGraveyard($graveyard);
         $returnValue = GraveyardProvider::getGraveyard();
         $this->assertSame($graveyard, $returnValue);