Merge pull request #58 from nullinside-development-group/feature/UpdateBotRules

Adding bot messages and test messages

Author: ProgrammingByPermutation

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

@@ -14,6 +14,7 @@ public class BestCheapViewersTests : AChatRuleUnitTestBase<BestCheapViewers> {
   /// </summary>
   /// <param name="badString">The string that should fail.</param>
   [Test]
+  [TestCase("Best\u036e vie\u0350wers \u0337on ***")]
   [TestCase("Best viewers on ***")]
   [TestCase("Best  viewers on ***")]
   [TestCase("Best vie̮wers on ***")]

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

@@ -14,6 +14,8 @@ public class DogehypeTests : AChatRuleUnitTestBase<Dogehype> {
   /// </summary>
   /// <param name="badString">The string that should fail.</param>
   [Test]
+  [TestCase("Visit dogehype dot com and watch your channel grow today!  @5MGxTnYl")]
+  [TestCase("Visit dogehype dot com today and climb the Twitch rankings with ease! Whether you're just starting out or looking to take your stream to the next level, DogeHype has the tools you need to succeed.  @gqznceDC")]
   [TestCase("Visit dogehype .biz com today and climb the Twitch rankings with ease! Whether you're just starting out or looking to take your stream to the next level, DogeHype has the tools you need to succeed.  @Axxq7ntz")]
   public async Task TestKnownStrings(string badString) {
     var rule = new Dogehype();

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

@@ -14,6 +14,8 @@ public class NezhnaTests : AChatRuleUnitTestBase<Nezhna> {
   /// </summary>
   /// <param name="badString">The string that should fail.</param>
   [Test]
+  [TestCase("Visit nezhna .com and watch your channel grow today!  @gG6SC5d3")]
+  [TestCase("Visit nezhna dot com to boost your viewers and climb the Twitch rankings. Join thousands of successful streamers now!  @0tlVpgrw")]
   [TestCase("Visit nezhna dot com com to boost your viewers and climb the Twitch rankings. Join thousands of successful streamers now!  @7xgkq3EK")]
   public async Task TestKnownStrings(string badString) {
     var rule = new Nezhna();

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

@@ -10,7 +10,7 @@ namespace Nullinside.Api.TwitchBot.ChatRules;
 ///   Handles the "nezhna dot com" bots.
 /// </summary>
 public class Nezhna : AChatRule {
-  private const string SPAM = "Visit nezhna dot com com to boost your viewers and climb the Twitch rankings. Join thousands of successful streamers now!";
+  private const string SPAM = "nezhna";
 
   /// <inheritdoc />
   public override bool ShouldRun(TwitchUserConfig config) {
@@ -24,11 +24,7 @@ public override async Task<bool> Handle(string channelId, ITwitchApiProxy botPro
       return true;
     }
 
-    // The number of spaces per message may chance, so normalize that and lowercase it for comparison.
-    string normalized = string.Join(' ', message.Message.Split(" ").Where(s => !string.IsNullOrWhiteSpace(s)))
-      .ToLowerInvariant();
-
-    if (normalized.StartsWith(SPAM, StringComparison.InvariantCultureIgnoreCase)) {
+    if (message.Message.Contains(SPAM, StringComparison.InvariantCultureIgnoreCase)) {
       await BanAndLog(channelId, botProxy, new[] { (message.UserId, message.Username) },
         "[Bot] Spam (Nezhna)", db, stoppingToken);
       return false;