Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
ce38bc8
MAGETWO-61209: Fixed issue with mage/calendar when setting `numberOfM…
Jun 20, 2018
ebccd5c
ENGCOM-2108: MAGETWO-61209: Backport - Fixed issue #7379 with mage/ca…
magento-engcom-team Jun 22, 2018
d8cab67
Added integration test for checking admin login attempts cleanup
rogyar Jun 22, 2018
66de2da
[Backport] array_push(...) calls behaving as '$array[] = ...', $arra…
lfluvisotto Jun 23, 2018
4650916
ENGCOM-2114: [Backport 2.1] Captcha: Added integration test for check…
magento-engcom-team Jun 23, 2018
1bb1b7a
[Backport] PHPDoc
lfluvisotto Jun 23, 2018
346eed2
Corrected invoice grid column subtotal & shipping handling value
ankurseepossible Apr 21, 2018
7122a88
ENGCOM-2113: [Backport] array_push(...) calls behaving as '$array[] …
magento-engcom-team Jun 24, 2018
30db7cd
ENGCOM-2113: [Backport] array_push(...) calls behaving as '$array[] …
Jun 25, 2018
4bcdb46
Fixed InterfaceGeneratorTest
VladimirZaets Jun 25, 2018
390da42
ENGCOM-2120: [Backport] PHPDoc #16366
magento-engcom-team Jun 25, 2018
59f298f
ENGCOM-2120: [Backport] PHPDoc #16366
Jun 25, 2018
fd585a8
ENGCOM-2114: [Backport 2.1] Captcha: Added integration test for check…
Jun 25, 2018
87c8f5a
Fixed static test failures
Jun 26, 2018
5f4b65c
ENGCOM-2136: [Backport] Invoice grid shows wrong shipping & handling …
magento-engcom-team Jun 26, 2018
12f7e9c
Fixed static test failures
Jun 26, 2018
a5d68a5
Fixed static test failures
Jun 26, 2018
a83ec42
Added integration test for checking customer login attempts captcha c…
rogyar Jun 26, 2018
74e60e3
ENGCOM-2142: [Backport 2.1] Captcha: Added integration tests for chec…
magento-engcom-team Jun 26, 2018
65cceab
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
Jun 26, 2018
a3b948c
ENGCOM-2142: [Backport 2.1] Captcha: Added integration tests for chec…
Jun 28, 2018
af2c7a6
ENGCOM-2136: [Backport] Invoice grid shows wrong shipping & handling …
Jun 28, 2018
b9fc7e2
ENGCOM-2108: MAGETWO-61209: Backport - Fixed issue #7379 with mage/ca…
Jun 28, 2018
1de3379
:arrows_clockwise: [EngCom] Public Pull Requests - 2.1-develop
Jun 29, 2018
d2a3065
Fix $useCache for container child blocks
Mar 9, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,9 @@ public function testGetIdentity($expectedSum, $cacheTypes)
$this->assertEquals($expectedSum, $this->_messageModel->getIdentity());
}

/**
* @return array
*/
public function getIdentityDataProvider()
{
$cacheTypeMock1 = $this->getMock('stdClass', ['getCacheType']);
Expand Down Expand Up @@ -95,6 +98,9 @@ public function testIsDisplayed($expected, $allowed, $cacheTypes)
$this->assertEquals($expected, $this->_messageModel->isDisplayed());
}

/**
* @return array
*/
public function isDisplayedDataProvider()
{
$cacheTypesMock = $this->getMock('stdClass', ['getCacheType']);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,9 @@ public function testIsDisplayed($expectedFirstRun, $data)
$this->assertEquals($expectedFirstRun, $model->isDisplayed());
}

/**
* @return array
*/
public function isDisplayedDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,9 @@ public function testIsDisplayed($expectedResult, $cached, $response)
$this->assertEquals($expectedResult, $this->_messageModel->isDisplayed());
}

/**
* @return array
*/
public function isDisplayedDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -198,6 +198,9 @@ public function testIsValidAddMessagesCall($value, $hasEmptyColumns, $customerGr
$this->tierPrice->isValid($value);
}

/**
* @return array
*/
public function isValidResultFalseDataProvider()
{
return [
Expand Down Expand Up @@ -303,6 +306,9 @@ public function isValidResultFalseDataProvider()
];
}

/**
* @return array
*/
public function isValidAddMessagesCallDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,9 @@ public function testGetAllWebsitesValue()
$this->assertEquals($expectedResult, $result);
}

/**
* @return array
*/
public function isValidReturnDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,9 @@ public function testInit()
$this->validator->init(null);
}

/**
* @return array
*/
public function isValidDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -751,6 +751,9 @@ public function testSaveProductPrices($priceData, $oldSkus, $priceIn, $callNum)
$this->invokeMethod($this->advancedPricing, 'saveProductPrices', [$priceData, 'table']);
}

/**
* @return array
*/
public function saveProductPricesDataProvider()
{
return [
Expand Down Expand Up @@ -822,6 +825,9 @@ public function testDeleteProductTierPrices(
);
}

/**
* @return array
*/
public function deleteProductTierPricesDataProvider()
{
return [
Expand Down Expand Up @@ -904,6 +910,9 @@ public function testProcessCountExistingPrices(
$this->invokeMethod($this->advancedPricing, 'processCountExistingPrices', [$prices, 'table']);
}

/**
* @return array
*/
public function processCountExistingPricesDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,9 @@ public function testGetAllowedResourcesByUser()
);
}

/**
* @return AclRetriever
*/
protected function createAclRetriever()
{
$this->roleMock = $this->getMock(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,9 @@ public function testGenerateHash($merchantMd5, $merchantApiLogin, $amount, $amou
);
}

/**
* @return array
*/
public function generateHashDataProvider()
{
return [
Expand All @@ -57,6 +60,13 @@ public function generateHashDataProvider()
];
}

/**
* @param $merchantMd5
* @param $merchantApiLogin
* @param $amount
* @param $transactionId
* @return string
*/
protected function generateHash($merchantMd5, $merchantApiLogin, $amount, $transactionId)
{
return strtoupper(md5($merchantMd5 . $merchantApiLogin . $transactionId . $amount));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@ class BackendModel extends Value
/** Minimum admin session lifetime */
const MIN_LIFETIME = 60;

/**
* @return Value
* @throws LocalizedException
*/
public function beforeSave()
{
$value = (int) $this->getValue();
Expand Down
8 changes: 8 additions & 0 deletions app/code/Magento/Backend/Model/View/Layout/Reader/Block.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,14 @@
*/
class Block extends Layout\Reader\Block
{
/**
* Block constructor.
* @param Layout\ScheduledStructure\Helper $helper
* @param Layout\Argument\Parser $argumentParser
* @param Layout\ReaderPool $readerPool
* @param InterpreterInterface $argumentInterpreter
* @param null $scopeType
*/
public function __construct(
Layout\ScheduledStructure\Helper $helper,
Layout\Argument\Parser $argumentParser,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -149,6 +149,9 @@ public function testProcessNotLoggedInUser($isIFrameParam, $isAjaxParam, $isForw
$this->assertEquals($expectedResult, $this->plugin->aroundDispatch($subject, $proceed, $request));
}

/**
* @return array
*/
public function processNotLoggedInUserDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,9 @@ public function testAroundDispatchWhenMassactionPrepareKeyRequestExists($postDat
);
}

/**
* @return array
*/
public function aroundDispatchDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,9 @@

class ActionStub extends \Magento\Backend\App\Action
{
/**
* @return \Magento\Framework\App\ResponseInterface|\Magento\Framework\Controller\ResultInterface|void
*/
public function execute()
{
// Empty method stub for test
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,9 @@ public function testIsHostBackend($url, $host, $useCustomAdminUrl, $customAdminU
$this->assertEquals($this->model->isHostBackend(), $expectedValue);
}

/**
* @return array
*/
public function hostsDataProvider()
{
return [
Expand Down
3 changes: 3 additions & 0 deletions app/code/Magento/Backend/Test/Unit/App/ConfigTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,9 @@ public function testIsSetFlag($configPath, $configValue, $expectedResult)
$this->assertEquals($expectedResult, $this->model->isSetFlag($configPath));
}

/**
* @return array
*/
public function isSetFlagDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,9 @@ public function testIsInProductionMode($mode, $expected)
$this->assertEquals($expected, $this->additonalBlock->isInProductionMode());
}

/**
* @return array
*/
public function isInProductionModeDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,9 @@ public function testGetAttributesHtml($data, $expect)
$this->assertRegExp($expect, $attributes);
}

/**
* @return array
*/
public function getAttributesHtmlDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,9 @@ public function testRender(array $rowData, $expectedResult)
$this->assertEquals($expectedResult, $this->_object->render(new \Magento\Framework\DataObject($rowData)));
}

/**
* @return array
*/
public function renderDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,9 @@ public function testRender(array $rowData, $expectedResult)
$this->assertEquals($expectedResult, $this->_object->render(new \Magento\Framework\DataObject($rowData)));
}

/**
* @return array
*/
public function renderDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,9 @@ public function testGetSortable($value)
$this->assertFalse($this->_block->getSortable());
}

/**
* @return array
*/
public function getSortableDataProvider()
{
return ['zero' => ['0'], 'false' => [false], 'null' => [null]];
Expand Down Expand Up @@ -379,6 +382,9 @@ public function testColumnIsGrouped($groupedData, $expected)
$this->assertEquals($expected, $block->isGrouped());
}

/**
* @return array
*/
public function columnGroupedDataProvider()
{
return [[[], false], [['grouped' => 0], false], [['grouped' => 1], true]];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,9 @@ public function testGetGridIdsJsonWithUseSelectAll(array $items, $result)
$this->assertEquals($result, $this->_block->getGridIdsJson());
}

/**
* @return array
*/
public function dataProviderGetGridIdsJsonWithUseSelectAll()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,9 @@ public function testItemsProcessing($itemId, $item, $expectedItem)
$this->assertNull($this->_block->getItem($itemId));
}

/**
* @return array
*/
public function itemsDataProvider()
{
return [
Expand Down Expand Up @@ -204,6 +207,9 @@ public function testSelected($param, $expectedJson, $expected)
$this->assertEquals($expected, $this->_block->getSelected());
}

/**
* @return array
*/
public function selectedDataProvider()
{
return [
Expand Down Expand Up @@ -259,6 +265,9 @@ public function testGetGridIdsJsonWithUseSelectAll(array $items, $result)
$this->assertEquals($result, $this->_block->getGridIdsJson());
}

/**
* @return array
*/
public function dataProviderGetGridIdsJsonWithUseSelectAll()
{
return [
Expand Down
3 changes: 3 additions & 0 deletions app/code/Magento/Backend/Test/Unit/Block/Widget/TabTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,9 @@ public function testGetters($method, $field, $value, $expected)
$this->assertEquals($expected, $object->{$method}());
}

/**
* @return array
*/
public function dataProvider()
{
return [
Expand Down
3 changes: 3 additions & 0 deletions app/code/Magento/Backend/Test/Unit/Helper/DataTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,9 @@ public function testPrepareFilterStringValues(array $inputString, array $expecte
$this->assertEquals($expected, $actual);
}

/**
* @return array
*/
public function getPrepareFilterStringValuesDataProvider()
{
return [
Expand Down
9 changes: 9 additions & 0 deletions app/code/Magento/Backend/Test/Unit/Model/Auth/SessionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,9 @@ public function testRefreshAcl($isUserPassedViaParams)
$this->assertSame($aclMock, $this->session->getAcl());
}

/**
* @return array
*/
public function refreshAclDataProvider()
{
return [
Expand Down Expand Up @@ -235,6 +238,9 @@ public function testIsAllowed($isUserDefined, $isAclDefined, $isAllowed, $expect
$this->assertEquals($expectedResult, $this->session->isAllowed('resource'));
}

/**
* @return array
*/
public function isAllowedDataProvider()
{
return [
Expand All @@ -255,6 +261,9 @@ public function testFirstPageAfterLogin($isFirstPageAfterLogin)
$this->assertEquals($isFirstPageAfterLogin, $this->session->isFirstPageAfterLogin());
}

/**
* @return array
*/
public function firstPageAfterLoginDataProvider()
{
return [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,9 @@ public function testBeforeSave($value, $errorMessage = null)
$model->beforeSave();
}

/**
* @return array
*/
public function adminSessionLifetimeDataProvider()
{
return [
Expand Down
3 changes: 3 additions & 0 deletions app/code/Magento/Backend/Test/Unit/Model/Menu/ConfigTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,9 @@ public function testGetMenuExceptionLogged($expectedException)
$this->_model->getMenu();
}

/**
* @return array
*/
public function getMenuExceptionLoggedDataProvider()
{
return [
Expand Down
Loading