Skip to content

Commit 5a78f0b

Browse files
committed
MAGETWO-69110: Incorrect status for order placed within Authorize.net with Fraud Filters Triggered (Filter Actions = Process as normal and report filter(s) triggered)
- Fixed static test
1 parent d6cee84 commit 5a78f0b

File tree

2 files changed

+19
-16
lines changed

2 files changed

+19
-16
lines changed

dev/tests/integration/testsuite/Magento/Authorizenet/Model/DirectpostTest.php

Lines changed: 13 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -5,19 +5,17 @@
55
*/
66
namespace Magento\Authorizenet\Model;
77

8-
use Magento\Framework\Simplexml\Element;
9-
use Magento\Framework\Api\FilterBuilder;
108
use Magento\Framework\Api\SearchCriteriaBuilder;
119
use Magento\Framework\App\ObjectManager;
1210
use Magento\Framework\HTTP\ZendClient;
1311
use Magento\Framework\HTTP\ZendClientFactory;
12+
use Magento\Framework\Simplexml\Element;
1413
use Magento\Sales\Api\Data\OrderInterface;
1514
use Magento\Sales\Api\OrderRepositoryInterface;
1615
use Magento\Sales\Model\Order;
1716
use Magento\Sales\Model\Order\Payment;
1817
use Magento\TestFramework\Helper\Bootstrap;
1918
use PHPUnit_Framework_MockObject_MockObject as MockObject;
20-
use Zend_Http_Response;
2119

2220
/**
2321
* Class contains tests for Direct Post integration
@@ -74,7 +72,7 @@ public function testCapture()
7472
->method('create')
7573
->willReturn($httpClient);
7674

77-
$response = $this->getMockBuilder(Zend_Http_Response::class)
75+
$response = $this->getMockBuilder('Zend_Http_Response')
7876
->disableOriginalConstructor()
7977
->setMethods(['getBody'])
8078
->getMock();
@@ -101,7 +99,6 @@ public function testCapture()
10199
static::assertEquals('UK', $payment->getOrder()->getShippingAddress()->getCountryId());
102100
}
103101

104-
105102
/**
106103
* Verifies that order is placed in correct state according the action taken for a transaction that
107104
* triggered one or more of the Advanced Fraud Detection Suite filters.
@@ -136,8 +133,16 @@ public function testProcessWithFdsFilterActionReportOnly($filterAction, $orderId
136133
public function fdsFilterActionDataProvider()
137134
{
138135
return [
139-
['filter_action' => 'authAndHold', 'order_id' => '100000003', 'expected_order_state' => Order::STATE_PAYMENT_REVIEW],
140-
['filter_action' => 'report', 'order_id' => '100000004', 'expected_order_state' => Order::STATE_PROCESSING],
136+
[
137+
'filter_action' => 'authAndHold',
138+
'order_id' => '100000003',
139+
'expected_order_state' => Order::STATE_PAYMENT_REVIEW
140+
],
141+
[
142+
'filter_action' => 'report',
143+
'order_id' => '100000004',
144+
'expected_order_state' => Order::STATE_PROCESSING
145+
],
141146
];
142147
}
143148

@@ -204,17 +209,9 @@ private function getSuccessResponse($orderId)
204209
*/
205210
private function getPayment($orderId)
206211
{
207-
/** @var FilterBuilder $filterBuilder */
208-
$filterBuilder = $this->objectManager->get(FilterBuilder::class);
209-
$filters = [
210-
$filterBuilder->setField(OrderInterface::INCREMENT_ID)
211-
->setValue($orderId)
212-
->create()
213-
];
214-
215212
/** @var SearchCriteriaBuilder $searchCriteriaBuilder */
216213
$searchCriteriaBuilder = $this->objectManager->get(SearchCriteriaBuilder::class);
217-
$searchCriteria = $searchCriteriaBuilder->addFilters($filters)
214+
$searchCriteria = $searchCriteriaBuilder->addFilter(OrderInterface::INCREMENT_ID, $orderId)
218215
->create();
219216

220217
$orderRepository = $this->objectManager->get(OrderRepositoryInterface::class);

dev/tests/integration/testsuite/Magento/Authorizenet/_files/transaction_details.xml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,10 @@
11
<?xml version="1.0" encoding="utf-8"?>
2+
<!--
3+
/**
4+
* Copyright © Magento, Inc. All rights reserved.
5+
* See COPYING.txt for license details.
6+
*/
7+
-->
28
<getTransactionDetailsResponse>
39
<messages>
410
<resultCode>Ok</resultCode>

0 commit comments

Comments
 (0)