Skip to content

Commit 56a1d99

Browse files
ENGCOM-7887: Unit Test / Bugfix / Inverted Expected value with actual one #29271
- Merge Pull Request #29271 from lbajsarowicz/magento2:bugfix/inverted-arguments - Merged commits: 1. ff0e8f6
2 parents 5a95d1f + ff0e8f6 commit 56a1d99

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

app/code/Magento/Newsletter/Test/Unit/Model/SubscriptionManagerTest.php

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -136,11 +136,8 @@ public function testSubscribe(
136136
->with(Subscriber::XML_PATH_CONFIRMATION_FLAG, ScopeInterface::SCOPE_STORE, $storeId)
137137
->willReturn($isConfirmNeed);
138138

139-
$this->assertEquals(
140-
$subscriber,
141-
$this->subscriptionManager->subscribe($email, $storeId)
142-
);
143-
$this->assertEquals($subscriber->getData(), $expectedData);
139+
$this->assertEquals($subscriber, $this->subscriptionManager->subscribe($email, $storeId));
140+
$this->assertEquals($expectedData, $subscriber->getData());
144141
}
145142

146143
/**
@@ -308,7 +305,7 @@ public function testSubscribeCustomer(
308305
$subscriber,
309306
$this->subscriptionManager->subscribeCustomer($customerId, $storeId)
310307
);
311-
$this->assertEquals($subscriber->getData(), $expectedData);
308+
$this->assertEquals($expectedData, $subscriber->getData());
312309
}
313310

314311
/**
@@ -553,7 +550,7 @@ public function testUnsubscribeCustomer(
553550
$subscriber,
554551
$this->subscriptionManager->unsubscribeCustomer($customerId, $storeId)
555552
);
556-
$this->assertEquals($subscriber->getData(), $expectedData);
553+
$this->assertEquals($expectedData, $subscriber->getData());
557554
}
558555

559556
/**

0 commit comments

Comments
 (0)