Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
ESDEV-3779 Provide user for getPaymentList
task: ESDEV-4571

`getPaymentList` implementation inside the shop is able to handle
the case when no user is specified, unfortunately not all modules
are created with this same expectation thus this change allows
to not cause fatal errors for modules which are not capable of
working with undefined users.

This issue was noticed while working with PayOne module.
  • Loading branch information
rezonanc-oxid committed Jun 20, 2017
1 parent 1e6d589 commit eedde9b
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions tests/Unit/Application/Controller/PaymentTest.php
Expand Up @@ -288,6 +288,7 @@ public function testIsOldDebitValidationEnabled_configSkipEnabled_disabled()

public function testGetDynValueSetInSession()
{
$this->setSessionParam('usr', 'oxdefaultadmin');
$this->setSessionParam('dynvalue', 'test');
$this->setRequestParameter('dynvalue', 'test2');

Expand All @@ -297,6 +298,7 @@ public function testGetDynValueSetInSession()

public function testGetDynValueNotSetInSession()
{
$this->setSessionParam('usr', 'oxdefaultadmin');
$this->setSessionParam('dynvalue', null);
$this->setRequestParameter('dynvalue', 'test2');

Expand Down

0 comments on commit eedde9b

Please sign in to comment.