#111. Add more coverage on BaseRelationsTrait

src/extension/BaseRelationsTrait.php

@@ -236,6 +236,7 @@ private function savePivot(string $pivotEntity, string $lowEntity, string $entit
     }
 
     /**
+     * Saves model with related id from linked table full duplex
      * @param string $ucEntity
      * @param string $lowEntity
      * @param int|string $eId
@@ -245,7 +246,18 @@ private function saveModel(string $ucEntity, string $lowEntity, $eId, $rId) : vo
     {
         $relEntity = Classes::getModelEntity($ucEntity);
         $model     = $this->getModelEntity($relEntity, $rId);
+        // swap table and field trying to find rels with inverse
+        if (!property_exists($model, $lowEntity . PhpInterface::UNDERSCORE . RamlInterface::RAML_ID)) {
+            $ucTmp     = $ucEntity;
+            $ucEntity  = ucfirst($lowEntity);
+            $relEntity = Classes::getModelEntity($ucEntity);
+            $model     = $this->getModelEntity($relEntity, $eId);
+            $lowEntity = strtolower($ucTmp);
 
+            $model->{$lowEntity . PhpInterface::UNDERSCORE . RamlInterface::RAML_ID} = $rId;
+            $model->save();
+            return;
+        }
         $model->{$lowEntity . PhpInterface::UNDERSCORE . RamlInterface::RAML_ID} = $eId;
         $model->save();
     }

tests/_data/TagFixture.php

@@ -0,0 +1,27 @@
+<?php
+namespace rjapitest\_data;
+
+use Modules\V2\Entities\Tag;
+
+class TagFixture
+{
+    /**
+     * @return Tag
+     */
+    public static function createAndGet() : Tag
+    {
+        $user              = new Tag();
+        $user->title  = 'Foo Bar Baz';
+        $user->save();
+
+        return $user;
+    }
+
+    /**
+     * @param $id
+     */
+    public static function delete($id) : void
+    {
+        Tag::destroy($id);
+    }
+}

tests/_data/TopicFixture.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace rjapitest\_data;
+
+use Modules\V2\Entities\Topic;
+
+class TopicFixture
+{
+    /**
+     * @return Topic
+     */
+    public static function createAndGet() : Topic
+    {
+        $user        = new Topic();
+        $user->title = 'Foo Bar Baz';
+        $user->save();
+
+        return $user;
+    }
+
+    /**
+     * @param $id
+     */
+    public static function delete($id) : void
+    {
+        Topic::destroy($id);
+    }
+}

tests/unit/extension/BaseRelationsTraitTest.php

@@ -0,0 +1,57 @@
+<?php
+
+namespace rjapitest\unit\extensions;
+
+
+use Modules\V2\Entities\Article;
+use rjapi\extension\BaseRelationsTrait;
+use rjapi\types\RamlInterface;
+use rjapitest\_data\ArticleFixture;
+use rjapitest\_data\TopicFixture;
+use rjapitest\unit\TestCase;
+
+
+class BaseRelationsTraitTest extends TestCase
+{
+    use BaseRelationsTrait;
+
+    private $article;
+    private $entity;
+    private $topic;
+    private $modelEntity;
+
+    public function setUp()
+    {
+        parent::setUp();
+        $this->article = ArticleFixture::createAndGet();
+        $this->topic   = TopicFixture::createAndGet();
+        $this->entity  = 'Article';
+        $this->modelEntity = Article::class;
+    }
+
+    /**
+     * @test
+     */
+    public function it_sets_relationships()
+    {
+        $this->setRelationships([
+            RamlInterface::RAML_DATA => [
+                'type'                            => 'article',
+                'id'                              => $this->article->id,
+                RamlInterface::RAML_RELATIONSHIPS => [
+                    'topic' => [
+                        'data' => ['type' => 'topic', 'id' => $this->topic->id]
+                    ],
+                ],
+            ]], $this->article->id);
+        $article = $this->getEntity($this->article->id);
+        $this->assertEquals($this->topic->id, $article->topic_id);
+    }
+
+    public function tearDown()
+    {
+        ArticleFixture::delete($this->article->id);
+        TopicFixture::delete($this->topic->id);
+        parent::tearDown();
+    }
+}