Skip to content

Commit 2ececb4

Browse files
committed
AC-7566::2.4.6-alpha2-Composer Integration Test Failures (Broken tests)
1 parent a1b6d99 commit 2ececb4

File tree

5 files changed

+24
-13
lines changed

5 files changed

+24
-13
lines changed

dev/tests/api-functional/testsuite/Magento/CheckoutAgreements/Api/CheckoutAgreementsRepositoryTest.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,10 @@ public function testGetListReturnsEmptyListIfCheckoutAgreementsAreDisabledOnFron
6565
*/
6666
public function testGetListReturnsTheListOfActiveCheckoutAgreements()
6767
{
68-
69-
$this->markTestSkipped('This test relies on system configuration state..');
68+
$this->markTestSkipped('This test relies on system configuration state.');
69+
// checkout/options/enable_agreements must be set to 1 in system configuration
70+
// @todo remove above statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
71+
7072
$agreementModel = $this->getAgreementByName('Checkout Agreement (active)');
7173

7274
$agreements = $this->_webApiCall($this->listServiceInfo, []);

dev/tests/api-functional/testsuite/Magento/GiftMessage/Api/CartRepositoryTest.php

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,10 @@ public function testGetForMyCart()
102102
*/
103103
public function testSave()
104104
{
105+
$this->markTestSkipped('This test relies on system configuration state.');
106+
// sales/gift_options/allow_order must be set to 1 in system configuration
107+
// @todo remove above statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
105108

106-
$this->markTestSkipped('This test relies on system configuration state..');
107109
/** @var \Magento\Quote\Model\Quote $quote */
108110
$quote = $this->objectManager->create(\Magento\Quote\Model\Quote::class);
109111
$quote->load('test_order_item_with_message', 'reserved_order_id');
@@ -154,8 +156,9 @@ public function testSaveForMyCart()
154156
);
155157
$token = $customerTokenService->createCustomerAccessToken('[email protected]', 'password');
156158

157-
158-
$this->markTestSkipped('This test relies on system configuration state..');
159+
$this->markTestSkipped('This test relies on system configuration state.');
160+
// sales/gift_options/allow_order must be set to 1 in system configuration
161+
// @todo remove above statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
159162

160163
$serviceInfo = [
161164
'rest' => [

dev/tests/api-functional/testsuite/Magento/GiftMessage/Api/GuestCartRepositoryTest.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,10 @@ public function testGet()
7373
*/
7474
public function testSave()
7575
{
76-
77-
$this->markTestSkipped('This test relies on system configuration state..');
76+
$this->markTestSkipped('This test relies on system configuration state.');
77+
// sales/gift_options/allow_order must be set to 1 in system configuration
78+
// @todo remove above statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
79+
7880
/** @var \Magento\Quote\Model\Quote $quote */
7981
$quote = $this->objectManager->create(\Magento\Quote\Model\Quote::class);
8082
$quote->load('test_order_item_with_message', 'reserved_order_id');

dev/tests/api-functional/testsuite/Magento/GiftMessage/Api/GuestItemRepositoryTest.php

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,10 @@ public function testGet()
7878
*/
7979
public function testSave()
8080
{
81-
8281
$this->markTestSkipped('This test relies on system configuration state.');
83-
82+
// sales/gift_options/allow_items must be set to 1 in system configuration
83+
// @todo remove above statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
84+
8485
/** @var \Magento\Quote\Model\Quote $quote */
8586
$quote = $this->objectManager->create(\Magento\Quote\Model\Quote::class);
8687
$quote->load('test_order_item_with_message', 'reserved_order_id');

dev/tests/api-functional/testsuite/Magento/GiftMessage/Api/ItemRepositoryTest.php

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -112,8 +112,10 @@ public function testGetForMyCart()
112112
*/
113113
public function testSave()
114114
{
115-
116-
$this->markTestSkipped('This test relies on system configuration state..');
115+
$this->markTestSkipped('This test relies on system configuration state.');
116+
// sales/gift_options/allow_items must be set to 1 in system configuration
117+
// @todo remove above statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
118+
117119
/** @var \Magento\Quote\Model\Quote $quote */
118120
$quote = $this->objectManager->create(\Magento\Quote\Model\Quote::class);
119121
$quote->load('test_order_item_with_message', 'reserved_order_id');
@@ -165,9 +167,10 @@ public function testSaveForMyCart()
165167
);
166168
$token = $customerTokenService->createCustomerAccessToken('[email protected]', 'password');
167169

168-
// sales/gift_options/allow_items must be set to 1 in system configuration
169-
// @todo remove next statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
170170
$this->markTestSkipped('This test relies on system configuration state.');
171+
// sales/gift_options/allow_items must be set to 1 in system configuration
172+
// @todo remove above statement when \Magento\TestFramework\TestCase\WebapiAbstract::_updateAppConfig is fixed
173+
171174
/** @var \Magento\Quote\Model\Quote $quote */
172175
$quote = $this->objectManager->create(\Magento\Quote\Model\Quote::class);
173176
$quote->load('test_order_item_with_message', 'reserved_order_id');

0 commit comments

Comments
 (0)