Fix: Allow specifying targets with @. Closes #146

src/player_controller.js

@@ -971,16 +971,23 @@ class MafiaPlayerController {
 			command.args.shift();
 		}
 
-		const target = command.args.shift();
-		//const gameName = Utils.argParse(command.args, []) || command.parent.ids.topic;
-
-
+		let target = command.args.shift();
+
 		if (!target) {
 			command.reply('Invalid command: Usage `!chat with somePlayer`');
 			return Promise.resolve();
 		}
 
 
+
+		if (target.startsWith('@')) {
+			target = target.substring(1);
+		}
+		//const gameName = Utils.argParse(command.args, []) || command.parent.ids.topic;
+
+
+
+
 		let game = null;
 		let postmanToggle;
 		let user;

test/unit/playerControllerTest.js

@@ -2487,6 +2487,18 @@ describe('player controller', () => {
 				controller.forum.User.getByName.should.be.calledWith(target);
 			});
 		});
+		it('should include target when target has an "@"', () => {
+			const target = `user${Math.random()}`,
+				user = {
+					id: Math.random()
+				};
+
+				command.args = [`@${target}`];
+			controller.forum.User.getByName.resolves(user);
+			return controller.createChatHandler(command).then(() => {
+				game.getPlayer.should.be.calledWith(target);
+			});
+		});
 		it('should send expected message', () => {
 			game.name = `NAME ${Math.random()} NAME`;
 			const expected = `This is an officially sanctioned chat for ${game.name}`;