Skip to content

Commit ff0e8f6

Browse files
committed
Unit Test / Bugfix / Inverted Expected value with actual one
1 parent 1ee7e1f commit ff0e8f6

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)