Skip to content

Commit e103aaa

Browse files
committed
Merge branch '2.4-develop' of github.com:magento-gl/magento2ce into AC-16072-SalesRule-Order
2 parents 3478a74 + b5e99d2 commit e103aaa

File tree

69 files changed

+2439
-3765
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

69 files changed

+2439
-3765
lines changed

app/code/Magento/Catalog/Test/Mftf/Test/StorefrontValidatePriceChangesForConfigurableProductWithMultipleAttributesInWidgetTest.xml

Lines changed: 218 additions & 0 deletions
Large diffs are not rendered by default.

app/code/Magento/Catalog/Test/Mftf/test-dependency-allowlist

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,4 +43,6 @@ AdminSetQuantityToEachSkusConfigurableProductActionGroup
4343
SelectStorefrontSideBarAttributeOption
4444
AdminAddProductVideoWithPreviewActionGroup
4545
AdminSetBaseCurrencyActionGroup
46-
AdminSetDefaultCurrencyActionGroup
46+
AdminSetDefaultCurrencyActionGroup
47+
ChangeConfigurableProductChildProductPriceActionGroup
48+
SaveConfigurableProductAddToCurrentAttributeSetActionGroup

app/code/Magento/Checkout/Test/Mftf/Test/CheckCheckoutSuccessPageTest/CheckCheckoutSuccessPageAsRegisterCustomerTest.xml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -135,10 +135,11 @@
135135
<waitForElementVisible selector="{{CheckoutSuccessMainSection.printLink}}" stepKey="waitVisiblePrint"/>
136136
<seeElement selector="{{CheckoutSuccessMainSection.printLink}}" stepKey="seeVisiblePrint2"/>
137137

138-
<!-- See print page -->
139-
<click selector="{{CheckoutSuccessMainSection.printLink}}" stepKey="clickPrintLink"/>
140-
<waitForPageLoad stepKey="waitForPrintOrderPageLoad"/>
141-
<seeElement selector="{{StorefrontCustomerOrderViewSection.orderTitle}}" stepKey="seeOrderTitleOnPrint"/>
142-
<switchToWindow stepKey="switchToWindow2"/>
138+
<!-- Validate print page URL -->
139+
<grabAttributeFrom selector="{{CheckoutSuccessMainSection.printLink}}" userInput="href" stepKey="orderIdFromPrintUrl"/>
140+
<assertRegExp message="Assert print order url has correct format" stepKey="assertPrintUrlHasCorrectFormat">
141+
<expectedResult type="string">/^.+\/sales\/order\/print\/order_id\/\d+\/$/</expectedResult>
142+
<actualResult type="variable">orderIdFromPrintUrl</actualResult>
143+
</assertRegExp>
143144
</test>
144145
</tests>

app/code/Magento/Cms/Test/Mftf/ActionGroup/FillOutCMSPageContentActionGroup.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515

1616
<fillField selector="{{CmsNewPagePageBasicFieldsSection.pageTitle}}" userInput="{{_duplicatedCMSPage.title}}" stepKey="fillFieldTitle"/>
1717
<click selector="{{CmsNewPagePageContentSection.header}}" stepKey="clickExpandContentTabForPage"/>
18+
<waitForElementVisible selector="{{CmsNewPagePageContentSection.contentHeading}}" stepKey="waitForContentHeading"/>
1819
<fillField selector="{{CmsNewPagePageContentSection.contentHeading}}" userInput="{{_duplicatedCMSPage.content_heading}}" stepKey="fillFieldContentHeading"/>
1920
<scrollTo selector="{{CmsNewPagePageContentSection.content}}" stepKey="scrollToPageContent"/>
2021
<fillField selector="{{CmsNewPagePageContentSection.content}}" userInput="{{_duplicatedCMSPage.content}}" stepKey="fillFieldContent"/>

app/code/Magento/ConfigurableProduct/Test/Mftf/Test/AdminCreateConfigurableProductWithChineseLocaleTest.xml

Lines changed: 161 additions & 0 deletions
Large diffs are not rendered by default.

app/code/Magento/ImportExport/Controller/Adminhtml/Import/Start.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,7 @@ public function execute()
124124

125125
$this->addErrorMessages($resultBlock, $errorAggregator);
126126
$resultBlock->addSuccess(__('Import successfully done'));
127+
$this->_eventManager->dispatch('log_admin_import');
127128
}
128129

129130
return $resultLayout;

app/code/Magento/ImportExport/Controller/Adminhtml/Import/Validate.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ public function execute()
5151
$import = $this->getImport()->setData($data);
5252
try {
5353
$source = $import->uploadFileAndGetSource();
54+
$this->_eventManager->dispatch('log_admin_import');
5455
$this->processValidationResult($import->validateSource($source), $resultBlock);
5556
$ids = $import->getValidatedIds();
5657
if (count($ids) > 0) {

app/code/Magento/ImportExport/Test/Unit/Controller/Adminhtml/Import/ValidateTest.php

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
use Magento\ImportExport\Model\Import;
2828
use Magento\ImportExport\Model\Import\AbstractSource;
2929
use Magento\ImportExport\Model\Import\ErrorProcessing\ProcessingErrorAggregatorInterface;
30+
use Magento\Framework\Event\ManagerInterface as EventManagerInterface;
3031

3132
/**
3233
* @SuppressWarnings(PHPMD.CouplingBetweenObjects)
@@ -83,6 +84,11 @@ class ValidateTest extends TestCase
8384
*/
8485
private $abstractSourceMock;
8586

87+
/**
88+
* @var EventManagerInterface|MockObject
89+
*/
90+
private $eventManagerMock;
91+
8692
protected function setUp(): void
8793
{
8894
$objectManagerHelper = new ObjectManagerHelper($this);
@@ -147,6 +153,13 @@ protected function setUp(): void
147153
->disableOriginalConstructor()
148154
->getMockForAbstractClass();
149155

156+
$this->eventManagerMock = $this->getMockBuilder(EventManagerInterface::class)
157+
->getMockForAbstractClass();
158+
159+
$this->contextMock->expects($this->any())
160+
->method('getEventManager')
161+
->willReturn($this->eventManagerMock);
162+
150163
$this->validate = new Validate(
151164
$this->contextMock,
152165
$this->reportProcessorMock,
@@ -337,6 +350,10 @@ public function testFileVerifiedWithImport()
337350
->method('getAllErrors')
338351
->willReturn($errorAggregatorMock);
339352

353+
$this->eventManagerMock->expects($this->once())
354+
->method('dispatch')
355+
->with('log_admin_import');
356+
340357
$this->resultFactoryMock->expects($this->any())
341358
->method('create')
342359
->with(ResultFactory::TYPE_LAYOUT)

app/code/Magento/Paypal/Test/Mftf/Test/StorefrontPaypalExpressCheckoutWithDiscountCouponTest.xml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,13 @@
3434
</createData>
3535
</before>
3636
<after>
37-
<deleteData createDataKey="createCustomer" stepKey="deleteCustomer"/>
38-
<deleteData createDataKey="createProduct" stepKey="deleteProduct"/>
37+
<!-- Logout customer first -->
38+
<actionGroup ref="StorefrontCustomerLogoutActionGroup" stepKey="logoutCustomer"/>
39+
<!-- Delete via Admin UI instead of API -->
40+
<actionGroup ref="AdminDeleteCustomerActionGroup" stepKey="deleteCustomer">
41+
<argument name="customerEmail" value="$$createCustomer.email$$"/>
42+
</actionGroup>
43+
<actionGroup ref="DeleteAllProductsUsingProductGridActionGroup" stepKey="deleteProduct" />
3944
<!-- Delete Cart Price Rule -->
4045
<deleteData createDataKey="createCartPriceRule" stepKey="deleteCartPriceRule"/>
4146
<actionGroup ref="AdminLogoutActionGroup" stepKey="logout"/>

app/code/Magento/Paypal/Test/Mftf/Test/StorefrontRewardPointsAppliedAfterApplyingCouponTest.xml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,13 @@
5353
<after>
5454
<magentoCLI command="config:set payment/paypal_express/active 0" stepKey="disablePayPalExpress"/>
5555
<magentoCLI command="config:set {{StorefrontPaypalExpressAuthorizationPaymentActionOptionConfigData.path}} {{StorefrontPaypalExpressAuthorizationPaymentActionOptionConfigData.value}}" stepKey="setPaymentAction"/>
56+
<!-- Logout customer first -->
5657
<actionGroup ref="StorefrontCustomerLogoutActionGroup" stepKey="logoutCustomer"/>
57-
<actionGroup ref="AdminDeleteAllCustomerActionGroup" stepKey="deleteCustomer"/>
58-
<actionGroup ref="DeleteAllProductsUsingProductGridActionGroup" stepKey="deleteProduct"/>
58+
<!-- Delete via Admin UI instead of API -->
59+
<actionGroup ref="AdminDeleteCustomerActionGroup" stepKey="deleteCustomer">
60+
<argument name="customerEmail" value="$$createCustomer.email$$"/>
61+
</actionGroup>
62+
<actionGroup ref="DeleteAllProductsUsingProductGridActionGroup" stepKey="deleteProduct" />
5963
<!--Delete cart price rule-->
6064
<actionGroup ref="DeleteCartPriceRuleByName" stepKey="deleteCartPriceRule">
6165
<argument name="ruleName" value="{{_defaultCoupon.code}}"/>

0 commit comments

Comments
 (0)