Merge pull request #65 from nullinside-development-group/feature/BetterLogging

Feature/better logging

Author: ProgrammingByPermutation

src/Nullinside.Api.TwitchBot.Tests/ChatRules/BestCheapViewersTests.cs

@@ -15,6 +15,10 @@ public class BestCheapViewersTests : AChatRuleUnitTestBase<BestCheapViewers> {
   /// </summary>
   /// <param name="badString">The string that should fail.</param>
   [Test]
+  [TestCase("B͐est vi̯ewers o͎n on streamboo .com ( remove the space )  @9F3Wnft0")]
+  [TestCase("Ch̚eͅap viewers on ***  @STGPMoLg")]
+  [TestCase("C̭heap viewe͌rs on̆ ***  @R1QXrXPM")]
+  [TestCase("C̭heap viewe͌rs on̆ vwrpro.ru  @8v2JcQFL")]
   [TestCase("C\u032dheap viewe\u034crs on\u0306 vwrpro.ru @8v2JcQFL")]
   [TestCase("Ch\u031ae\u0345ap viewers on ***  @STGPMoLg")]
   [TestCase("Best\u036e vie\u0350wers \u0337on ***")]

src/Nullinside.Api.TwitchBot/ChatRules/Discord.cs

@@ -13,9 +13,11 @@ namespace Nullinside.Api.TwitchBot.ChatRules;
 /// </summary>
 public class Discord : AChatRule {
   private readonly string[] KNOWN_PHRASES = [
-    "add me on discord",
-    "my username is",
-    "my discord username is",
+    "add me",
+    "message me",
+    "chat with me",
+    "my discord username",
+    "my username",
     "connect on discord"
   ];
 

src/Nullinside.Api.TwitchBot/ChatRules/TwitchChatMessageMonitorConsumer.cs

@@ -4,6 +4,8 @@
 
 using Microsoft.EntityFrameworkCore;
 
+using Mysqlx.Expr;
+
 using Nullinside.Api.Common.Twitch;
 using Nullinside.Api.Model;
 using Nullinside.Api.Model.Ddl;
@@ -160,7 +162,7 @@ private async void MainLoop() {
               }
             }
             catch (Exception e) {
-              LOG.Error($"{channel}: Failed to evaluate rule", e);
+              LOG.Error($"{channel}: Failed to evaluate rule on {message.Username}({message.UserId}): {message.Message}", e);
             }
           }
         }