Skierowanie improvements (#59)

* Skierowanie improvments - better links, bug fixes

* Apply suggestions from code review

Author: Adam Siekierski

src/commands/skierowanie.spec.ts

@@ -10,7 +10,13 @@ import * as Discord from 'discord.js';
 describe('skierowanie', () => {
   const mockAuthor = { username: 'user', avatarURL: 'url' };
 
-  it('it should send a message', async () => {
+  const mockLinksEmbed = (links: string[]) =>
+    new Discord.RichEmbed().addField(
+      'Z powyższym skierowaniem należy udać się na poniższe strony internetowe:',
+      links.join('\n')
+    );
+
+  it('it should send two messages', async () => {
     const msg = getMessageMock('msg', { author: mockAuthor });
 
     await skierowanie.execute((msg as unknown) as Discord.Message, ['user']);
@@ -24,13 +30,12 @@ describe('skierowanie', () => {
     await skierowanie.execute((msg as unknown) as Discord.Message, ['user', 'react']);
 
     const linksMessageMock = [
-      'Z powyższym skierowaniem należy udać się na poniższe strony internetowe:',
       'https://reactjs.org/docs',
       'https://developer.mozilla.org/en-US/docs/Learn',
       'https://typeofweb.com',
       'https://frontlive.pl',
     ];
 
-    await expect(msg.channel.send).to.have.been.calledWithExactly(linksMessageMock);
+    await expect(msg.channel.send).to.have.been.calledWithExactly(mockLinksEmbed(linksMessageMock));
   });
 });

src/commands/skierowanie.ts

@@ -17,22 +17,25 @@ const skierowanie: Command = {
   args: true,
   cooldown: 10,
   async execute(msg, args) {
+    // Filter out the empty args (whitespaces, etc)
+    args = args.filter((val) => val);
+
     const skierowanieEmbed = new Discord.RichEmbed()
       .setColor('#5ab783')
       .setAuthor(
         `Type of Web oraz ${msg.author.username}`,
         msg.author.avatarURL,
         'https://typeofweb.com'
       )
-      .setTitle('Skierowanie na naukę podstaw')
+      .setTitle('Skierowanie na naukę podstaw 🚑')
       .setThumbnail('https://typeofweb.com/wp-content/uploads/2020/04/logo_kwadrat11.png')
       .addField(
         'Działając na podstawie mojej intuicji oraz wiadomości wysłanych przez osobę skierowaną, kieruję użytkownika/użytkowniczkę',
         args[0]
       )
       .addField(
-        `na naukę podstaw wybranej przez siebie technologii, w celu lepszego zrozumienia fundamentów jej działania oraz poznania informacji niezbędnych do rozszerzania swojej wiedzy o bardziej zaawansowane zagadnienia`,
-        `Obecnie posiadana wiedza przez wyżej wymienioną osobę jest zbyt wąska, by mogła ona dalej kontynuować swoją naukę w trudniejszych zagadnieniach w sposób efektywny i zrozumiały`
+        `na naukę podstaw wybranej przez siebie technologii,`,
+        `w celu lepszego zrozumienia fundamentów jej działania oraz poznania informacji niezbędnych do rozszerzania swojej wiedzy o bardziej zaawansowane zagadnienia`
       )
       .setTimestamp()
       .setFooter(
@@ -43,12 +46,17 @@ const skierowanie: Command = {
     const categoryFilter = args[1]?.toLowerCase();
     const linksFiltered = categoryFilter
       ? links.filter(({ category }) => !category || category === categoryFilter)
-      : links;
+      : links.filter(({ category }) => !category);
 
     const linksMessage = 'Z powyższym skierowaniem należy udać się na poniższe strony internetowe:';
 
+    const linksEmbed = new Discord.RichEmbed().addField(
+      linksMessage,
+      linksFiltered.map((l) => l.url).join('\n')
+    );
+
     await msg.channel.send(skierowanieEmbed);
-    return msg.channel.send([linksMessage, ...linksFiltered.map((link) => link.url)]);
+    return msg.channel.send(linksEmbed);
   },
 };