more and more tests

Author: nekufa

src/Entity.php

@@ -2,7 +2,6 @@
 
 namespace Tarantool\Mapper;
 
-use Closure;
 use LogicException;
 
 class Entity implements Contracts\Entity
@@ -33,10 +32,6 @@ public function __set($key, $value)
     public function __get($key)
     {
         if (array_key_exists($key, $this->data)) {
-            if ($this->data[$key] instanceof Closure) {
-                $this->data[$key] = $this->data[$key]();
-            }
-
             return $this->data[$key];
         }
     }

src/Repository.php

@@ -47,8 +47,8 @@ public function create($data = null)
             $data = $newData;
         }
 
-        foreach($data as $k => $v) {
-            if(!$this->type->hasProperty($k)) {
+        foreach ($data as $k => $v) {
+            if (!$this->type->hasProperty($k)) {
                 throw new \Exception("Unknown property $k");
             }
         }

src/Schema/Meta.php

@@ -69,7 +69,7 @@ public function create($type, array $fields = null)
         if ($fields) {
             foreach ($fields as $index => $field) {
                 if ($field instanceof Contracts\Type) {
-                    if(!is_numeric($index)) {
+                    if (!is_numeric($index)) {
                         $instance->reference($field, $index);
                     } else {
                         $instance->reference($field);

tests/MapperTest.php

@@ -8,7 +8,6 @@ public function testEntityPropertiesCheck()
         $manager->getMeta()->create('person', ['firstName', 'lastName']);
         $this->setExpectedException(Exception::class);
         $manager->create('person', ['name' => 'Dmitry Krokhin']);
-
     }
     public function testEntityConstructorCheck()
     {
@@ -189,5 +188,7 @@ public function testById()
 
         $company = $manager->create('company', 'basis.company');
         $manager->get('company')->find(1);
+
+        $this->assertSame($company->getData(), ['name' => 'basis.company', 'id' => 1]);
     }
 }

tests/MigrationTest.php

@@ -19,6 +19,9 @@ public function testSecondMigration()
         $migrator = new Migrator();
         $migrator->migrate($manager);
         $migrator->migrate($manager);
+
+        $boo = new Tarantool\Mapper\Migrations\Bootstrap();
+        $boo->migrate($manager);
     }
     public function testUsage()
     {

tests/SchemaTest.php

@@ -0,0 +1,23 @@
+<?php
+
+
+class SchemaTest extends PHPUnit_Framework_TestCase
+{
+    public function testSpaceName()
+    {
+        $manager = Helper::createManager();
+        $this->assertSame('_space', $manager->getSchema()->getSpaceName(280));
+    }
+
+    public function testConventionOverride()
+    {
+        $meta = Helper::createManager()->getMeta();
+        $this->assertNotNull($meta->getConvention());
+
+        $convention = new Tarantool\Mapper\Schema\Convention();
+        $meta->setConvention($convention);
+        $this->assertSame($meta->getConvention(), $convention);
+
+        $this->assertSame([1], $convention->encode('array', [1]));
+    }
+}