TarfinLabs\Parasut\Tests\ContactTest

Complexity

Total Complexity

5

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Importance

Changes	10
Bugs	0	Features	0

5 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
A	user_can_edit_a_contact()	0	22	1
A	user_can_delete_a_contact()	0	12	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
        );
    }

    /** @test */
    public function user_can_edit_a_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();
        $contact = $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?

        $newContact = factory(Contact::class)->make();
        $newContact->id = $contact->id;

The property id does not seem to exist on Illuminate\Database\Eloquent\Collection.

        ParasutMock::updateContact($contact);
        $updatedContact = $contactRepository->update($newContact);

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

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

        $this->assertEquals(
            $updatedContact->name,

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

            $newContact->name
        );
    }

    /** @test */
    public function user_can_delete_a_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();
        $contact = $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?

        ParasutMock::deleteContact($contact);
        $result = $contactRepository->delete($contact);

        $this->assertTrue($result);
    }
}