Order::testUpdateValues()   A
last analyzed

Complexity

Conditions 1
Paths 1

Size

Total Lines 23

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
dl 0
loc 23
rs 9.552
c 0
b 0
f 0
nc 1
nop 2
1
<?php
2
/**
3
 * integer_net Magento Module
4
 *
5
 * @category   IntegerNet
6
 * @package    IntegerNet_Anonymizer
7
 * @copyright  Copyright (c) 2015 integer_net GmbH (http://www.integer-net.de/)
8
 * @author     Fabian Schmengler <[email protected]>
9
 */
10
11
/**
12
 * @group IntegerNet_Anonymizer
13
 */
14
class IntegerNet_Anonymizer_Test_Model_Bridge_Entity_Order
15
    extends IntegerNet_Anonymizer_Test_Model_Bridge_Entity_Abstract
16
{
17
    /**
18
     * @param $orderId
19
     * @test
20
     * @dataProvider dataProvider
21
     * @dataProviderFile testOrderBridge.yaml
22
     * @loadExpectation bridge.yaml
23
     * @loadFixture customers.yaml
24
     */
25
    public function testGetValues($orderId)
26
    {
27
        /** @var IntegerNet_Anonymizer_Test_Model_Bridge_Entity_Order $bridge */
28
        $bridge = Mage::getModel('integernet_anonymizer/bridge_entity_order');
29
        /** @var Mage_Sales_Model_Order $order */
30
        $order = $this->_loadEntityByCollection('entity_id', $orderId, $bridge);
31
        $expected = $this->expected('order_%d', $orderId);
32
33
        $this->_testGetValues($bridge, $order, $expected);
0 ignored issues
show
Documentation introduced by
$bridge is of type object<IntegerNet_Anonym...el_Bridge_Entity_Order>, but the function expects a object<IntegerNet_Anonym...Bridge_Entity_Abstract>.

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...
34
    }
35
36
    /**
37
     * @param $orderId
38
     * @param $customerId
39
     * @test
40
     * @dataProvider dataProvider
41
     * @dataProviderFile testOrderBridge.yaml
42
     * @loadFixture customers.yaml
43
     */
44
    public function testUpdateValues($orderId, $customerId)
45
    {
46
        static $changedEmail = '[email protected]',
47
               $changedMiddlename = 'trouble';
48
49
        /** @var IntegerNet_Anonymizer_Model_Bridge_Entity_Order $bridge */
50
        $bridge = Mage::getModel('integernet_anonymizer/bridge_entity_order');
51
52
        $bridge->setRawData(array(
53
            'entity_id' => $orderId,
54
            'increment_id' => '1000000001',
55
            'customer_id' => $customerId,
56
            'customer_email'     => $changedEmail,
57
            'customer_middlename' => $changedMiddlename
58
            ));
59
60
        $this->_updateValues($bridge);
61
62
        $order = Mage::getModel('sales/order')->load($orderId);
63
        $this->assertEquals($changedMiddlename, $order->getCustomerMiddlename());
64
        $this->assertEquals($changedEmail, $order->getCustomerEmail());
65
        $this->assertNotEmpty($order->getIncrementId(), 'Increment ID should not be empty');
66
    }
67
68
}