Skip to content

Commit 07d6e7b

Browse files
committed
Fixed Web API test failure
1 parent d93c747 commit 07d6e7b

File tree

1 file changed

+39
-25
lines changed

1 file changed

+39
-25
lines changed

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

Lines changed: 39 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -435,62 +435,73 @@ public function testGetMatchingOrdersForLowerQueryLength()
435435
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart2.id$']),
436436
DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart2.id$']),
437437
DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart2.id$']),
438-
DataFixture(PlaceOrder::class, ['cart_id' => '$cart2.id$'], '100000002'),
438+
DataFixture(PlaceOrder::class, ['cart_id' => '$cart2.id$'], 'or2'),
439+
]
439440

441+
#[
440442
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart3'),
441443
DataFixture(ProductFixture::class, ['sku' => '100000003', 'price' => 10], 'p3'),
442444
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart3.id$', 'product_id' => '$p3.id$']),
443445
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart3.id$']),
444446
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart3.id$']),
445447
DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart3.id$']),
446448
DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart3.id$']),
447-
DataFixture(PlaceOrder::class, ['cart_id' => '$cart3.id$'], '100000003'),
449+
DataFixture(PlaceOrder::class, ['cart_id' => '$cart3.id$'], 'or3'),
450+
]
448451

452+
#[
449453
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart4'),
450454
DataFixture(ProductFixture::class, ['sku' => '100000004', 'price' => 10], 'p4'),
451455
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart4.id$', 'product_id' => '$p4.id$']),
452456
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart4.id$']),
453457
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart4.id$']),
454458
DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart4.id$']),
455459
DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart4.id$']),
456-
DataFixture(PlaceOrder::class, ['cart_id' => '$cart4.id$'], '100000004'),
460+
DataFixture(PlaceOrder::class, ['cart_id' => '$cart4.id$'], 'or4'),
461+
]
457462

463+
#[
458464
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart5'),
459465
DataFixture(ProductFixture::class, ['sku' => '100000005', 'price' => 10], 'p5'),
460466
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart5.id$', 'product_id' => '$p5.id$']),
461467
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart5.id$']),
462468
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart5.id$']),
463469
DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart5.id$']),
464470
DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart5.id$']),
465-
DataFixture(PlaceOrder::class, ['cart_id' => '$cart5.id$'], '100000005'),
471+
DataFixture(PlaceOrder::class, ['cart_id' => '$cart5.id$'], 'or5'),
472+
]
466473

474+
#[
467475
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart6'),
468476
DataFixture(ProductFixture::class, ['sku' => '100000006', 'price' => 10], 'p6'),
469477
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart6.id$', 'product_id' => '$p6.id$']),
470478
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart6.id$']),
471479
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart6.id$']),
472480
DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart6.id$']),
473481
DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart6.id$']),
474-
DataFixture(PlaceOrder::class, ['cart_id' => '$cart6.id$'], '100000006'),
482+
DataFixture(PlaceOrder::class, ['cart_id' => '$cart6.id$'], 'or6'),
483+
]
475484

485+
#[
476486
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart7'),
477487
DataFixture(ProductFixture::class, ['sku' => '100000007', 'price' => 10], 'p7'),
478488
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart7.id$', 'product_id' => '$p7.id$']),
479489
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart7.id$']),
480490
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart7.id$']),
481491
DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart7.id$']),
482492
DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart7.id$']),
483-
DataFixture(PlaceOrder::class, ['cart_id' => '$cart7.id$'], '100000007'),
493+
DataFixture(PlaceOrder::class, ['cart_id' => '$cart7.id$'], 'or7'),
494+
]
484495

496+
#[
485497
DataFixture(CustomerCart::class, ['customer_id' => '$customer.id$'], 'cart8'),
486498
DataFixture(ProductFixture::class, ['sku' => '100000008', 'price' => 10], 'p8'),
487499
DataFixture(AddProductToCartFixture::class, ['cart_id' => '$cart8.id$', 'product_id' => '$p8.id$']),
488500
DataFixture(SetBillingAddress::class, ['cart_id' => '$cart8.id$']),
489501
DataFixture(SetShippingAddress::class, ['cart_id' => '$cart8.id$']),
490502
DataFixture(SetDeliveryMethod::class, ['cart_id' => '$cart8.id$']),
491503
DataFixture(SetPaymentMethod::class, ['cart_id' => '$cart8.id$']),
492-
DataFixture(PlaceOrder::class, ['cart_id' => '$cart8.id$'], '100000008'),
493-
504+
DataFixture(PlaceOrder::class, ['cart_id' => '$cart8.id$'], 'or8'),
494505
]
495506
public function testGetCustomerDescendingSortedOrders()
496507
{
@@ -526,23 +537,26 @@ public function testGetCustomerDescendingSortedOrders()
526537
$this->assertArrayHasKey('items', $response['customer']['orders']);
527538
$customerOrderItemsInResponse = $response['customer']['orders']['items'];
528539

529-
$o2 = '100000002';
530-
$o3 = '100000003';
531-
$o4 = '100000004';
532-
$o5 = '100000005';
533-
$o6 = '100000006';
534-
$o7 = '100000007';
535-
$o8 = '100000008';
536-
537-
$order2 = $this->fixtures->get($o2)->getIncrementId();
538-
$order3 = $this->fixtures->get($o3)->getIncrementId();
539-
$order4 = $this->fixtures->get($o4)->getIncrementId();
540-
$order5 = $this->fixtures->get($o5)->getIncrementId();
541-
$order6 = $this->fixtures->get($o6)->getIncrementId();
542-
$order7 = $this->fixtures->get($o7)->getIncrementId();
543-
$order8 = $this->fixtures->get($o8)->getIncrementId();
544-
545-
$expectedOrderNumbers = [$order8, $order7, $order6, $order5, $order4, $order3, $order2 ];
540+
$order2 = $this->fixtures->get('or2')->getIncrementId();
541+
$order3 = $this->fixtures->get('or3')->getIncrementId();
542+
$order4 = $this->fixtures->get('or4')->getIncrementId();
543+
$order5 = $this->fixtures->get('or5')->getIncrementId();
544+
$order6 = $this->fixtures->get('or6')->getIncrementId();
545+
$order7 = $this->fixtures->get('or7')->getIncrementId();
546+
$order8 = $this->fixtures->get('or8')->getIncrementId();
547+
548+
$expectedOrderNumbersOptions = [$order8, $order7, $order6, $order5, $order4, $order3, $order2 ];
549+
$expectedOrderNumbers = $scalarTemp = [];
550+
$compDate = '';
551+
foreach ($expectedOrderNumbersOptions as $comKey => $comData) {
552+
if ($compDate == $customerOrderItemsInResponse[$comKey]['order_date']) {
553+
$expectedOrderNumbers = array_unshift($expectedOrderNumbers, $comData);
554+
} else {
555+
$scalarTemp = (array)$comData;
556+
$expectedOrderNumbers[$comKey] = $scalarTemp[0];
557+
}
558+
$compDate = $customerOrderItemsInResponse[$comKey]['order_date'];
559+
}
546560

547561
foreach ($expectedOrderNumbers as $key => $data) {
548562
$orderItemInResponse = $customerOrderItemsInResponse[$key];

0 commit comments

Comments
 (0)