33297: Eliminate AspectMock from DataExtensionUtilTest

karyna-tsymbal-atwix commented 3 years ago

Description

Eliminated AspectMock usage from dev/tests/unit/Magento/FunctionalTestFramework/DataGenerator/Util/DataExtensionUtilTest.php

In the scope of this ticket I also refactored dev/tests/unit/Magento/FunctionalTestFramework/DataGenerator/Persist/OperationDataArrayResolverTest.php

DataObjectHandler instance, mocked with AspectMock, is not cleaned in OperationDataArrayResolverTest, thus it broke DataExtensionUtilTest after my refactoring. As our main purpose to eliminate AspectMock completely, I decided to refactor OperationDataArrayResolverTest also.

Fixed Issues (if relevant)

Fixes magento/magento2#33297: [MFTF] Eliminate AspectMock from DataExtensionUtilTest (Complex!)

Contribution checklist

[ ] Pull request has a meaningful description of its purpose
[ ] All commits are accompanied by meaningful commit messages
[ ] All new or changed code is covered with unit/verification tests (if applicable)
[ ] All automated tests passed successfully (all builds on Travis CI are green)
[ ] Changes to Framework doesn't have backward incompatible changes for tests or have related Pull Request with fixes to tests

jilu1 commented 3 years ago

@magento import pull request to https://github.com/magento-commerce/magento2-functional-testing-framework

magento-engcom-team commented 3 years ago

@jilu1 the pull request successfully imported.

jilu1 commented 3 years ago

@karyna-tsymbal-atwix Any reason you closed this PR?

karyna-tsymbal-atwix commented 3 years ago

Hi @jilu1 , there were some changes, a new PR is here: https://github.com/magento/magento2-functional-testing-framework/pull/856

magento / magento2-functional-testing-framework