Skip to content

Commit 22fec03

Browse files
author
Stanislav Idolov
committed
MAGETWO-43446: Clean Up \Magento\Quote\Api\Data\PaymentInterface
1 parent 3b770b2 commit 22fec03

File tree

2 files changed

+11
-47
lines changed

2 files changed

+11
-47
lines changed

dev/tests/api-functional/testsuite/Magento/Quote/Api/GuestPaymentMethodManagementTest.php

Lines changed: 5 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -81,11 +81,7 @@ public function testReSetPayment()
8181
"cartId" => $cartId,
8282
"method" => [
8383
'method' => 'checkmo',
84-
'po_number' => null,
85-
'cc_owner' => 'John',
86-
'cc_type' => null,
87-
'cc_exp_year' => null,
88-
'cc_exp_month' => null,
84+
'po_number' => null
8985
],
9086
];
9187

@@ -118,11 +114,7 @@ public function testSetPaymentWithVirtualProduct()
118114
"cartId" => $cartId,
119115
"method" => [
120116
'method' => 'checkmo',
121-
'po_number' => '200',
122-
'cc_owner' => 'tester',
123-
'cc_type' => 'test',
124-
'cc_exp_year' => '2014',
125-
'cc_exp_month' => '1',
117+
'po_number' => '200'
126118
],
127119
];
128120
$this->assertNotNull($this->_webApiCall($serviceInfo, $requestData));
@@ -154,11 +146,7 @@ public function testSetPaymentWithSimpleProduct()
154146
"cartId" => $cartId,
155147
"method" => [
156148
'method' => 'checkmo',
157-
'po_number' => '200',
158-
'cc_owner' => 'tester',
159-
'cc_type' => 'test',
160-
'cc_exp_year' => '2014',
161-
'cc_exp_month' => '1',
149+
'po_number' => '200'
162150
],
163151
];
164152

@@ -193,11 +181,7 @@ public function testSetPaymentWithSimpleProductWithoutAddress()
193181
"cartId" => $cartId,
194182
"method" => [
195183
'method' => 'checkmo',
196-
'po_number' => '200',
197-
'cc_owner' => 'tester',
198-
'cc_type' => 'test',
199-
'cc_exp_year' => '2014',
200-
'cc_exp_month' => '1',
184+
'po_number' => '200'
201185
],
202186
];
203187
$this->assertNotNull($this->_webApiCall($serviceInfo, $requestData));
@@ -275,7 +259,7 @@ public function testGet()
275259
*/
276260
protected function getPaymentMethodFieldsForAssert()
277261
{
278-
return ['method', 'po_number', 'cc_owner', 'cc_type', 'cc_exp_year', 'cc_exp_month', 'additional_data'];
262+
return ['method', 'po_number', 'additional_data'];
279263
}
280264

281265
/**

dev/tests/api-functional/testsuite/Magento/Quote/Api/PaymentMethodManagementTest.php

Lines changed: 6 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -47,11 +47,7 @@ public function testReSetPayment()
4747
"cartId" => $cartId,
4848
"method" => [
4949
'method' => 'checkmo',
50-
'po_number' => null,
51-
'cc_owner' => 'John',
52-
'cc_type' => null,
53-
'cc_exp_year' => null,
54-
'cc_exp_month' => null,
50+
'po_number' => null
5551
],
5652
];
5753

@@ -84,11 +80,7 @@ public function testSetPaymentWithVirtualProduct()
8480
"cartId" => $cartId,
8581
"method" => [
8682
'method' => 'checkmo',
87-
'po_number' => '200',
88-
'cc_owner' => 'tester',
89-
'cc_type' => 'test',
90-
'cc_exp_year' => '2014',
91-
'cc_exp_month' => '1',
83+
'po_number' => '200'
9284
],
9385
];
9486
$this->assertNotNull($this->_webApiCall($serviceInfo, $requestData));
@@ -120,11 +112,7 @@ public function testSetPaymentWithSimpleProduct()
120112
"cartId" => $cartId,
121113
"method" => [
122114
'method' => 'checkmo',
123-
'po_number' => '200',
124-
'cc_owner' => 'tester',
125-
'cc_type' => 'test',
126-
'cc_exp_year' => '2014',
127-
'cc_exp_month' => '1',
115+
'po_number' => '200'
128116
],
129117
];
130118

@@ -159,11 +147,7 @@ public function testSetPaymentWithSimpleProductWithoutAddress()
159147
"cartId" => $cartId,
160148
"method" => [
161149
'method' => 'checkmo',
162-
'po_number' => '200',
163-
'cc_owner' => 'tester',
164-
'cc_type' => 'test',
165-
'cc_exp_year' => '2014',
166-
'cc_exp_month' => '1',
150+
'po_number' => '200'
167151
],
168152
];
169153
$this->assertNotNull($this->_webApiCall($serviceInfo, $requestData));
@@ -317,11 +301,7 @@ public function testSetPaymentWithSimpleProductMine()
317301
$requestData = [
318302
"method" => [
319303
'method' => 'checkmo',
320-
'po_number' => '200',
321-
'cc_owner' => 'tester',
322-
'cc_type' => 'test',
323-
'cc_exp_year' => '2014',
324-
'cc_exp_month' => '1',
304+
'po_number' => '200'
325305
],
326306
];
327307

@@ -333,7 +313,7 @@ public function testSetPaymentWithSimpleProductMine()
333313
*/
334314
protected function getPaymentMethodFieldsForAssert()
335315
{
336-
return ['method', 'po_number', 'cc_owner', 'cc_type', 'cc_exp_year', 'cc_exp_month', 'additional_data'];
316+
return ['method', 'po_number', 'additional_data'];
337317
}
338318

339319
/**

0 commit comments

Comments
 (0)