Skip to content

Commit af73eea

Browse files
eliseacornejowip44850
eliseacornejo
authored and
wip44850
committed
LYNX-314: Been able to run tests locally when add product to cart fixture
1 parent 1b2e87c commit af73eea

File tree

7 files changed

+40
-8
lines changed

7 files changed

+40
-8
lines changed

‎app/code/Magento/Indexer/Test/Fixture/Indexer.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,9 @@ public function apply(array $data = []): ?DataObject
3737
$this->indexerCollection->load();
3838
/** @var IndexerModel $indexer */
3939
foreach ($this->indexerCollection->getItems() as $indexer) {
40-
$indexer->reindexAll();
40+
if ($indexer->getState()->getData('status') === 'invalid') {
41+
$indexer->reindexAll();
42+
}
4143
}
4244
return null;
4345
}

‎dev/tests/api-functional/testsuite/Magento/GraphQl/OrderCancellation/CancelOrderTest.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
use Magento\Framework\ObjectManagerInterface;
1111
use Magento\GraphQl\GetCustomerAuthenticationHeader;
12+
use Magento\Indexer\Test\Fixture\Indexer;
1213
use Magento\Quote\Api\CartManagementInterface;
1314
use Magento\Sales\Api\Data\OrderInterface;
1415
use Magento\Sales\Api\OrderRepositoryInterface;
@@ -54,6 +55,7 @@
5455
),
5556
DataFixture(ProductFixture::class, as: 'product'),
5657
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
58+
DataFixture(Indexer::class, as: 'indexer'),
5759
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$product.id$']),
5860
DataFixture(SetBillingAddressFixture::class, ['cart_id' => '$cart.id$']),
5961
DataFixture(SetShippingAddressFixture::class, ['cart_id' => '$cart.id$']),
@@ -244,6 +246,7 @@ public function testAttemptToCancelNonExistingOrder()
244246
}
245247

246248
#[
249+
DataFixture(Store::class),
247250
DataFixture(ProductFixture::class, as: 'product'),
248251
DataFixture(
249252
Customer::class,
@@ -262,6 +265,7 @@ public function testAttemptToCancelNonExistingOrder()
262265
'another'
263266
),
264267
DataFixture(CustomerCart::class, ['customer_id' => '$another.id$'], as: 'cart'),
268+
DataFixture(Indexer::class, as: 'indexer'),
265269
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$product.id$']),
266270
DataFixture(SetBillingAddressFixture::class, ['cart_id' => '$cart.id$']),
267271
DataFixture(SetShippingAddressFixture::class, ['cart_id' => '$cart.id$']),
@@ -384,6 +388,7 @@ public function testAttemptToCancelOrderWithSomeStatuses(string $status, string
384388
),
385389
DataFixture(ProductFixture::class, as: 'product'),
386390
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
391+
DataFixture(Indexer::class, as: 'indexer'),
387392
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$product.id$']),
388393
DataFixture(SetBillingAddressFixture::class, ['cart_id' => '$cart.id$']),
389394
DataFixture(SetShippingAddressFixture::class, ['cart_id' => '$cart.id$']),
@@ -455,6 +460,7 @@ public function testAttemptToCancelOrderWithOfflinePaymentFullyInvoicedFullyShip
455460
),
456461
DataFixture(ProductFixture::class, as: 'product'),
457462
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
463+
DataFixture(Indexer::class, as: 'indexer'),
458464
DataFixture(
459465
AddProductToCartFixture::class,
460466
[
@@ -539,6 +545,7 @@ public function testAttemptToCancelOrderWithOfflinePaymentFullyInvoicedPartially
539545
),
540546
DataFixture(ProductFixture::class, as: 'product'),
541547
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
548+
DataFixture(Indexer::class, as: 'indexer'),
542549
DataFixture(
543550
AddProductToCartFixture::class,
544551
[
@@ -611,6 +618,7 @@ public function testAttemptToCancelOrderWithOfflinePaymentFullyInvoicedFullyRefu
611618
),
612619
DataFixture(ProductFixture::class, as: 'product'),
613620
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
621+
DataFixture(Indexer::class, as: 'indexer'),
614622
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$product.id$']),
615623
DataFixture(SetBillingAddressFixture::class, ['cart_id' => '$cart.id$']),
616624
DataFixture(SetShippingAddressFixture::class, ['cart_id' => '$cart.id$']),
@@ -753,6 +761,7 @@ public function testCancelOrderWithOfflinePaymentFullyInvoiced()
753761
),
754762
DataFixture(ProductFixture::class, as: 'product'),
755763
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
764+
DataFixture(Indexer::class, as: 'indexer'),
756765
DataFixture(
757766
AddProductToCartFixture::class,
758767
[
@@ -906,6 +915,7 @@ public function testCancelOrderAttemptingXSSPassedThroughReasonField()
906915
DataFixture(ProductFixture::class, as: 'product1'),
907916
DataFixture(ProductFixture::class, as: 'product2'),
908917
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
918+
DataFixture(Indexer::class, as: 'indexer'),
909919
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$product1.id$']),
910920
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$product2.id$']),
911921
DataFixture(SetBillingAddressFixture::class, ['cart_id' => '$cart.id$']),

‎dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/AddSimpleProductToCartSingleMutationTest.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,12 @@
88
namespace Magento\GraphQl\Quote;
99

1010
use Magento\Catalog\Test\Fixture\Product as ProductFixture;
11+
use Magento\Indexer\Test\Fixture\Indexer;
1112
use Magento\Quote\Model\QuoteIdToMaskedQuoteIdInterface;
1213
use Magento\Quote\Test\Fixture\AddProductToCart as AddProductToCartFixture;
1314
use Magento\Quote\Test\Fixture\GuestCart as GuestCartFixture;
1415
use Magento\Store\Test\Fixture\Group as StoreGroupFixture;
16+
use Magento\Store\Test\Fixture\Store;
1517
use Magento\Store\Test\Fixture\Store as StoreFixture;
1618
use Magento\Store\Test\Fixture\Website as WebsiteFixture;
1719
use Magento\TestFramework\Fixture\DataFixture;
@@ -297,10 +299,12 @@ public function testAddMultipleProductsToEmptyCart(): void
297299
}
298300

299301
#[
302+
DataFixture(Store::class),
300303
DataFixture(ProductFixture::class, as: 'p1'),
301304
DataFixture(ProductFixture::class, as: 'p2'),
302305
DataFixture(ProductFixture::class, as: 'p3'),
303306
DataFixture(GuestCartFixture::class, as: 'cart'),
307+
DataFixture(Indexer::class, as: 'indexer'),
304308
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$p1.id$', 'qty' => 1]),
305309
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$p2.id$', 'qty' => 1]),
306310
]

‎dev/tests/api-functional/testsuite/Magento/GraphQl/Quote/Guest/CartTotalsTest.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,13 @@
1010
use Magento\Catalog\Test\Fixture\Product as ProductFixture;
1111
use Magento\Framework\GraphQl\Exception\GraphQlNoSuchEntityException;
1212
use Magento\GraphQl\Quote\GetMaskedQuoteIdByReservedOrderId;
13+
use Magento\Indexer\Test\Fixture\Indexer;
1314
use Magento\Quote\Model\QuoteIdToMaskedQuoteIdInterface;
1415
use Magento\Quote\Test\Fixture\AddProductToCart as AddProductToCartFixture;
1516
use Magento\Quote\Test\Fixture\GuestCart as GuestCartFixture;
1617
use Magento\Checkout\Test\Fixture\SetBillingAddress as SetBillingAddressFixture;
1718
use Magento\Checkout\Test\Fixture\SetShippingAddress as SetShippingAddressFixture;
19+
use Magento\Store\Test\Fixture\Store;
1820
use Magento\TestFramework\Fixture\DataFixture;
1921
use Magento\TestFramework\Fixture\DataFixtureStorage;
2022
use Magento\TestFramework\Fixture\DataFixtureStorageManager;
@@ -239,8 +241,10 @@ public function testGetCartTotalsWithEmptyCart()
239241
}
240242

241243
#[
244+
DataFixture(Store::class),
242245
DataFixture(ProductFixture::class, as: 'p'),
243246
DataFixture(GuestCartFixture::class, as: 'cart'),
247+
DataFixture(Indexer::class, as: 'indexer'),
244248
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$p.id$', 'qty' => 2]),
245249
DataFixture(SetBillingAddressFixture::class, ['cart_id' => '$cart.id$']),
246250
DataFixture(SetShippingAddressFixture::class, ['cart_id' => '$cart.id$']),

‎dev/tests/api-functional/testsuite/Magento/GraphQl/Sales/CustomerOrdersTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
use Magento\Customer\Test\Fixture\Customer;
1818
use Magento\Framework\Exception\LocalizedException;
1919
use Magento\Framework\ObjectManagerInterface;
20+
use Magento\Indexer\Test\Fixture\Indexer;
2021
use Magento\Quote\Test\Fixture\AddProductToCart;
2122
use Magento\Quote\Test\Fixture\CustomerCart;
2223
use Magento\Store\Test\Fixture\Group as StoreGroupFixture;
@@ -78,6 +79,7 @@ protected function setUp(): void
7879
as: 'customer'
7980
),
8081
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'quote1', scope: 'store2'),
82+
DataFixture(Indexer::class, as: 'indexer'),
8183
DataFixture(AddProductToCart::class, ['cart_id' => '$quote1.id$', 'product_id' => '$product.id$', 'qty' => 1]),
8284
DataFixture(SetBillingAddress::class, ['cart_id' => '$quote1.id$']),
8385
DataFixture(SetShippingAddress::class, ['cart_id' => '$quote1.id$']),

‎dev/tests/api-functional/testsuite/Magento/GraphQl/Sales/InvoiceTest.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,13 +16,15 @@
1616
use Magento\Checkout\Test\Fixture\SetShippingAddress as SetShippingAddressFixture;
1717
use Magento\Customer\Test\Fixture\Customer;
1818
use Magento\Framework\Registry;
19+
use Magento\Indexer\Test\Fixture\Indexer;
1920
use Magento\Quote\Test\Fixture\AddProductToCart as AddProductToCartFixture;
2021
use Magento\Quote\Test\Fixture\CustomerCart;
2122
use Magento\Quote\Test\Fixture\GuestCart as GuestCartFixture;
2223
use Magento\Sales\Api\OrderRepositoryInterface;
2324
use Magento\Sales\Model\ResourceModel\Order\Collection;
2425
use Magento\Sales\Test\Fixture\Invoice as InvoiceFixture;
2526
use Magento\Sales\Test\Fixture\InvoiceComment as InvoiceCommentFixture ;
27+
use Magento\Store\Test\Fixture\Store;
2628
use Magento\TestFramework\Fixture\DataFixture;
2729
use Magento\TestFramework\Helper\Bootstrap;
2830
use Magento\TestFramework\TestCase\GraphQlAbstract;
@@ -425,9 +427,11 @@ public function testPartialInvoiceForCustomerWithTaxesAndDiscounts()
425427
}
426428

427429
#[
430+
DataFixture(Store::class),
428431
DataFixture(Customer::class, ['email' => 'customer@search.example.com'], as: 'customer'),
429432
DataFixture(ProductFixture::class, as: 'product'),
430433
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], as: 'cart'),
434+
DataFixture(Indexer::class, as: 'indexer'),
431435
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart.id$', 'product_id' => '$product.id$']),
432436
DataFixture(SetBillingAddressFixture::class, ['cart_id' => '$cart.id$']),
433437
DataFixture(SetShippingAddressFixture::class, ['cart_id' => '$cart.id$']),

‎dev/tests/api-functional/testsuite/Magento/GraphQl/Sales/RetrieveOrdersByOrderNumberTest.php

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,10 @@
1212
use Magento\Framework\Exception\AuthenticationException;
1313
use Magento\Framework\Registry;
1414
use Magento\GraphQl\GetCustomerAuthenticationHeader;
15+
use Magento\Indexer\Test\Fixture\Indexer;
1516
use Magento\Sales\Api\OrderRepositoryInterface;
1617
use Magento\Sales\Model\ResourceModel\Order\Collection;
18+
use Magento\Store\Test\Fixture\Store;
1719
use Magento\TestFramework\Helper\Bootstrap;
1820
use Magento\TestFramework\TestCase\GraphQlAbstract;
1921
use Magento\Catalog\Test\Fixture\Product as ProductFixture;
@@ -433,10 +435,20 @@ public function testGetMatchingOrdersForLowerQueryLength()
433435
* @return void
434436
* @throws AuthenticationException
435437
*/
438+
#[
439+
DataFixture(Store::class),
440+
DataFixture(ProductFixture::class, ['sku' => '100000002', 'price' => 10], 'p2'),
441+
DataFixture(ProductFixture::class, ['sku' => '100000003', 'price' => 10], 'p3'),
442+
DataFixture(ProductFixture::class, ['sku' => '100000004', 'price' => 10], 'p4'),
443+
DataFixture(ProductFixture::class, ['sku' => '100000005', 'price' => 10], 'p5'),
444+
DataFixture(ProductFixture::class, ['sku' => '100000006', 'price' => 10], 'p6'),
445+
DataFixture(ProductFixture::class, ['sku' => '100000007', 'price' => 10], 'p7'),
446+
DataFixture(ProductFixture::class, ['sku' => '100000008', 'price' => 10], 'p8'),
447+
DataFixture(Indexer::class, as: 'indexer')
448+
]
436449
#[
437450
DataFixture(Customer::class, ['email' => 'customer@example.com'], 'customer'),
438451
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart2'),
439-
DataFixture(ProductFixture::class, ['sku' => '100000002', 'price' => 10], 'p2'),
440452
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart2.id$', 'product_id' => '$p2.id$']),
441453
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart2.id$']),
442454
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart2.id$']),
@@ -447,7 +459,6 @@ public function testGetMatchingOrdersForLowerQueryLength()
447459

448460
#[
449461
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart3'),
450-
DataFixture(ProductFixture::class, ['sku' => '100000003', 'price' => 10], 'p3'),
451462
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart3.id$', 'product_id' => '$p3.id$']),
452463
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart3.id$']),
453464
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart3.id$']),
@@ -458,7 +469,6 @@ public function testGetMatchingOrdersForLowerQueryLength()
458469

459470
#[
460471
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart4'),
461-
DataFixture(ProductFixture::class, ['sku' => '100000004', 'price' => 10], 'p4'),
462472
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart4.id$', 'product_id' => '$p4.id$']),
463473
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart4.id$']),
464474
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart4.id$']),
@@ -469,7 +479,6 @@ public function testGetMatchingOrdersForLowerQueryLength()
469479

470480
#[
471481
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart5'),
472-
DataFixture(ProductFixture::class, ['sku' => '100000005', 'price' => 10], 'p5'),
473482
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart5.id$', 'product_id' => '$p5.id$']),
474483
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart5.id$']),
475484
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart5.id$']),
@@ -480,7 +489,6 @@ public function testGetMatchingOrdersForLowerQueryLength()
480489

481490
#[
482491
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart6'),
483-
DataFixture(ProductFixture::class, ['sku' => '100000006', 'price' => 10], 'p6'),
484492
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart6.id$', 'product_id' => '$p6.id$']),
485493
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart6.id$']),
486494
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart6.id$']),
@@ -491,7 +499,6 @@ public function testGetMatchingOrdersForLowerQueryLength()
491499

492500
#[
493501
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart7'),
494-
DataFixture(ProductFixture::class, ['sku' => '100000007', 'price' => 10], 'p7'),
495502
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart7.id$', 'product_id' => '$p7.id$']),
496503
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart7.id$']),
497504
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart7.id$']),
@@ -502,7 +509,6 @@ public function testGetMatchingOrdersForLowerQueryLength()
502509

503510
#[
504511
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart8'),
505-
DataFixture(ProductFixture::class, ['sku' => '100000008', 'price' => 10], 'p8'),
506512
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart8.id$', 'product_id' => '$p8.id$']),
507513
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart8.id$']),
508514
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart8.id$']),

0 commit comments

Comments
 (0)