DMSDocumentCartControllerTest::testView() - Code Metrics - Inspection of "FIX Add maximum quantity allowed to validation err..." - silverstripe/silverstripe-dms-cart - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#38)

by Robbie

created 2017-06-07 00:24 UTC

DMSDocumentCartControllerTest::testView() A

↳ Parent: DMSDocumentCartControllerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	6
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	4
nc	1
nop	0

<?php

/**
 * Class DMSDocumentCartControllerTest contains all the tests for {@link DMSDocumentCartController}
 */
class DMSDocumentCartControllerTest extends FunctionalTest
namespace YourVendor;

class YourClass { }
{
    protected static $fixture_file = 'dms-cart/tests/DMSDocumentCartTest.yml';

    /**
     * @var DMSDocumentCartController
     */
    protected $controller;

    /**
     * @var DMSDocumentCart
     */
    protected $cart;

    public function setUp()
    {
        parent::setUp();
        DMSDocumentCartController::add_extension('StubDMSDocumentCheckoutPageExtension');
        $this->controller = DMSDocumentCartController::create();
        $this->cart = $this->controller->getCart();
    }

    /**
     * Test the items method of the controller
     */
    public function testItems()
    {
        /** @var DMSDocument $doc1 */
        $doc1 = $this->objFromFixture('DMSDocument', 'doc1');
        /** @var DMSDocument $doc2 */
        $doc2 = $this->objFromFixture('DMSDocument', 'doc2');
        /** @var DMSRequestItem $item */
        $item1 = DMSRequestItem::create()->setDocument($doc1)->setQuantity(2);
        $item2 = DMSRequestItem::create()->setDocument($doc2)->setQuantity(5);
        $this->cart->addItem($item1);
        $this->cart->addItem($item2);
        $this->assertInstanceOf(

            'ArrayList',
            $this->controller->items(),
            'DMSDocumentCartController->Items() returned an ArrayList'
        );
        $this->assertCount(

            2,
            $this->controller->items(),
            'DMSDocumentCartController->Items()->count() returned the requisite number of items'
        );
    }

    public function testReceiverInfo()
    {
        // Now add some info
        $this->cart->setReceiverInfo(array('Name' => 'Joe', 'Surname' => 'Soap'));
        $this->assertCount(2, $this->controller->getReceiverInfo());

    }

    public function testIsCartEmpty()
    {
        $this->assertTrue($this->controller->getIsCartEmpty());

        /** @var DMSDocument $doc */
        $doc1 = $this->objFromFixture('DMSDocument', 'doc1');
        /** @var DMSRequestItem $item */
        $item1 = DMSRequestItem::create()->setDocument($doc1)->setQuantity(2);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->cart->addItem($item1);
        $this->assertFalse($this->controller->getIsCartEmpty());

    }

    public function testAdd()
    {
        $this->logInWithPermission();
        $doc1 = $this->objFromFixture('DMSDocument', 'doc1');
        // Check cart is initially empty
        $this->assertTrue($this->controller->getIsCartEmpty());

        // Now call controller add
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity' => 5));
        $request->setRouteParams(array('ID' => $doc1->ID));
        $this->controller->add($request);
        $this->assertFalse($this->controller->getIsCartEmpty());


        // Do it again and assert quantity was updated
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity' => 7));
        $request->setRouteParams(array('ID' => $doc1->ID));
        $this->controller->add($request);

        // Test ajax
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity' => 7, 'ajax' => 1));
        $request->setRouteParams(array('ID' => $doc1->ID));
        $response = $this->controller->add($request);
        $this->assertTrue($request->isAjax());

        $this->assertJson($response, 'Confirmed that an ajax call to add() responded with json JSON');


        $item = $this->cart->getItem($doc1->ID);
        $this->assertEquals(19, $item->getQuantity());

    }



    public function testDeduct()
    {
        $doc1 = $this->objFromFixture('DMSDocument', 'doc1');
        // Check catty is initially empty
        $this->assertTrue($this->controller->getIsCartEmpty());

        // Now call controller add
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity'=>5));
        $request->setRouteParams(array('ID' => $doc1->ID));
        $this->controller->deduct($request);
        // Assert cart still empty because item doesn't exist
        $this->assertTrue($this->controller->getIsCartEmpty());

        // Now add item
        $this->controller->add($request);
        $this->assertFalse($this->controller->getIsCartEmpty());



        // Now try and deduct 2 from the items
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity' => -2));
        $request->setRouteParams(array('ID' => $doc1->ID));
        $this->controller->deduct($request);

        $item = $this->cart->getItem($doc1->ID);
        $this->assertEquals(3, $item->getQuantity());


        // Test ajax
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity' => 7, 'ajax' => 1));
        $request->setRouteParams(array('ID' => $doc1->ID));
        $response = $this->controller->deduct($request);
        $this->assertTrue($request->isAjax());

        $this->assertJson($response, 'Confirmed that an ajax call to deduct() method responded with JSON');

    }

    public function testRemove()
    {
        $doc1 = $this->objFromFixture('DMSDocument', 'doc1');
        // Check catty is initially empty
        $this->assertTrue($this->controller->getIsCartEmpty());

        // Now call controller add
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity'=>5));
        $request->setRouteParams(array('ID' => $doc1->ID));
        // Now add item
        $this->controller->add($request);
        $this->assertFalse($this->controller->getIsCartEmpty());

        $this->controller->remove($request);
        $this->assertTrue($this->controller->getIsCartEmpty());


        // Test ajax
        $request = new SS_HTTPRequest('POST', '', array(), array('quantity' => 7, 'ajax' => 1));
        $request->setRouteParams(array('ID' => $doc1->ID));
        $response = $this->controller->remove($request);
        $this->assertTrue($request->isAjax());

        $this->assertJson($response, 'Confirmed that  an ajax call to remove() method responded with JSON');

    }

    /**
     * Ensure that a validation error is shown when requesting to add more of a document that is allowed
     */
    public function testCannotAddMoreThanSuggestedQuantityOfItem()
    {
        $document = $this->objFromFixture('DMSDocument', 'limited_supply');
        $result = $this->get('/documentcart/add/' . $document->ID . '?quantity=5&ajax=1');
        $this->assertContains(
            'Maximum of 3 documents exceeded for \"Doc3\", please select a lower quantity.',
            (string) $result->getBody()
        );
    }

    /**
     * Ensure that when a document that cannot be added to the cart is added to the cart, a validation error is
     * returned
     */
    public function testValidationErrorReturnedOnInvalidAdd()
    {
        $document = $this->objFromFixture('DMSDocument', 'not_allowed_in_cart');
        $result = $this->get('/documentcart/add/' . $document->ID . '?ajax=1');
        $this->assertContains('You are not allowed to add this document', (string) $result->getBody());
    }

    /**
     * Tests whether the cart items are updated from the controller
     */
    public function testUpdateCartItems()
    {
        $doc = $this->objFromFixture('DMSDocument', 'doc1');
        /** @var DMSRequestItem $item */
        $item = DMSRequestItem::create()->setDocument($doc)->setQuantity(2);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->cart->addItem($item);

        $invalidQuantities = array(
            'ItemQuantity' => array($doc->ID => 'non-numeric')
        );

        $sameQuantities = array(
            'ItemQuantity' => array($doc->ID => 2)
        );

        $updatedQuantities = array(
            'ItemQuantity' => array($doc->ID => 5),
        );
        $form = Form::create(
            $this->controller,
            'Test',
            FieldList::create(),
            FieldList::create()
        );
        $request = new SS_HTTPRequest('POST', '');
        // Test invalids leave it unchanged
        $response = $this->controller->updateCartItems($invalidQuantities, $form, $request);
        $this->assertEquals(2, $this->cart->getItem($item->getItemId())->getQuantity());


        // Test quantity remains the same
        $response->removeHeader('Location');
        $response =$this->controller->updateCartItems($sameQuantities, $form, $request);
        $this->assertEquals(2, $this->cart->getItem($item->getItemId())->getQuantity());


        $response->removeHeader('Location');
        $response = $this->controller->updateCartItems($updatedQuantities, $form, $request);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $this->assertEquals(5, $this->cart->getItem($item->getItemId())->getQuantity());

    }

    /**
     * Tests DMSCartEditForm form has a FieldList
     */
    public function testDMSCartEditForm()
    {
        $form = $this->controller->DMSCartEditForm();
        $this->assertInstanceOf('FieldList', $form->Fields());

    }

    /**
     * Tests if the DMSCartEditForm is extensible
     */
    public function testDMSCartEditFormIsExtensible()
    {
        $controller = $this->controller;
        $form = $controller->DMSCartEditForm();
        $this->assertNotNull(

            $form->Fields()->fieldByName('NewTextField'),
            'DMSDocumentRequestForm() is extensible as it included the field from the extension'
        );
    }

    /**
     * Tests that the cart summary view is returned.
     */
    public function testView()
    {
        $result = $this->get('documentcart/view');
        $this->assertInstanceOf('SS_HTTPResponse', $result);

        $this->assertContains('Updating cart items', $result->getBody());
    }

    /**
     * Ensure the link is "friendly", not a class name
     */
    public function testLink()
    {
        $this->assertSame('documentcart', $this->controller->Link());

        $this->assertSame('documentcart/view', $this->controller->Link('view'));

    }
}


1			<?php
2
3			/**
4			* Class DMSDocumentCartControllerTest contains all the tests for {@link DMSDocumentCartController}
5			*/
6			class DMSDocumentCartControllerTest extends FunctionalTest
			0 ignored issues – show Coding Style Compatibility introduced 2017-05-23 04:54 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
7			{
8			protected static $fixture_file = 'dms-cart/tests/DMSDocumentCartTest.yml';
9
10			/**
11			* @var DMSDocumentCartController
12			*/
13			protected $controller;
14
15			/**
16			* @var DMSDocumentCart
17			*/
18			protected $cart;
19
20			public function setUp()
21			{
22			parent::setUp();
23			DMSDocumentCartController::add_extension('StubDMSDocumentCheckoutPageExtension');
24			$this->controller = DMSDocumentCartController::create();
25			$this->cart = $this->controller->getCart();
26			}
27
28			/**
29			* Test the items method of the controller
30			*/
31			public function testItems()
32			{
33			/** @var DMSDocument $doc1 */
34			$doc1 = $this->objFromFixture('DMSDocument', 'doc1');
35			/** @var DMSDocument $doc2 */
36			$doc2 = $this->objFromFixture('DMSDocument', 'doc2');
37			/** @var DMSRequestItem $item */
38			$item1 = DMSRequestItem::create()->setDocument($doc1)->setQuantity(2);
39			$item2 = DMSRequestItem::create()->setDocument($doc2)->setQuantity(5);

silverstripe / silverstripe-dms-cart

Pull Request — master (#38)

DMSDocumentCartControllerTest::testView() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like