SilvershopVariationsJsonResponseTest::testVariations()

Complexity

Conditions	1
Paths	1

Size

Total Lines	59
Code Lines	41

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function testVariations(): void
    {
        $ballRoot = $this->objFromFixture(Product::class, 'ball');
        // add parent for categories in JSON response
        $parent = $this->objFromFixture(ProductCategory::class, 'products');
        $ballRoot->ParentID = $parent->ID;
        $ballRoot->write();

        $ballRoot->copyVersionToStage('Stage', 'Live');
        $ball1 = $this->objFromFixture(Variation::class, 'redLarge');
        $ball2 = $this->objFromFixture(Variation::class, 'redSmall');
        $this->logInWithPermission('ADMIN');
        $ball1->publishSingle();
        $ball2->publishSingle();

        // Add the two variation items
        $response = $this->get(ShoppingCartController::add_item_link($ball1) . "?ajax=1");

$ball1 of type SilverStripe\ORM\DataObject is incompatible with the type SilverShop\Model\Buyable expected by parameter $buyable of SilverShop\Cart\ShoppingCartController::add_item_link().

Are you sure SilverShop\Cart\ShoppingCartController::add_item_link($ball1) of type false|string can be used in concatenation?

        $this->assertEquals(
            200,
            $response->getStatusCode(),
            "Response status code is 200"
        );
        $this->assertEquals(
            "application/json; charset=utf-8",
            $response->getHeader("Content-Type"),
            "Json response header"
        );
        $this->assertJson(
            $response->getBody(),
            "Contains json in the body of the response"
        );
        $this->assertStringContainsString(
            "Size:Large, Color:Red",
            $response->getBody(),
            "Contains json in the body of the cart id"
        );

        $response = $this->get(ShoppingCartController::add_item_link($ball2) . "?ajax=1");

The assignment to $response is dead and can be removed.

Are you sure SilverShop\Cart\ShoppingCartController::add_item_link($ball2) of type false|string can be used in concatenation?

        $items = ShoppingCart::curr()->Items();
        $this->assertEquals(
            $items->Count(),
            2,
            "There are 2 items in the cart"
        );

        // Remove one and see what happens
        $this->get(ShoppingCartController::remove_all_item_link($ball1) . "?ajax=1");

Are you sure SilverShop\Cart\ShoppingCartController::remove_all_item_link($ball1) of type false|string can be used in concatenation?

$ball1 of type SilverStripe\ORM\DataObject is incompatible with the type SilverShop\Model\Buyable expected by parameter $buyable of SilverShop\Cart\ShoppingCartController::remove_all_item_link().

        $this->assertEquals(
            $items->Count(),
            1,
            "There is 1 item in the cart"
        );
        $this->assertNull(
            $this->cart->get($ball1),
            "first item not in cart"
        );
        $this->assertNotNull(
            $this->cart->get($ball2),
            "second item is still in cart"
        );