OrderTest_TestStatusChangeExtension::reset() - Code Metrics - Inspection of "Merge pull request #509 from bummzack/fix-496-part..." - silvershop/silvershop-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 2.0 ( 6c824d...3cf5ce )

by Roman

created 2016-06-24 14:16 UTC

OrderTest_TestStatusChangeExtension::reset() A

↳ Parent: OrderTest_TestStatusChangeExtension

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
c	1
b	0
f	1
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	0

<?php

/**
 * Order Unit Tests
 *
 * @link       Order
 * @package    shop
 * @subpackage tests
 */
class OrderTest extends SapphireTest
{
    public static $fixture_file = 'silvershop/tests/fixtures/shop.yml';

    public function setUp()
    {
        parent::setUp();
        ShopTest::setConfiguration();
        $this->mp3player = $this->objFromFixture('Product', 'mp3player');
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->mp3player->publish('Stage', 'Live');
        $this->socks = $this->objFromFixture('Product', 'socks');
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->socks->publish('Stage', 'Live');
        $this->beachball = $this->objFromFixture('Product', 'beachball');
class MyClass { }

$x = new MyClass();
$x->foo = true;
        $this->beachball->publish('Stage', 'Live');
    }

    public function tearDown()
    {
        parent::tearDown();
        unset($this->mp3player);
        unset($this->socks);
        unset($this->beachball);
    }

    public function testCMSFields()
    {
        singleton('Order')->getCMSFields();
        $this->markTestIncomplete('assertions!');

    }

    public function testSearchFields()
    {
        singleton('Order')->scaffoldSearchFields();
        $this->markTestIncomplete('assertions!');

    }

    public function testDebug()
    {
        $order = $this->objFromFixture("Order", "cart");
        $order->debug();
        $this->markTestIncomplete('assertions!');

    }

    public function testOrderItems()
    {
        $order = $this->objFromFixture("Order", "paid");
        $items = $order->Items();
        $this->assertNotNull($items);

        $this->assertDOSEquals(
            array(
                array('ProductID' => $this->mp3player->ID, 'Quantity' => 2, 'CalculatedTotal' => 400),
                array('ProductID' => $this->socks->ID, 'Quantity' => 1, 'CalculatedTotal' => 8),
            ),
            $items
        );
        $this->assertEquals(3, $items->Quantity(), "Quantity is 3");

        $this->assertTrue($items->Plural(), "There is more than one item");

        $this->assertEquals(0.7, $items->Sum('Weight', true), "Total order weight sums correctly", 0.0001);

    }

    public function testTotals()
    {
        $order = $this->objFromFixture("Order", "paid");
        $this->assertEquals(408, $order->SubTotal(), "Subtotal is correct"); // 200 + 200 + 8

        $this->assertEquals(408, $order->GrandTotal(), "Grand total is correct");

        $this->assertEquals(200, $order->TotalPaid(), "Outstanding total is correct");

        $this->assertEquals(208, $order->TotalOutstanding(), "Outstanding total is correct");

    }

    public function testRounding()
    {
        //create an order with unrounded total
        $order = new Order(
            array(
                'Total'  => 123.257323,
                //NOTE: setTotal isn't called here, so un-rounded data *could* get in to the object
                'Status' => 'Unpaid',
            )
        );
        $order->Total = 123.257323; //setTotal IS called here
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
        $this->assertEquals(123.26, $order->Total(), "Check total rounds appropriately");

        $this->assertEquals(123.26, $order->TotalOutstanding(), "Check total outstanding rounds appropriately");

    }

    public function testPlacedOrderImmutability()
    {

        $order = $this->objFromFixture("Order", "paid");
        $processor = OrderProcessor::create($order)->placeOrder();
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertEquals(408, $order->Total(), "check totals");


        //make a changes to existing products
        $this->mp3player->BasePrice = 100;
        $this->mp3player->write();
        $this->socks->BasePrice = 20;
        $this->socks->write();

        //total doesn't change
        $this->assertEquals(408, $order->Total());

        $this->assertFalse($order->isCart());


        //item values don't change
        $items = $order->Items()
            //hack join to make thigns work
            ->innerJoin(
                "Product_OrderItem",
                '"OrderItem"."ID" = "Product_OrderItem"."ID"'
            );
        $this->assertNotNull($items);

        $this->assertDOSEquals(
            array(
                array('ProductID' => $this->mp3player->ID, 'Quantity' => 2, 'CalculatedTotal' => 400),
                array('ProductID' => $this->socks->ID, 'Quantity' => 1, 'CalculatedTotal' => 8),
            ),
            $items
        );

        $mp3player = $items->find('ProductID', $this->mp3player->ID);//join needed to provide ProductID
        $this->assertNotNull($mp3player, "MP3 player is in order");

        $this->assertEquals(200, $mp3player->UnitPrice(), "Unit price remains the same");

        $this->assertEquals(400, $mp3player->Total(), "Total remains the same");


        $socks = $items->find('ProductID', $this->socks->ID);
        $this->assertNotNull($socks, "Socks are in order");

        $this->assertEquals(8, $socks->UnitPrice(), "Unit price remains the same");

        $this->assertEquals(8, $socks->Total(), "Total remains the same");

    }

    public function testCanFunctions()
    {
        $order = $this->objFromFixture("Order", "cart");
        $order->calculate();
        $this->assertTrue($order->canPay(), "can pay when order is in cart");

        $this->assertFalse($order->canCancel(), "can't cancel when order is in cart");

        $this->assertFalse($order->canDelete(), "never allow deleting orders");

        $this->assertTrue($order->canEdit(), "orders can be edited by anyone");

        $this->assertFalse($order->canCreate(), "no body can create orders manually");


        $order = $this->objFromFixture("Order", "unpaid");
        $this->assertTrue($order->canPay(), "can pay an order that is unpaid");

        $this->assertTrue($order->canCancel());

        $this->assertFalse($order->canDelete(), "never allow deleting orders");


        // Override config
        Config::inst()->update('Order', 'cancel_before_payment', false);
        $this->assertFalse($order->canCancel());


        $order = $this->objFromFixture("Order", "paid");
        $this->assertFalse($order->canPay(), "paid order can't be paid for");

        $this->assertFalse($order->canCancel(), "paid order can't be cancelled");

        $this->assertFalse($order->canDelete(), "never allow deleting orders");


        Config::inst()->update('Order', 'cancel_before_processing', true);
        $this->assertTrue($order->canCancel(), "paid order can be cancelled when expcicitly set via config");


        $order->Status = 'Processing';
        $this->assertFalse($order->canPay(), "Processing order can't be paid for");

        $this->assertFalse($order->canCancel(), "Processing order can't be cancelled");

        $this->assertFalse($order->canDelete(), "never allow deleting orders");


        Config::inst()->update('Order', 'cancel_before_sending', true);
        $this->assertTrue($order->canCancel(), "Processing order can be cancelled when expcicitly set via config");


        $order->Status = 'Sent';
        $this->assertFalse($order->canPay(), "Sent order can't be paid for");

        $this->assertFalse($order->canCancel(), "Sent order can't be cancelled");

        $this->assertFalse($order->canDelete(), "never allow deleting orders");


        Config::inst()->update('Order', 'cancel_after_sending', true);
        $this->assertTrue($order->canCancel(), "Sent order can be cancelled when expcicitly set via config");

        Config::inst()->update('Order', 'cancel_after_sending', false);

        $order->Status = 'Complete';
        $this->assertFalse($order->canPay(), "Complete order can't be paid for");

        $this->assertFalse($order->canCancel(), "Complete order can't be cancelled");

        $this->assertFalse($order->canDelete(), "never allow deleting orders");


        Config::inst()->update('Order', 'cancel_after_sending', true);
        $this->assertTrue($order->canCancel(), "Completed order can be cancelled when expcicitly set via config");


        $order->Status = 'AdminCancelled';
        $this->assertFalse($order->canPay(), "Cancelled order can't be paid for");

        $this->assertFalse($order->canCancel(), "Cancelled order can't be cancelled");

        $this->assertFalse($order->canDelete(), "never allow deleting orders");


        $order->Status = 'MemberCancelled';
        $this->assertFalse($order->canPay(), "Cancelled order can't be paid for");

        $this->assertFalse($order->canCancel(), "Cancelled order can't be cancelled");

        $this->assertFalse($order->canDelete(), "never allow deleting orders");

    }

    public function testDelete()
    {
        Config::inst()->update('FlatTaxModifier', 'rate', 0.25);
        Config::inst()->update('Order', 'modifiers', array('FlatTaxModifier'));

        $order = Order::create();
        $shirt = $this->objFromFixture("Product", "tshirt");
        $mp3player = $this->objFromFixture("Product", "mp3player");
        $order->Items()->add($shirt->createItem(3));
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $order->Items()->add($mp3player->createItem(1));
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $order->write();
        $order->calculate();

        $statusLogId = OrderStatusLog::create(array(
            'Title' => 'Test status log',
            'OrderID' => $order->ID
        ))->write();

        $paymentId = Payment::create(array(
            'OrderID' => $order->ID
        ))->init('Manual', 343.75, 'NZD')->write();


        $this->assertEquals(4, $order->Items()->Quantity());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $this->assertEquals(1, $order->Modifiers()->count());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $this->assertEquals(1, $order->OrderStatusLogs()->count());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $this->assertEquals(1, $order->Payments()->count());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }

        $itemIds = Product_OrderItem::get()->filter('OrderID', $order->ID)->column('ID');
        $modifierIds = OrderModifier::get()->filter('OrderID', $order->ID)->column('ID');

        $order->delete();

        // Items should no longer be linked to order
        $this->assertEquals(0, $order->Items()->count());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $this->assertEquals(0, $order->Modifiers()->count());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $this->assertEquals(0, $order->OrderStatusLogs()->count());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        $this->assertEquals(0, $order->Payments()->count());
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }

        // Ensure the order items have been deleted!
        $this->assertEquals(0, Product_OrderItem::get()->filter('ID', $itemIds)->count());

        $this->assertEquals(0, OrderModifier::get()->filter('ID', $modifierIds)->count());

        $this->assertEquals(0, OrderStatusLog::get()->filter('ID', $statusLogId)->count());


        // Keep the payment… it might be relevant for book keeping
        $this->assertEquals(1, Payment::get()->filter('ID', $paymentId)->count());

    }

    public function testStatusChange()
    {
        Config::inst()->update('Order', 'extensions', array('OrderTest_TestStatusChangeExtension'));

        $order = Order::create();
        $orderId = $order->write();

        $order->Status = 'Unpaid';
        $order->write();

        $this->assertEquals(array(

            array('Cart' => 'Unpaid')
        ), OrderTest_TestStatusChangeExtension::$stack);

        OrderTest_TestStatusChangeExtension::reset();

        $order = Order::get()->byID($orderId);
        $order->Status = 'Paid';
        $order->write();

        $this->assertEquals(array(

            array('Unpaid' => 'Paid')
        ), OrderTest_TestStatusChangeExtension::$stack);

        $this->assertTrue((boolean)$order->Paid, 'Order paid date should be set');

    }
}

class OrderTest_TestStatusChangeExtension extends DataExtension implements TestOnly
{
    public static $stack = array();

    public static function reset()
    {
        self::$stack = array();
    }

    public function onStatusChange($fromStatus, $toStatus)
    {
        self::$stack[] = array(
            $fromStatus => $toStatus
        );
    }
}


1			<?php
2
3			/**
4			* Order Unit Tests
5			*
6			* @link Order
7			* @package shop
8			* @subpackage tests
9			*/
10			class OrderTest extends SapphireTest
11			{
12			public static $fixture_file = 'silvershop/tests/fixtures/shop.yml';
13
14			public function setUp()
15			{
16			parent::setUp();
17			ShopTest::setConfiguration();
18			$this->mp3player = $this->objFromFixture('Product', 'mp3player');
			0 ignored issues – show Bug introduced 2016-02-23 23:20 UTC by Report Bug Copy Issue Report The property `mp3player` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
19			$this->mp3player->publish('Stage', 'Live');
20			$this->socks = $this->objFromFixture('Product', 'socks');
			0 ignored issues – show Bug introduced 2016-02-23 23:20 UTC by Report Bug Copy Issue Report The property `socks` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
21			$this->socks->publish('Stage', 'Live');
22			$this->beachball = $this->objFromFixture('Product', 'beachball');
			0 ignored issues – show Bug introduced 2016-02-23 23:20 UTC by Report Bug Copy Issue Report The property `beachball` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
23			$this->beachball->publish('Stage', 'Live');
24			}
25
26			public function tearDown()
27			{
28			parent::tearDown();
29			unset($this->mp3player);
30			unset($this->socks);
31			unset($this->beachball);
32			}
33
34			public function testCMSFields()
35			{
36			singleton('Order')->getCMSFields();
37			$this->markTestIncomplete('assertions!');

silvershop / silvershop-core

Push — 2.0 ( 6c824d...3cf5ce )

OrderTest_TestStatusChangeExtension::reset() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like