VGirol\JsonApiFaker\Factory\HasRelationships

Complexity

Total Complexity

4

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	addRelationship()	0	5	1
A	getRelationships()	0	3	1
A	fakeRelationships()	0	12	2

<?php

declare(strict_types=1);

namespace VGirol\JsonApiFaker\Factory;

use VGirol\JsonApiConstant\Members;
use VGirol\JsonApiFaker\Contract\RelationshipContract;

/**
 * Add "relationships" member to a factory.
 */
trait HasRelationships
{
    /**
     * The "relationships" member
     *
     * @var array An array of RelationshipFactory
     */
    protected $relationships;

    /**
     * Get the relationships collection
     *
     * @return array|null
     */
    public function getRelationships(): ?array
    {
        return $this->relationships;
    }

    /**
     * Add a single relationship
     *
     * @param string $name
     * @param RelationshipContract $relationship
     *
     * @return static
     */
    public function addRelationship(string $name, $relationship)
    {
        $this->addToObject(Members::RELATIONSHIPS, $name, $relationship);

It seems like addToObject() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this;
    }

    /**
     * Fill the "relationships" member with fake values.
     *
     * @param integer $count The number of relationships to generate.
     *
     * @return static
     */
    public function fakeRelationships($count = 2)
    {
        $faker = \Faker\Factory::create();

        for ($i = 0; $i < $count; $i++) {
            $this->addRelationship(
                $faker->unique()->numerify('relationship##'),
                $this->generator->relationship()->fake()
            );
        }

        return $this;
    }
}