TarfinLabs\Parasut\Tests\ContactTest

Complexity

Total Complexity

3

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Importance

Changes	8
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	user_can_create_a_new_contact()	0	18	1
A	user_can_view_a_contact()	0	16	1
A	user_can_list_contacts()	0	10	1

<?php

namespace TarfinLabs\Parasut\Tests;

use Faker\Factory;
use TarfinLabs\Parasut\Models\Contact;
use TarfinLabs\Parasut\Repositories\ContactRepository;
use TarfinLabs\Parasut\Tests\Mocks\ParasutMock;

class ContactTest extends TestCase
{
    /** @test */
    public function user_can_list_contacts(): void
    {
        ParasutMock::allContacts();

        $contactRepository = new ContactRepository();

        $contacts = $contactRepository->all();

        $this->assertNotNull(Contact::all());
        $this->assertInstanceOf(Contact::class, $contacts->first());
    }

    /** @test */
    public function user_can_create_a_new_contact(): void
    {
        $contact = factory(Contact::class)->make();

        ParasutMock::createContact($contact);

It seems like $contact can also be of type Illuminate\Database\Eloquent\Collection; however, parameter $contact of TarfinLabs\Parasut\Tests\Mocks\ParasutMock::createContact() does only seem to accept TarfinLabs\Parasut\Models\Contact, maybe add an additional type check?

        $contactRepository = new ContactRepository();

        $contactReturned = $contactRepository->create($contact);

It seems like $contact can also be of type Illuminate\Database\Eloquent\Collection; however, parameter $model of TarfinLabs\Parasut\Repositories\BaseRepository::create() does only seem to accept TarfinLabs\Parasut\Models\BaseModel, maybe add an additional type check?

        $this->assertInstanceOf(
            Contact::class,
            $contact
        );

        $this->assertEquals(
            $contact->name,
            $contactReturned->name

The property name does not seem to exist on TarfinLabs\Parasut\Models\BaseModel. Are you sure there is no database migration missing?

        );
    }

    /** @test */
    public function user_can_view_a_contact(): void
    {
        $contactId = ParasutMock::findContact();

        $contactRepository = new ContactRepository();

        $contact = $contactRepository->find($contactId);

        $this->assertInstanceOf(
            Contact::class,
            $contact
        );

        $this->assertEquals(
            $contactId,
            $contact->id
        );
    }
}