Skip to content

Commit fdf8c58

Browse files
committed
MAGETWO-97278: Incorrect use of cookies for customer
1 parent ba47946 commit fdf8c58

File tree

6 files changed

+18
-8
lines changed

6 files changed

+18
-8
lines changed
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
<?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+
-->
8+
<pages xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
9+
xsi:noNamespaceSchemaLocation="urn:magento:mftf:Page/etc/PageObject.xsd">
10+
<page name="StorefrontCustomerLogoutSuccessPage" url="customer/account/logoutSuccess/" area="storefront" module="Magento_Customer"/>
11+
</pages>

app/code/Magento/Persistent/Block/Header/Additional.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,7 @@ public function getCustomerId(): int
103103
*/
104104
public function getConfig(): string
105105
{
106-
return
107-
$this->jsonSerializer->serialize(
106+
return $this->jsonSerializer->serialize(
108107
[
109108
'expirationLifetime' => $this->persistentHelper->getLifeTime(),
110109
]

app/code/Magento/Persistent/Model/Plugin/PersistentCustomerContext.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6+
declare(strict_types=1);
7+
68
namespace Magento\Persistent\Model\Plugin;
79

810
/**

app/code/Magento/Persistent/Test/Mftf/Test/StorefrontCorrectWelcomeMessageAfterCustomerIsLoggedOutTest.xml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -48,13 +48,13 @@
4848
</actionGroup>
4949

5050
<!--Check customer name and last name in welcome message-->
51-
<seeCurrentUrlMatches regex="~/customer/account/~" stepKey="seeCustomerAccountPageUrl"/>
51+
<seeInCurrentUrl url="{{StorefrontCustomerDashboardPage.url}}" stepKey="seeCustomerAccountPageUrl"/>
5252
<see userInput="Welcome, $$createCustomer.firstname$$ $$createCustomer.lastname$$!"
5353
selector="{{StorefrontHeaderSection.welcomeMessage}}"
5454
stepKey="seeLoggedInCustomerWelcomeMessage"/>
5555
<!--Logout and check default welcome message-->
5656
<actionGroup ref="CustomerLogoutStorefrontByMenuItemsActionGroup" stepKey="storefrontCustomerLogout"/>
57-
<seeCurrentUrlMatches regex="~/customer/account/logoutSuccess/~" wait="5" stepKey="seeCustomerSignOutPageUrl"/>
57+
<seeInCurrentUrl url="{{StorefrontCustomerLogoutSuccessPage.url}}" wait="5" stepKey="seeCustomerSignOutPageUrl"/>
5858
<see userInput="Default welcome msg!"
5959
selector="{{StorefrontHeaderSection.welcomeMessage}}"
6060
stepKey="seeDefaultWelcomeMessage"/>
@@ -64,14 +64,14 @@
6464
<argument name="Customer" value="$$createCustomerForPersistent$$"/>
6565
</actionGroup>
6666
<!--Check customer name and last name in welcome message-->
67-
<seeCurrentUrlMatches regex="~/customer/account/~" stepKey="seeCustomerAccountPageUrl1"/>
67+
<seeInCurrentUrl url="{{StorefrontCustomerDashboardPage.url}}" stepKey="seeCustomerAccountPageUrl1"/>
6868
<see userInput="Welcome, $$createCustomerForPersistent.firstname$$ $$createCustomerForPersistent.lastname$$!"
6969
selector="{{StorefrontHeaderSection.welcomeMessage}}"
7070
stepKey="seeLoggedInCustomerWelcomeMessage1"/>
7171

7272
<!--Logout and check persistent customer welcome message-->
7373
<actionGroup ref="CustomerLogoutStorefrontByMenuItemsActionGroup" stepKey="storefrontCustomerLogout1"/>
74-
<seeCurrentUrlMatches regex="~/customer/account/logoutSuccess/~" wait="5" stepKey="seeCustomerSignOutPageUrl1"/>
74+
<seeInCurrentUrl url="{{StorefrontCustomerLogoutSuccessPage.url}}" wait="5" stepKey="seeCustomerSignOutPageUrl1"/>
7575
<see userInput="Welcome, $$createCustomerForPersistent.firstname$$ $$createCustomerForPersistent.lastname$$! Not you?"
7676
selector="{{StorefrontHeaderSection.welcomeMessage}}"
7777
stepKey="seePersistentWelcomeMessage"/>

app/code/Magento/Sales/Test/Mftf/Section/AdminCreditMemoTotalSection.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
<element name="emailCopy" type="checkbox" selector=".order-totals-actions #send_email"/>
2020
<element name="refundStoreCredit" type="checkbox" selector=".order-totals-actions .field-refund-store-credit input[type='checkbox']"/>
2121
<element name="submitRefundOffline" type="button" selector=".order-totals-actions button[data-ui-id='order-items-submit-button']" timeout="30"/>
22-
<element name="submitRefundOfflineEnabled" type="button" selector=".order-totals-actions button[data-ui-id='order-items-submit-button'][class='action-default scalable save submit-button primary']" timeout="30"/>
2322
<element name="creditMemoItem" type="text" selector="#sales_order_view_tabs_order_creditmemos"/>
2423
<element name="viewMemo" type="text" selector="div#sales_order_view_tabs_order_creditmemos_content a.action-menu-item"/>
2524
</section>

app/code/Magento/Sales/Test/Mftf/Section/AdminInvoiceItemsSection.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,5 @@
2828
<element name="discountAmountColumn" type="text" selector=".order-invoice-tables .col-discount .price"/>
2929
<element name="totalColumn" type="text" selector=".order-invoice-tables .col-total .price"/>
3030
<element name="updateQty" type="button" selector=".order-invoice-tables tfoot button[data-ui-id='order-items-update-button']"/>
31-
<element name="updateQtyEnabled" type="button" selector=".order-invoice-tables tfoot button[data-ui-id='order-items-update-button'][class='action-default scalable update-button']"/>
3231
</section>
3332
</sections>

0 commit comments

Comments
 (0)