Fixing test logging consumption to avoid working against the live underlying collections

Author: fabiocav

test/WebJobs.Script.Tests.Integration/TestFunctionHost.cs

@@ -110,7 +110,7 @@ public void SetNugetPackageSources(params string[] sources)
             }
         }
 
-        public IEnumerable<LogMessage> GetLogMessages() => _loggerProvider.GetAllLogMessages();
+        public IList<LogMessage> GetLogMessages() => _loggerProvider.GetAllLogMessages();
 
         public IEnumerable<LogMessage> GetLogMessages(string category) => GetLogMessages().Where(p => p.Category == category);
 

test/WebJobs.Script.Tests.Shared/TestLoggerProvider.cs

@@ -33,7 +33,7 @@ public ILogger CreateLogger(string categoryName)
             return logger;
         }
 
-        public IEnumerable<LogMessage> GetAllLogMessages() => CreatedLoggers.SelectMany(l => l.GetLogMessages()).OrderBy(p => p.Timestamp);
+        public IList<LogMessage> GetAllLogMessages() => CreatedLoggers.SelectMany(l => l.GetLogMessages()).OrderBy(p => p.Timestamp).ToList();
 
         public void ClearAllLogMessages()
         {