Refactoring.

Author: AlenPelin

src/SIM.Instances/InstanceHelper.cs

@@ -2,13 +2,38 @@
 {
   using System;
   using System.IO;
+  using System.Linq;
   using System.Net;
+  using System.Text.RegularExpressions;
   using Sitecore.Diagnostics.Base;
+  using Sitecore.Diagnostics.Base.Annotations;
 
   public static class InstanceHelper
   {
+    [NotNull]
+    private static readonly Regex LogGroupRegex = new Regex(@"(.+)(\.\d\d\d\d\d\d\d\d)(\.\d\d\d\d\d\d)?\.txt", RegexOptions.Compiled);
+
     #region Public methods
 
+    public static string[] GetLogGroups(string[] files)
+    {
+      var groups = files
+        .Where(x => !string.IsNullOrEmpty(x))
+        .Select(x => new
+        {
+          FilePath = x,
+          Position = x.LastIndexOf('\\')
+        })
+        .Select(x => x.Position < 0 ? x.FilePath : x.FilePath.Substring(x.Position + 1))
+        .Select(x => LogGroupRegex.Match(x))
+        .Where(x => x.Success)
+        .Select(x => x.Groups[1].Value)
+        .Distinct()
+        .ToArray();
+
+      return groups;
+    }
+
     public static bool IsInstanceResponsive(Instance instance, string reason = null)
     {
       try

src/SIM.Tests/Tool/Base/InstanceHelperExTests.cs

@@ -2,6 +2,7 @@
 {
   using System.Linq;
   using Microsoft.VisualStudio.TestTools.UnitTesting;
+  using SIM.Instances;
   using SIM.Tool.Base;
 
   [TestClass]
@@ -24,7 +25,7 @@ public void GetLogGroupsTest()
         "WebDAV.log.20140905.135957.txt", 
         "WebDAV.log.20140905.txt"
       };
-      var results = InstanceHelperEx.GetLogGroups(files).OrderBy(x => x).Select(x => x.ToLower()).ToArray();
+      var results = InstanceHelper.GetLogGroups(files).OrderBy(x => x).Select(x => x.ToLower()).ToArray();
       var expected = new[]
       {
         "crawling.log", "log", "search.log", "webdav.log"

src/SIM.Tool.Base/EnvironmentHelper.cs

@@ -5,11 +5,11 @@
   using System.Data.SqlClient;
   using System.Linq;
   using System.ServiceProcess;
-  using SIM.Adapters.SqlServer;
-  using SIM.Tool.Base.Profiles;
   using Sitecore.Diagnostics.Base;
   using Sitecore.Diagnostics.Base.Annotations;
   using Sitecore.Diagnostics.Logging;
+  using SIM.Adapters.SqlServer;
+  using SIM.Tool.Base.Profiles;
 
   public class EnvironmentHelper
   {
@@ -47,7 +47,7 @@ public static bool CheckSqlServer()
       {
         using (new ProfileSection("Check SQL Server"))
         {
-          Profile profile = ProfileManager.Profile;
+          var profile = ProfileManager.Profile;
           Assert.IsNotNull(profile, "Profile is unavailable");
 
           var ds = new SqlConnectionStringBuilder(profile.ConnectionString).DataSource;

src/SIM.Tool.Base/InstanceHelperEx.cs

@@ -13,13 +13,6 @@ namespace SIM.Tool.Base
 
   public static class InstanceHelperEx
   {
-    #region Fields
-
-    [NotNull]
-    private static readonly Regex LogGroupRegex = new Regex(@"(.+)(\.\d\d\d\d\d\d\d\d)(\.\d\d\d\d\d\d)?\.txt", RegexOptions.Compiled);
-
-    #endregion
-
     #region Public methods
 
     public static void BrowseInstance([NotNull] Instance instance, [NotNull] Window owner, [NotNull] string virtualPath, bool isFrontEnd, [CanBeNull] string browser = null, [CanBeNull] string[] parameters = null)
@@ -41,24 +34,6 @@ public static void BrowseInstance([NotNull] Instance instance, [NotNull] Window
       }
     }
 
-    public static string[] GetLogGroups(string[] files)
-    {
-      var groups = files
-        .Where(x => !string.IsNullOrEmpty(x))
-        .Select(x => new
-        {
-          FilePath = x, 
-          Position = x.LastIndexOf('\\')
-        })
-        .Select(x => x.Position < 0 ? x.FilePath : x.FilePath.Substring(x.Position + 1))
-        .Select(x => LogGroupRegex.Match(x))
-        .Where(x => x.Success)
-        .Select(x => x.Groups[1].Value)
-        .Distinct()
-        .ToArray();
-      return groups;
-    }
-
     public static void OpenCurrentLogFile([NotNull] Instance instance, [NotNull] Window owner, [CanBeNull] string logFileType = null)
     {
       Assert.ArgumentNotNull(instance, "instance");
@@ -260,7 +235,7 @@ private static string GetLogFileTypes(Window owner, string logsFolderPath)
       const string Pattern = "*" + Suffix;
       var files = FileSystem.FileSystem.Local.Directory.GetFiles(logsFolderPath, Pattern);
 
-      var groups = GetLogGroups(files);
+      var groups = InstanceHelper.GetLogGroups(files);
 
       if (groups.Any())
       {

src/SIM.Tool.Base/SIM.Tool.Base.csproj

@@ -100,11 +100,11 @@
     <Reference Include="WindowsBase" />
   </ItemGroup>
   <ItemGroup>
-    <Compile Include="AuthenticationHelper.cs" />
+    <Compile Include="EnvironmentHelper.cs" />
     <Compile Include="WinAppSettings.cs" />
+    <Compile Include="AuthenticationHelper.cs" />
     <Compile Include="Converters\IsNotNullOrEmptyAndDirectoryExists.cs" />
     <Compile Include="Converters\Product.cs" />
-    <Compile Include="EnvironmentHelper.cs" />
     <Compile Include="InstanceHelperEx.cs" />
     <Compile Include="LicenseUpdater.cs" />
     <Compile Include="Plugins\IMainWindowButton.cs" />

src/SIM.Tool.Windows/MainWindowComponents/DatabaseManagerButton.cs

@@ -6,6 +6,7 @@
   using SIM.Tool.Base.Plugins;
   using SIM.Tool.Windows.Dialogs;
   using Sitecore.Diagnostics.Base.Annotations;
+  using SIM.Core;
 
   [UsedImplicitly]
   public class DatabaseManagerButton : IMainWindowButton

src/SIM.Tool.Windows/MainWindowComponents/InstallInstanceButton.cs

@@ -11,6 +11,7 @@
   using SIM.Tool.Wizards;
   using Sitecore.Diagnostics.Base;
   using Sitecore.Diagnostics.Base.Annotations;
+  using SIM.Core;
 
   [UsedImplicitly]
   public class InstallInstanceButton : IMainWindowButton

src/SIM.Tool.Windows/MainWindowComponents/OpenToolboxButton.cs

@@ -9,6 +9,7 @@
   using SIM.Tool.Base;
   using SIM.Tool.Base.Plugins;
   using Sitecore.Diagnostics.Base.Annotations;
+  using SIM.Core;
 
   [UsedImplicitly]
   public class OpenToolboxButton : IMainWindowButton

src/SIM.Tool.Windows/MainWindowComponents/OpenVisualStudioButton.cs

@@ -11,6 +11,7 @@
   using SIM.Tool.Base.Plugins;
   using Sitecore.Diagnostics.Base;
   using Sitecore.Diagnostics.Base.Annotations;
+  using SIM.Core;
 
   [UsedImplicitly]
   public class OpenVisualStudioButton : IMainWindowButton

src/SIM.Tool.Windows/UserControls/Install/Modules/ConfigurationPackages.xaml.cs

@@ -15,6 +15,7 @@
   using SIM.Tool.Base.Wizards;
   using Sitecore.Diagnostics.Base;
   using Sitecore.Diagnostics.Base.Annotations;
+  using SIM.Core;
 
   public partial class ConfigurationPackages : IWizardStep, ICustomButton, IFlowControl
   {