Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MFTF] Refactoring AdminCreateCreditMemoForOrderWithCashOnDeliveryTest #31395

4 changes: 4 additions & 0 deletions app/code/Magento/Payment/Test/Mftf/Data/PaymentMethodData.xml
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,8 @@
<entity name="CashOnDeliveryPaymentMethodDefault" type="cashondelivery_payment_method">
<requiredEntity type="active">CashOnDeliveryEnableConfigData</requiredEntity>
</entity>

<entity name="CashOnDeliveryPaymentMethod" type="payment_method">
<data key="method">cashondelivery</data>
</entity>
</entities>
6 changes: 6 additions & 0 deletions app/code/Magento/Quote/Test/Mftf/Data/CustomerCartData.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,4 +24,10 @@
<requiredEntity type="payment_method">PaymentMethodCheckMoneyOrder</requiredEntity>
<requiredEntity type="billing_address">BillingAddressTX</requiredEntity>
</entity>

<entity name="CashOnDeliveryOrderPaymentMethod" type="CustomerPaymentInformation">
<var key="cart_id" entityKey="return" entityType="CustomerCart"/>
<requiredEntity type="payment_method">CashOnDeliveryPaymentMethod</requiredEntity>
<requiredEntity type="billing_address">BillingAddressTX</requiredEntity>
</entity>
</entities>
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
/**
* Copyright © Magento, Inc. All rights reserved.
* See COPYING.txt for license details.
*/
-->

<actionGroups xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/actionGroupSchema.xsd">
<actionGroup name="AdminClickRefundOfflineOnMemoDetailPageActionGroup">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, use the original CreditMemo entity name. Since just Memo might be confusing (it has many other meanings).

Suggested change
<actionGroup name="AdminClickRefundOfflineOnMemoDetailPageActionGroup">
<actionGroup name="AdminClickRefundOfflineOnCreditMemoDetailPageActionGroup">

Thanks.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Has been refactored.


<click selector="{{AdminCreditMemoTotalSection.submitRefundOffline}}" stepKey="clickRefundOffline"/>
<waitForElementVisible selector="{{AdminMessagesSection.success}}" stepKey="waitForSuccesMessage"/>
<see selector="{{AdminMessagesSection.success}}" userInput="You created the credit memo." stepKey="seeSuccessMessage"/>
</actionGroup>
</actionGroups>
Original file line number Diff line number Diff line change
@@ -0,0 +1,99 @@
<?xml version="1.0" encoding="UTF-8"?>
<!--
/**
* Copyright © Magento, Inc. All rights reserved.
* See COPYING.txt for license details.
*/
-->

<tests xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/testSchema.xsd">
<test name="AdminCreateCreditMemoForOrderWithCashOnDeliveryTest">
<annotations>
<stories value="Credit memo entity"/>
<title value="Create Credit Memo with cash on delivery payment method"/>
<description value="Create Credit Memo with cash on delivery payment and assert 0 shipping refund"/>
<severity value="CRITICAL"/>
<testCaseId value="MC-15863"/>
<group value="sales"/>
<group value="mtf_migrated"/>
</annotations>
<before>
<actionGroup ref="AdminLoginActionGroup" stepKey="LoginAsAdmin"/>

<createData entity="Simple_US_Customer" stepKey="createCustomer"/>
<createData entity="_defaultCategory" stepKey="createCategory"/>
<createData entity="defaultSimpleProduct" stepKey="createProduct">
<requiredEntity createDataKey="createCategory"/>
</createData>

<magentoCLI command="config:set {{enabledCashOnDeliveryPayment.label}} {{enabledCashOnDeliveryPayment.value}}" stepKey="enableBankTransfer"/>

<createData entity="CustomerCart" stepKey="createCustomerCart">
<requiredEntity createDataKey="createCustomer"/>
</createData>

<createData entity="CustomerCartItem" stepKey="addCartItem">
<requiredEntity createDataKey="createCustomerCart"/>
<requiredEntity createDataKey="createProduct"/>
</createData>

<createData entity="CustomerAddressInformation" stepKey="addCustomerOrderAddress">
<requiredEntity createDataKey="createCustomerCart"/>
</createData>

<updateData createDataKey="createCustomerCart" entity="CashOnDeliveryOrderPaymentMethod" stepKey="sendCustomerPaymentInformation">
<requiredEntity createDataKey="createCustomerCart"/>
</updateData>

<createData entity="Invoice" stepKey="invoiceOrderOne">
<requiredEntity createDataKey="createCustomerCart"/>
</createData>

</before>
<after>
<magentoCLI command="config:set {{disabledCashOnDeliveryPayment.label}} {{disabledCashOnDeliveryPayment.value}}" stepKey="disableBankTransfer"/>
<deleteData createDataKey="createCustomer" stepKey="deleteCustomer"/>
<deleteData createDataKey="createCategory" stepKey="deleteCategory"/>
<deleteData createDataKey="createProduct" stepKey="deleteProduct"/>
<actionGroup ref="AdminLogoutActionGroup" stepKey="logout"/>
</after>

<actionGroup ref="AdminOrdersPageOpenActionGroup" stepKey="onOrderPage"/>
<actionGroup ref="AdminOrdersGridClearFiltersActionGroup" stepKey="clearFilters"/>
<grabTextFrom selector="{{AdminOrdersGridSection.orderIdByIncrementId($createCustomerCart.return$)}}" stepKey="grabOrderId"/>
<actionGroup ref="OpenOrderByIdActionGroup" stepKey="filterOrdersGridById">
<argument name="orderId" value="{$grabOrderId}"/>
</actionGroup>

<actionGroup ref="AdminOpenAndFillCreditMemoRefundActionGroup" stepKey="fillCreditMemoRefund">
<argument name="itemQtyToRefund" value="1"/>
<argument name="shippingRefund" value="0"/>
<argument name="adjustmentRefund" value="5"/>
<argument name="adjustmentFee" value="10"/>
</actionGroup>

<actionGroup ref="AdminClickRefundOfflineOnMemoDetailPageActionGroup" stepKey="clickRefundOffline"/>

<actionGroup ref="AdminOpenCreditMemoFromOrderPageActionGroup" stepKey="openCreditMemo"/>

<actionGroup ref="AssertAdminCreditMemoViewPageTotalsActionGroup" stepKey="assertCreditMemoViewPageTotals">
<argument name="subtotal" value="$560.00"/>
<argument name="adjustmentRefund" value="$5.00"/>
<argument name="adjustmentFee" value="$10.00"/>
<argument name="grandTotal" value="$555.00"/>
</actionGroup>

<actionGroup ref="LoginToStorefrontActionGroup" stepKey="loginAsCustomer">
<argument name="Customer" value="$$createCustomer$$"/>
</actionGroup>

<actionGroup ref="StorefrontGoToCustomerOrderDetailsPageActionGroup" stepKey="openOrderDetailPage">
<argument name="orderId" value="$createCustomerCart.return$"/>
<argument name="orderNumber" value="{$grabOrderId}"/>
</actionGroup>

<actionGroup ref="StorefrontClickRefundTabCustomerOrderViewActionGroup" stepKey="clickRefund"/>
<see selector="{{StorefrontCustomerOrderSection.grandTotalRefund}}" userInput="555.00" stepKey="seeGrandTotal"/>
</test>
</tests>
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,18 @@

<tests xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/testSchema.xsd">
<test name="AdminCreateCreditMemoWithCashOnDeliveryTest">
<test name="AdminCreateCreditMemoWithCashOnDeliveryTest" deprecated="Use AdminCreateCreditMemoForOrderWithCashOnDeliveryTest instead">
<annotations>
<stories value="Credit memo entity"/>
<title value="Create Credit Memo with cash on delivery payment method"/>
<title value="DEPRECATED. Create Credit Memo with cash on delivery payment method"/>
<description value="Create Credit Memo with cash on delivery payment and assert 0 shipping refund"/>
<severity value="CRITICAL"/>
<testCaseId value="MC-15863"/>
<group value="sales"/>
<group value="mtf_migrated"/>
<skip>
<issueId value="DEPRECATED">Use AdminCreateCreditMemoForOrderWithCashOnDeliveryTest instead</issueId>
</skip>
</annotations>
<before>
<actionGroup ref="AdminLoginActionGroup" stepKey="LoginAsAdmin"/>
Expand Down