-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Optimise test framework auto topup #120 #123
Conversation
88b5149
to
36529a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests/BunqSdkTestBase.php
Outdated
*/ | ||
const SPENDING_MONEY_AMOUNT = '500'; | ||
const SPENDING_MONEY_RECIPIENT = 'sugardaddy@bunq.com'; | ||
const SPENDING_MONEY_DESCRIPTION = 'sdk php test, thanks daddy <3 - OG'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why - OG
? I'd suggest to keep it clean.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where's the fun in that? 😜
new Amount(self::PAYMENT_AMOUNT_IN_EUR, self::PAYMENT_CURRENCY), | ||
$this->getPointerUserBravo(), | ||
self::PAYMENT_DESCRIPTION | ||
); | ||
|
||
static::assertNotNull($response); | ||
} | ||
|
||
/** | ||
* Test sending money to other monetaryAccount. | ||
* | ||
* This test has no assertion as of its testing to see if the code runs without errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test has no assertion as it is testing whether the code runs without errors.
@@ -82,12 +84,14 @@ public function testSendMoneyToOtherMonetaryAccount() | |||
* | |||
* @depends testSendMoneyToOtherMonetaryAccount | |||
*/ | |||
public function testSendMessageToPayment() | |||
public function testSendMessageToPayment(BunqResponseInt $paymentId) | |||
{ | |||
$chatId = PaymentChat::create( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fits on one line? On purpose formatted like this?
59b2aff
to
8f959d2
Compare
@sandervdo please 👀 |
This PR closes/fixes the following issues: