Merge pull request #406 from iceljc/features/refine-agent-refresh

add log in agent refresh

Author: Oceania2018

src/Infrastructure/BotSharp.Core/Agents/Services/AgentService.RefreshAgents.cs

@@ -51,10 +51,12 @@ public async Task<string> RefreshAgents()
                 var isAgentDeleted = _db.DeleteAgent(agent.Id);
                 if (isAgentDeleted)
                 {
+                    await Task.Delay(100);
                     _db.BulkInsertAgents(new List<Agent> { agent });
                     _db.BulkInsertUserAgents(new List<UserAgent> { userAgent });
                     _db.BulkInsertAgentTasks(tasks);
                     refreshedAgents.Add(agent.Name);
+                    _logger.LogInformation($"Agent {agent.Name} has been migrated.");
                 }
             }
             catch (Exception ex)
@@ -66,7 +68,7 @@ public async Task<string> RefreshAgents()
         if (!refreshedAgents.IsNullOrEmpty())
         {
             Utilities.ClearCache();
-            refreshResult = $"Agents are migrated! {string.Join("\r\n", refreshedAgents)}";
+            refreshResult = $"Agents are migrated!\r\n{string.Join("\r\n", refreshedAgents)}";
         }
         else
         {

src/Infrastructure/BotSharp.Core/Agents/Services/AgentService.UpdateAgent.cs

@@ -41,10 +41,13 @@ public async Task UpdateAgent(Agent agent, AgentField updateField)
 
     public async Task<string> UpdateAgentFromFile(string id)
     {
+        string updateResult;
         var agent = _db.GetAgent(id);
         if (agent == null)
         {
-            return $"Cannot find agent ${id}";
+            updateResult = $"Cannot find agent ${id}";
+            _logger.LogError(updateResult);
+            return updateResult;
         }
 
         var dbSettings = _services.GetRequiredService<BotSharpDatabaseSettings>();
@@ -57,7 +60,9 @@ public async Task<string> UpdateAgentFromFile(string id)
         var foundAgent = FetchAgentFileById(agent.Id, filePath);
         if (foundAgent == null)
         {
-            return $"Cannot find agent {agent.Name} in file directory: {filePath}";
+            updateResult = $"Cannot find agent {agent.Name} in file directory: {filePath}";
+            _logger.LogError(updateResult);
+            return updateResult;
         }
 
         try
@@ -79,11 +84,16 @@ public async Task<string> UpdateAgentFromFile(string id)
 
             _db.UpdateAgent(clonedAgent, AgentField.All);
             Utilities.ClearCache();
-            return $"Agent {agent.Name} has been migrated!";
+
+            updateResult = $"Agent {agent.Name} has been migrated!";
+            _logger.LogInformation(updateResult);
+            return updateResult;
         }
         catch (Exception ex)
         {
-            return $"Failed to migrate agent {agent.Name} in file directory {filePath}.\r\nError: {ex.Message}";
+            updateResult = $"Failed to migrate agent {agent.Name} in file directory {filePath}.\r\nError: {ex.Message}";
+            _logger.LogError(updateResult);
+            return updateResult;
         }
     }