From 2e9036b26aa7923031cb5d23649ff397dc59e58d Mon Sep 17 00:00:00 2001
From: Noikoio <noikoio1@gmail.com>
Date: Tue, 29 Aug 2017 22:39:59 -0700
Subject: [PATCH] Log autoresponse configuration load

---
 Feature/AutoRespond/AutoRespond.cs        | 8 +++++---
 Feature/AutoRespond/ResponseDefinition.cs | 4 +++-
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/Feature/AutoRespond/AutoRespond.cs b/Feature/AutoRespond/AutoRespond.cs
index 51283d3..623a091 100644
--- a/Feature/AutoRespond/AutoRespond.cs
+++ b/Feature/AutoRespond/AutoRespond.cs
@@ -42,13 +42,15 @@ namespace Noikoio.RegexBot.Feature.AutoRespond
         }
 
         [ConfigSection("autoresponses")]
-        public override Task<object> ProcessConfiguration(JToken configSection)
+        public override async Task<object> ProcessConfiguration(JToken configSection)
         {
             var responses = new List<ResponseDefinition>();
             foreach (var def in configSection.Children<JProperty>())
             {
-                // Everything is left to the constructor
-                responses.Add(new ResponseDefinition(def));
+                // All validation is left to the constructor
+                var resp = new ResponseDefinition(def);
+                responses.Add(resp);
+                await Log($"Added definition '{resp.Label}'");
             }
 
             return Task.FromResult<object>(responses.AsReadOnly());
diff --git a/Feature/AutoRespond/ResponseDefinition.cs b/Feature/AutoRespond/ResponseDefinition.cs
index 5aaa268..2e66502 100644
--- a/Feature/AutoRespond/ResponseDefinition.cs
+++ b/Feature/AutoRespond/ResponseDefinition.cs
@@ -8,7 +8,7 @@ namespace Noikoio.RegexBot.Feature.AutoRespond
     /// <summary>
     /// Represents a single autoresponse definition.
     /// </summary>
-    struct ResponseDefinition
+    class ResponseDefinition
     {
         public enum ResponseType { None, Exec, Reply }
 
@@ -96,5 +96,7 @@ namespace Noikoio.RegexBot.Feature.AutoRespond
                 }
             }
         }
+
+        public override string ToString() => $"Autoresponse definition '{Label}'";
     }
 }