嘲笑數據庫查詢laravel嘲弄

Question

我試圖用PhpUnit / Mockery / Laravel來圍繞單元測試。 這並不容易。 我一直在閱讀幾十個教程，但仍然不能在現實生活場景中應用它。

我將介紹一些我想測試的代碼。 誰能指點我如何測試類SoldProductModifier的方法modifyBasedOnItemCode（） ？

首先解釋幾句：我希望用戶能夠輸入產品代碼（項目代碼）和數量，我希望系統自動更新product_id以及SoldProduct模型的category_id屬性。 為此我創建了我現在要測試的類。

另請參閱： 我的數據庫的簡化圖（僅與我的問題相關的表）

現在相關代碼：

要測試的類

use App\Models\Product;

    class SoldProductModifier 
    {
        private $sold_product;

        public function __construct(SoldProduct $sold_product) 
        {
            $this->sold_product = $sold_product;
        }

        public function modifyBasedOnItemCode($item_code)
        {
            if (! isset($item_code) || $item_code == '')
            {
                $product = Product::findByItemCode($item_code); 

                if (isset($product) && $product != false)
                {
                    $this->sold_product->category_id = $product->category->id;
                    $this->sold_product->product_id = $product->id;
                }
            }

            return $this->sold_product;
        }
    }

產品型號

...

    public static function findByItemCode($item_code) {
        return self::where('item_code', $item_code)->first();
    }

    ...

我的控制器引用SUT

...

    $sold_product = new SoldProduct($request->all());

    $modifier = new SoldProductModifier($sold_product);
    $sold_product = $modifier->modifyBasedOnItemCode($request->item_code);

    $sold_product->save();

    ...

我的考試班

class SoldProductModifierTest extends TestCase {


        public function setUp()
        {
            parent::setUp();

            $this->soldProductMock = $this->mock('App\Models\SoldProduct');
            $this->productMock = $this->mock('App\Models\Product');
        }

        public function tearDown()
        {
            Mockery::close();
        }


        public function testDoesNotModifyIfItemCodeEmpty()
        {
            $soldProductModifier = new SoldProductModifier($this->soldProductMock);

            $modifiedSoldProduct = $soldProductModifier->modifyBasedOnItemCode('');

            $this->assertEquals($this->soldProductMock, $modifiedSoldProduct);
        }

        public function testModifiesBasedOnItemCode() 
        {
           // how do I test positive case scenario ?
    ...

我粘貼了我的第一個測試，以防有人認為它不是應該做的方式，並且會建議另一種方法來接近這個。

但現在問我的問題：

我如何在這里模擬對數據庫的調用：Product :: findByItemCode（$ item_code）？

我應該在SoldProductModifier中創建$ product屬性，並使用為此目的創建的setter方法設置它，例如：

public function setProduct(Product $product)
    {
        $this->product = $product;
    }

然后在我的控制器中添加額外的行：

...

        $modifier = new SoldProductModifier($sold_product);
        $modifier->setProduct(Product::findByItemCode($item_code)); // --- extra line 
        $sold_product = $modifier->modifyBasedOnItemCode(); // --- parameter removed


    ...

？

我盡量讓控制器保持苗條，所以想避免這種情況？ 那么解決這種情況的最佳方法是什么？

謝謝

Answer 1

您應該通過構造函數注入Product ，以便Laravel可以為您處理。

use App\Models\Product;

class SoldProductModifier 
{
    private $sold_product;

    protected $product;

    public function __construct(SoldProduct $sold_product, Product $product) 
    {
        $this->sold_product = $sold_product;

        $this->product = $product;
    }
}

現在，您需要通過函數為每個“路徑”編寫一個單元測試。

// Build your mock object.
$mockProduct = Mockery::mock(new App\Models\Product);

// Have Laravel return the mocked object instead of the actual model.
$this->app->instance('App\Models\Product', $mockProduct);

// Tell your mocked instance what methods it should receive.
$mockProduct
    ->shouldReceive('findByItemCode')
    ->once()
    ->andReturn(false);

// Now you can instantiate your class and call the methods on it to be sure it's returning items and setting class properties correctly.

你應該多次編寫這個測試並讓你的$mockProduct返回不同的東西，直到覆蓋了所有的代碼行。 例如，您可能想要執行以下操作...

$product = new stdClass;
$product->id = 45;

$category = new stdClass;
$category-id = 60;

$product->category = $category;

$mockProduct
    ->shouldReceive('findByItemCode')
    ->once()
    ->andReturn($product);

現在，在函數運行之后，您需要確保sold_product->category_id等於60且sold_product->product_id等於45.如果它們是私有的且您無法從測試中檢查它們，您可能希望為這些對象編寫一個getter，這樣你就可以更容易地從測試中看到它們的值。

編輯

關於您的評論，您將使用以下內容。

new SoldProductModifier($sold_product, new Product);

然后你的功能看起來應該......

public function modifyBasedOnItemCode($item_code)
{
    if (! isset($item_code) || $item_code == '')
    {
        $product = $this->product->findByItemCode($item_code);

        if (isset($product) && $product != false)
        {
            $this->sold_product->category_id = $product->category->id;
            $this->sold_product->product_id = $product->id;
        }
    }

    return $this->sold_product;
}

我看到它是一個靜態函數，所以你可能想要稍微改變一下。 如果它只是因為這個原因是靜態的，那么你就不能讓它靜止。 如果其他東西依賴於它，你可能會創建一個非靜態的新函數，它通過self::findByItemCode($id)調用靜態函數

這里的一般經驗法則是，除非它是在config.php文件中設置的外觀，您應該允許Laravel為您處理注入。 這樣，當您進行測試時，您可以創建模擬對象，然后讓Laravel通過$this->app->instance()了解它們，這樣它就會注入那些代替真實的對象。