From d8908338c55559d6a4a0586625af66d12325f062 Mon Sep 17 00:00:00 2001 From: "Tristan B. Velloza Kildaire" Date: Thu, 9 Mar 2023 14:34:53 +0200 Subject: [PATCH] Client - 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[])` --- source/birchwood/client.d | 17 ++++++++++++++++- 1 file changed, 16 insertions(+), 1 deletion(-) diff --git a/source/birchwood/client.d b/source/birchwood/client.d index 01c2d45..c27ce46 100644 --- a/source/birchwood/client.d +++ b/source/birchwood/client.d @@ -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"]); + + /**