tarfin-labs / parasut-laravel

tests/ProductTest.php

<?php

namespace TarfinLabs\Parasut\Tests;

use TarfinLabs\Parasut\Mocks\ProductMock;
use TarfinLabs\Parasut\Models\Product;
use TarfinLabs\Parasut\Repositories\ProductRepository;

class ProductTest extends TestCase
{
    /** @test */
    public function user_can_list_products(): void
    {
        ProductMock::all();

        $productRepository = new ProductRepository();

        $products = $productRepository->all();

        $this->assertNotNull(Product::all());
        $this->assertInstanceOf(Product::class, $products->first());
    }

    /** @test */
    public function user_can_create_a_new_product(): void
    {
        $product = factory(Product::class)
            ->states(['creation', 'response'])
            ->make();

        ProductMock::create($product);

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

        $ProductRepository = new ProductRepository();

        $productReturned = $ProductRepository->create($product);

It seems like $product 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(
            Product::class,
            $product
        );

        $this->assertEquals(
            $product->name,
            $productReturned->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_product(): void
    {
        $productId = ProductMock::find();

        $ProductRepository = new ProductRepository();

        $product = $ProductRepository->find($productId);

        $this->assertInstanceOf(
            Product::class,
            $product
        );

        $this->assertEquals(
            $productId,
            $product->id
        );
    }

    /** @test */
    public function user_can_edit_a_product(): void
    {
        $product = factory(Product::class)
            ->states(['creation', 'response'])
            ->make();

        ProductMock::create($product);

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

        $ProductRepository = new ProductRepository();
        $product = $ProductRepository->create($product);

It seems like $product 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?

        $newProduct = factory(Product::class)->make();
        $newProduct->id = $product->id;

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

        ProductMock::update($product);
        $updatedContact = $ProductRepository->update($newProduct);

It seems like $newProduct 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(
            Product::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?

            $newProduct->name
        );
    }

    /** @test */
    public function user_can_delete_a_product(): void
    {
        $product = factory(Product::class)
            ->states(['creation', 'response'])
            ->make();

        ProductMock::create($product);

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

        $ProductRepository = new ProductRepository();
        $product = $ProductRepository->create($product);

It seems like $product 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?

        ProductMock::delete($product);
        $result = $ProductRepository->delete($product);

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