deavmi/birchwood
deavmi/birchwood
1
0
Fork 0
mirror of https://github.com/deavmi/birchwood synced 2024-09-19 23:03:44 +02:00
Code Issues Releases Activity

Client

Browse Source 
- Fixed bug with `directMessage(string, string[])` when more than one user in a multi-case was passed in

Unit tests

- Added two more unit tests relating to direct messages
- Namely testing out two scenarios for `directMessage(string, string[])`
This commit is contained in:
Tristan B. Velloza Kildaire 2023-03-09 14:34:53 +02:00
parent 802fd9823d
commit d8908338c5
1 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
source/birchwood/client.d
View File

@ -409,6 +409,9 @@ public class Client : Thread
/* Ensure valid characters in first recipient */
if(isValidText(recipientLine))
{
/* Append on a trailing `,` */
recipientLine ~= ",";
for(ulong i = 1; i < recipients.length; i++)
{
string currentRecipient = recipients[i];
@ -1226,7 +1229,19 @@ public class Client : Thread
/**
* Test sending a message to myself (singular)
*/
client.directMessage("Message to myself", "birchwood");
client.directMessage("(1) Message to myself", "birchwood");
/**
* Test sending a message to myself (multi)
*/
client.directMessage("(2) Message to myself (multi)", ["birchwood"]);
/**
* Test sending a message to myself 2x (multi)
*/
client.directMessage("(3) Message to myself (multi)", ["birchwood", "birchwood"]);
/**