fixed wrong exception namespace, added some tests

composer.json

@@ -17,11 +17,14 @@
         "doctrine/orm": "^2.4"
     },
     "require-dev": {
-       "phpunit/phpunit": "^4.6"
+      "phpunit/phpunit": "~5.0"
     },
     "autoload": {
         "psr-4": {
             "Milio\\CQRS\\": "src"
         }
-    }
+    },
+  "autoload-dev": {
+    "psr-4": { "Milio\\CQRS\\Tests\\": "tests" }
+  }
 }

src/EventSourcing/EventSourcingRepository.php

@@ -17,4 +17,4 @@ public function __construct(EventStoreInterface $eventStore, EventBusInterface $
         $aggregateFactory = new PublicConstructorAggregateFactory();
         parent::__construct($eventStore, $eventBus, $aggregateClass, $aggregateFactory, []);
     }
-}
+}

src/ReadModel/Exception/NotFoundReadModelException.php

@@ -1,11 +1,10 @@
 <?php
 
-namespace Milio\CQRS\Readmodel\Exception;
+namespace Milio\CQRS\ReadModel\Exception;
 
 /**
  * Exception that gets thrown when read model could not be found by it's identifier
  */
 class NotFoundReadModelException extends \Exception
 {
-
-}
+}

src/ReadModel/InMemory/MilioInMemoryRepository.php

@@ -2,16 +2,16 @@
 
 namespace Milio\CQRS\ReadModel\InMemory;
 
-use Broadway\ReadModel\InMemory\InMemoryRepository;
 use Broadway\ReadModel\ReadModelInterface;
-use Milio\CQRS\Readmodel\Exception\NotFoundReadModelException;
+use Broadway\ReadModel\RepositoryInterface;
+use Milio\CQRS\ReadModel\Exception\NotFoundReadModelException;
 use Milio\CQRS\ReadModel\ReadModelRepositoryInterface;
 
 class MilioInMemoryRepository implements ReadModelRepositoryInterface
 {
     private $repository;
 
-    public function __construct(InMemoryRepository $repository)
+    public function __construct(RepositoryInterface $repository)
     {
         $this->repository = $repository;
     }
@@ -25,8 +25,9 @@ public function __construct(InMemoryRepository $repository)
     public function findOrFail($id)
     {
         $result = $this->repository->find($id);
-        if (empty($result)) {
-           throw new NotFoundReadModelException();
+
+        if (null == $result) {
+            throw new NotFoundReadModelException();
         }
 
         return $result;
@@ -90,4 +91,4 @@ public function remove($id)
     {
         return $this->repository->remove($id);
     }
-}
+}

tests/Identifier/AggregateIdentifierTest.php

@@ -1,8 +1,9 @@
 <?php
 
-namespace Milio\CQRS\Identifier;
+namespace Milio\CQRS\Tests\Identifier;
 
 use Milio\CQRS\Identifier\Testing\AggregateIdentifierTestCase;
+use Milio\CQRS\Identifier\AggregateIdentifier;
 
 class AggregateIdentifierTest extends AggregateIdentifierTestCase
 {
@@ -12,7 +13,7 @@ class AggregateIdentifierTest extends AggregateIdentifierTestCase
     public function generate_returns_an_object()
     {
         $test = TestIdentifier::generate();
-        $this->assertInstanceof('Milio\CQRS\Identifier\TestIdentifier', $test);
+        $this->assertInstanceof('Milio\CQRS\Tests\Identifier\TestIdentifier', $test);
     }
 
     /**
@@ -43,7 +44,7 @@ public function it_supports_returning_as_string()
     public function it_takes_a_string_as_constructor_argument()
     {
         $test = new TestIdentifier('foo');
-        $this->assertInstanceof('Milio\CQRS\Identifier\TestIdentifier', $test);
+        $this->assertInstanceof('Milio\CQRS\Tests\Identifier\TestIdentifier', $test);
         $this->assertEquals('foo', (string) $test);
         $this->assertEquals('foo', $test->getIdentifierString());
     }