Skip to content

Commit

Permalink
ENGCOM-7887: Unit Test / Bugfix / Inverted Expected value with actual…
Browse files Browse the repository at this point in the history
… one #29271

 - Merge Pull Request #29271 from lbajsarowicz/magento2:bugfix/inverted-arguments
 - Merged commits:
   1. ff0e8f6
  • Loading branch information
magento-engcom-team committed Jul 29, 2020
2 parents 5a95d1f + ff0e8f6 commit 56a1d99
Showing 1 changed file with 4 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -136,11 +136,8 @@ public function testSubscribe(
->with(Subscriber::XML_PATH_CONFIRMATION_FLAG, ScopeInterface::SCOPE_STORE, $storeId)
->willReturn($isConfirmNeed);

$this->assertEquals(
$subscriber,
$this->subscriptionManager->subscribe($email, $storeId)
);
$this->assertEquals($subscriber->getData(), $expectedData);
$this->assertEquals($subscriber, $this->subscriptionManager->subscribe($email, $storeId));
$this->assertEquals($expectedData, $subscriber->getData());
}

/**
Expand Down Expand Up @@ -308,7 +305,7 @@ public function testSubscribeCustomer(
$subscriber,
$this->subscriptionManager->subscribeCustomer($customerId, $storeId)
);
$this->assertEquals($subscriber->getData(), $expectedData);
$this->assertEquals($expectedData, $subscriber->getData());
}

/**
Expand Down Expand Up @@ -553,7 +550,7 @@ public function testUnsubscribeCustomer(
$subscriber,
$this->subscriptionManager->unsubscribeCustomer($customerId, $storeId)
);
$this->assertEquals($subscriber->getData(), $expectedData);
$this->assertEquals($expectedData, $subscriber->getData());
}

/**
Expand Down

0 comments on commit 56a1d99

Please sign in to comment.