DMSCheckoutControllerTest::testDoRequestSend() - Code Metrics - silverstripe/silverstripe-dms-cart - Measure and Improve Code Quality continuously with Scrutinizer

DMSCheckoutControllerTest::testDoRequestSend() A
last analyzed 2018-08-17 12:10 UTC

↳ Parent: DMSCheckoutControllerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	25
rs	9.52
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

class DMSCheckoutControllerTest extends FunctionalTest
namespace YourVendor;

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

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

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

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

        DMSCheckoutController::add_extension('StubDMSDocumentCheckoutPageExtension');
        Injector::inst()->registerService(new StubEmail(), 'Email');
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $this->controller = DMSCheckoutController::create();
        $this->cart = $this->controller->getCart();
    }

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

    }

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

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

    /**
     * Tests whether the recipient details are updated from the controller
     */
    public function testUpdateCartReceiverInfo()
    {
        $newInfo = array(
            'ReceiverName'            => 'Joe Soap',
            'ReceiverPhone'           => '111',
            'ReceiverEmail'           => '[email protected]',
            'DeliveryAddressLine1'    => 'A1',
            'DeliveryAddressLine2'    => 'A2',
            'DeliveryAddressCountry'  => 'NZ',
            'DeliveryAddressPostCode' => '6011',
        );
        $this->controller->updateCartReceiverInfo($newInfo);
        $this->assertEquals($newInfo, $this->cart->getReceiverInfo());
    }

    /**
     * Tests whether emails are sent. Emails are mocked so not actually sent.
     */
    public function testSend()
    {
        // Set admin email
        Config::inst()->update('Email', 'admin_email', 'admin');
        $data = array(
            'ReceiverName'            => 'Joe Soap',
            'ReceiverPhone'           => '111',
            'ReceiverEmail'           => '[email protected]',
            'DeliveryAddressLine1'    => 'A1',
            'DeliveryAddressLine2'    => 'A2',
            'DeliveryAddressCountry'  => 'NZ',
            'DeliveryAddressPostCode' => '6011'
        );
        $this->cart->setReceiverInfo($data);
        $result = $this->controller->send();
        $this->assertTrue(is_array($result));
        $this->assertEquals('[email protected]', $result['to']);
        $this->assertEquals('admin', $result['from']);
    }

    /**
     * Tests whether email sending is extensible.
     */
    public function testSendIsExtensible()
    {
        $result = $this->controller->send();
        $this->assertEquals('Subject is changed', $result['subject']);
    }

    /**
     * Test to see whether the cart is empty after a request is sent.
     */
    public function testDoRequestSend()
    {
        // Form for use later
        $form = $this->controller->DMSDocumentRequestForm();
        $doc = $this->objFromFixture('DMSDocument', 'doc1');
        // Add some an item to the cart to assert later that its empty
        $item = DMSRequestItem::create()->setDocument($doc)->setQuantity(15);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->controller->getCart()->addItem($item);
        $data = array(
            'ReceiverName'            => 'Joe Soap',
            'ReceiverPhone'           => '111',
            'ReceiverEmail'           => '[email protected]',
            'DeliveryAddressLine1'    => 'A1',
            'DeliveryAddressLine2'    => 'A2',
            'DeliveryAddressCountry'  => 'NZ',
            'DeliveryAddressPostCode' => '6011',
            'ItemQuantity'            => array($doc->ID => 5),
        );
        $request = new SS_HTTPRequest('POST', 'mock/url');
        // Assert cart is empty
        $this->assertFalse($this->controller->getCart()->isCartEmpty());

        $result = $this->controller->doRequestSend($data, $form, $request);
        $this->assertInstanceOf('SS_HTTPResponse', $result);

        $this->assertTrue($this->controller->getCart()->isCartEmpty());
    }

    /**
     * Test the checkout success page shows a pretty message
     */
    public function testCompletePage()
    {
        $result = (string) $this->get('checkout/complete')->getBody();
        $this->assertContains('Thanks!', $result);
        $this->assertContains('You will receive a confirmation email', $result);
    }

    /**
     * Test that the items in my cart are listed on the checkout page, and that some form fields exist
     */
    public function testIndexCheckoutForm()
    {
        $backend = DMSSessionBackend::singleton();
        $document = $this->objFromFixture('DMSDocument', 'limited_supply');
        $requestItem = DMSRequestItem::create($document);
        $backend->addItem($requestItem);

        $response = $this->get('checkout');
        $this->assertEquals(200, $response->getStatusCode());

        $body = (string) $response->getBody();
        $this->assertContains('Checkout', $body);
        $this->assertContains('Your request in summary', $body);
        $this->assertContains('Doc3', $body);
        $this->assertContains('Receiver Name', $body);
    }

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


1			<?php
2
3			class DMSCheckoutControllerTest extends FunctionalTest
			0 ignored issues – show Coding Style Compatibility introduced 2017-05-26 03:49 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...
4			{
5			protected static $fixture_file = 'dms-cart/tests/DMSDocumentCartTest.yml';
6
7			/**
8			* @var DMSCheckoutController
9			*/
10			protected $controller;
11
12			/**
13			* @var DMSDocumentCart
14			*/
15			protected $cart;
16
17			public function setUp()
18			{
19			parent::setUp();
20
21			DMSCheckoutController::add_extension('StubDMSDocumentCheckoutPageExtension');
22			Injector::inst()->registerService(new StubEmail(), 'Email');
			0 ignored issues – show Documentation introduced 2017-05-24 00:49 UTC by Report Bug Copy Issue Report `new \StubEmail()` is of type `object<StubEmail>`, but the function expects a `object<stdClass>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
23
24			$this->controller = DMSCheckoutController::create();
25			$this->cart = $this->controller->getCart();
26			}
27
28			/**
29			* Tests DMSDocumentRequest form has a FieldList
30			*/
31			public function testDMSDocumentRequestForm()
32			{
33			$form = $this->controller->DMSDocumentRequestForm();
34			$this->assertInstanceOf('FieldList', $form->Fields());
			0 ignored issues – show Bug introduced 2017-05-26 03:49 UTC by Report Bug Copy Issue Report The method `assertInstanceOf()` does not seem to exist on `object<DMSCheckoutControllerTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
35			}
36
37			/**
38			* Tests if the DMSDocumentRequestForm is extensible
39			*/
40			public function testDMSDocumentRequestFormIsExtensible()
41			{
42			$controller = $this->controller;
43			$form = $controller->DMSDocumentRequestForm();
44			$this->assertNotNull(
			0 ignored issues – show Bug introduced 2017-05-26 03:49 UTC by Report Bug Copy Issue Report The method `assertNotNull()` does not seem to exist on `object<DMSCheckoutControllerTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
45			$form->Fields()->fieldByName('NewTextField'),
46			'DMSDocumentRequestForm() is extensible as it included the field from the extension'
47			);
48			}
49
50			/**
51			* Tests whether the recipient details are updated from the controller
52			*/
53			public function testUpdateCartReceiverInfo()
54			{
55			$newInfo = array(
56			'ReceiverName' => 'Joe Soap',
57			'ReceiverPhone' => '111',
58			'ReceiverEmail' => '[email protected]',
59			'DeliveryAddressLine1' => 'A1',
60			'DeliveryAddressLine2' => 'A2',
61			'DeliveryAddressCountry' => 'NZ',
62			'DeliveryAddressPostCode' => '6011',
63			);
64			$this->controller->updateCartReceiverInfo($newInfo);
65			$this->assertEquals($newInfo, $this->cart->getReceiverInfo());
66			}
67
68			/**
69			* Tests whether emails are sent. Emails are mocked so not actually sent.
70			*/
71			public function testSend()
72			{
73			// Set admin email
74			Config::inst()->update('Email', 'admin_email', 'admin');
75			$data = array(
76			'ReceiverName' => 'Joe Soap',
77			'ReceiverPhone' => '111',
78			'ReceiverEmail' => '[email protected]',
79			'DeliveryAddressLine1' => 'A1',
80			'DeliveryAddressLine2' => 'A2',
81			'DeliveryAddressCountry' => 'NZ',
82			'DeliveryAddressPostCode' => '6011'
83			);
84			$this->cart->setReceiverInfo($data);
85			$result = $this->controller->send();
86			$this->assertTrue(is_array($result));
87			$this->assertEquals('[email protected]', $result['to']);
88			$this->assertEquals('admin', $result['from']);
89			}
90
91			/**
92			* Tests whether email sending is extensible.
93			*/
94			public function testSendIsExtensible()
95			{
96			$result = $this->controller->send();
97			$this->assertEquals('Subject is changed', $result['subject']);
98			}
99
100			/**
101			* Test to see whether the cart is empty after a request is sent.
102			*/
103			public function testDoRequestSend()
104			{
105			// Form for use later
106			$form = $this->controller->DMSDocumentRequestForm();
107			$doc = $this->objFromFixture('DMSDocument', 'doc1');
108			// Add some an item to the cart to assert later that its empty
109			$item = DMSRequestItem::create()->setDocument($doc)->setQuantity(15);
			0 ignored issues – show Documentation introduced 2017-05-24 00:49 UTC by Report Bug Copy Issue Report `$doc` is of type `object<DataObject>\|null`, but the function expects a `object<DMSDocument>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
110			$this->controller->getCart()->addItem($item);
111			$data = array(
112			'ReceiverName' => 'Joe Soap',
113			'ReceiverPhone' => '111',
114			'ReceiverEmail' => '[email protected]',
115			'DeliveryAddressLine1' => 'A1',
116			'DeliveryAddressLine2' => 'A2',
117			'DeliveryAddressCountry' => 'NZ',
118			'DeliveryAddressPostCode' => '6011',
119			'ItemQuantity' => array($doc->ID => 5),
120			);
121			$request = new SS_HTTPRequest('POST', 'mock/url');
122			// Assert cart is empty
123			$this->assertFalse($this->controller->getCart()->isCartEmpty());
			0 ignored issues – show Bug introduced 2017-05-26 03:49 UTC by Report Bug Copy Issue Report The method `assertFalse()` does not seem to exist on `object<DMSCheckoutControllerTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
124			$result = $this->controller->doRequestSend($data, $form, $request);
125			$this->assertInstanceOf('SS_HTTPResponse', $result);
			0 ignored issues – show Bug introduced 2017-05-26 03:49 UTC by Report Bug Copy Issue Report The method `assertInstanceOf()` does not seem to exist on `object<DMSCheckoutControllerTest>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
126			$this->assertTrue($this->controller->getCart()->isCartEmpty());
127			}
128
129			/**
130			* Test the checkout success page shows a pretty message
131			*/
132			public function testCompletePage()
133			{
134			$result = (string) $this->get('checkout/complete')->getBody();
135			$this->assertContains('Thanks!', $result);
136			$this->assertContains('You will receive a confirmation email', $result);
137			}
138
139			/**
140			* Test that the items in my cart are listed on the checkout page, and that some form fields exist
141			*/
142			public function testIndexCheckoutForm()
143			{
144			$backend = DMSSessionBackend::singleton();
145			$document = $this->objFromFixture('DMSDocument', 'limited_supply');
146			$requestItem = DMSRequestItem::create($document);
147			$backend->addItem($requestItem);
148
149			$response = $this->get('checkout');
150			$this->assertEquals(200, $response->getStatusCode());
151
152			$body = (string) $response->getBody();
153			$this->assertContains('Checkout', $body);
154			$this->assertContains('Your request in summary', $body);
155			$this->assertContains('Doc3', $body);
156			$this->assertContains('Receiver Name', $body);
157			}
158
159			/**
160			* Ensure the link is "friendly", not a class name
161			*/
162			public function testLink()
163			{
164			$this->assertSame('checkout', $this->controller->Link());
165			$this->assertSame('checkout/complete', $this->controller->Link('complete'));
166			}
167			}
168

silverstripe / silverstripe-dms-cart

DMSCheckoutControllerTest::testDoRequestSend() A last analyzed 2018-08-17 12:10 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

DMSCheckoutControllerTest::testDoRequestSend() A
last analyzed 2018-08-17 12:10 UTC