Skip to content

Commit abcdcff

Browse files
committed
MC-39898: Stabilize Integration Tests
- PR Suggestions
1 parent 23b1073 commit abcdcff

File tree

56 files changed

+18
-163
lines changed

Some content is hidden

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

56 files changed

+18
-163
lines changed

dev/tests/integration/framework/tests/unit/testsuite/Magento/Test/Annotation/AppAreaTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,8 @@ protected function setUp(): void
3939
*/
4040
public function testGetTestAppArea($annotations, $expectedArea)
4141
{
42-
$this->_testCaseMock->expects($this->once())->method('getAnnotations')->will($this->returnValue($annotations));
43-
$this->_applicationMock->expects($this->any())->method('getArea')->will($this->returnValue(null));
42+
$this->_testCaseMock->expects($this->once())->method('getAnnotations')->willReturn($annotations);
43+
$this->_applicationMock->expects($this->any())->method('getArea')->willReturn(null);
4444
$this->_applicationMock->expects($this->once())->method('reinitialize');
4545
$this->_applicationMock->expects($this->once())->method('loadArea')->with($expectedArea);
4646
$this->_object->startTest($this->_testCaseMock);

dev/tests/integration/framework/tests/unit/testsuite/Magento/Test/Helper/BootstrapTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,6 @@ public function testGetInstanceAllowed(\Magento\TestFramework\Helper\Bootstrap $
9292
/**
9393
* @depends testSetInstanceFirstAllowed
9494
*
95-
*
9695
*/
9796
public function testSetInstanceChangeProhibited()
9897
{

dev/tests/integration/testsuite/Magento/Backend/App/Request/BackendValidatorTest.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -269,8 +269,6 @@ public function testValidateWithValidKey()
269269
}
270270

271271
/**
272-
*
273-
*
274272
* @magentoConfigFixture admin/security/use_form_key 1
275273
* @magentoAppArea adminhtml
276274
*/
@@ -294,8 +292,6 @@ public function testValidateWithInvalidKey()
294292
}
295293

296294
/**
297-
*
298-
*
299295
* @magentoConfigFixture admin/security/use_form_key 0
300296
* @magentoAppArea adminhtml
301297
*/

dev/tests/integration/testsuite/Magento/Catalog/Model/ImageUploaderTest.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -98,8 +98,6 @@ public function testMoveFileFromTmp(): void
9898
}
9999

100100
/**
101-
*
102-
*
103101
* @return void
104102
*/
105103
public function testSaveFileToTmpDirWithWrongExtension(): void
@@ -126,8 +124,6 @@ public function testSaveFileToTmpDirWithWrongExtension(): void
126124
}
127125

128126
/**
129-
*
130-
*
131127
* @return void
132128
*/
133129
public function testSaveFileToTmpDirWithWrongFile(): void

dev/tests/integration/testsuite/Magento/Catalog/Model/Product/Gallery/CreateHandlerTest.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,6 @@ public function testExecuteWithImageDuplicate(): void
103103
*
104104
* @param string $imageFileName
105105
*
106-
*
107-
*
108106
* @dataProvider illegalFilenameDataProvider
109107
* @return void
110108
*/

dev/tests/integration/testsuite/Magento/Catalog/Model/ResourceModel/Eav/AttributeTest.php

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,9 +66,6 @@ public function testCRUD()
6666
/**
6767
* @magentoDataFixture Magento/Catalog/_files/product_attribute.php
6868
*
69-
*
70-
*
71-
*
7269
* @return void
7370
*/
7471
public function testAttributeSaveWithChangedEntityType(): void

dev/tests/integration/testsuite/Magento/CatalogImportExport/Model/Import/UploaderTest.php

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,8 +137,6 @@ public function testMoveWithFileOutsideTemp(): void
137137
/**
138138
* @magentoAppIsolation enabled
139139
* @return void
140-
*
141-
*
142140
*/
143141
public function testMoveWithInvalidFile(): void
144142
{

dev/tests/integration/testsuite/Magento/CatalogInventory/Model/Quote/Item/QuantityValidatorTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,6 @@ public function testQuoteWithOptionsWithErrors()
149149
/**
150150
* Set mock of Stock State Result to Quote Item Options.
151151
*
152-
*
153152
* @param \Magento\Quote\Model\Quote\Item $quoteItem
154153
* @param \PHPUnit\Framework\MockObject\MockObject $resultMock
155154
*/

dev/tests/integration/testsuite/Magento/CatalogRule/Model/ResourceModel/Product/ConditionsToCollectionApplierTest.php

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,9 +83,6 @@ function (Product $product) {
8383
}
8484

8585
/**
86-
*
87-
*
88-
*
8986
* @magentoDbIsolation disabled
9087
*/
9188
public function testExceptionUndefinedRuleOperator()
@@ -114,9 +111,6 @@ public function testExceptionUndefinedRuleOperator()
114111
}
115112

116113
/**
117-
*
118-
*
119-
*
120114
* @magentoDbIsolation disabled
121115
*/
122116
public function testExceptionUndefinedRuleAggregator()

dev/tests/integration/testsuite/Magento/CatalogSearch/Model/Search/FilterMapper/StockStatusFilterWithGeneralFilterTest.php

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,7 @@ protected function setUp(): void
5151

5252
/**
5353
* @return void
54-
*
55-
*
56-
*
57-
*/
54+
**/
5855
public function testApplyWithWrongType()
5956
{
6057
$this->expectExceptionMessage("Invalid filter type: some_wrong_type");

0 commit comments

Comments
 (0)