cleanup

Author: davidegiacometti

src/modules/Hosts/Hosts.Tests/HostsServiceTest.cs

@@ -282,7 +282,7 @@ public async Task Save_Hidden_Hosts()
         }
 
         [TestMethod]
-        public async Task Hosts_Backup_Not_Done()
+        public async Task Hosts_Backup_Not_Executed()
         {
             var content =
 @"10.1.1.1 host host.local # comment
@@ -307,7 +307,7 @@ 10.1.1.2 host2 host2.local # another comment
         }
 
         [TestMethod]
-        public async Task Hosts_Backup_Done()
+        public async Task Hosts_Backup_Executed_Once()
         {
             var content =
 @"10.1.1.1 host host.local # comment
@@ -326,6 +326,7 @@ 10.1.1.2 host2 host2.local # another comment
             var entries = data.Entries.ToList();
             entries.Add(new Entry(0, "10.1.1.30", "host30 host30.local", "new entry", false));
             await service.WriteAsync(data.AdditionalLines, data.Entries);
+            await service.WriteAsync(data.AdditionalLines, data.Entries);
 
             Assert.AreEqual(1, fileSystem.Directory.GetFiles(BackupPath).Length);
             var backupContent = fileSystem.File.ReadAllText(fileSystem.Directory.GetFiles(BackupPath)[0]);

src/modules/Hosts/HostsUILib/Helpers/BackupManager.cs

@@ -47,10 +47,10 @@ public void DeleteBackups()
             {
                 case HostsDeleteBackupMode.Count:
                     DeleteBackupsByCount(_userSettings.DeleteBackupsCount);
-                    return;
+                    break;
                 case HostsDeleteBackupMode.Age:
                     DeleteBackupsByAge(_userSettings.DeleteBackupsDays, _userSettings.DeleteBackupsCount);
-                    return;
+                    break;
             }
         }
 

src/modules/Hosts/HostsUILib/Helpers/IHostsService.cs

@@ -12,7 +12,7 @@ namespace HostsUILib.Helpers
 {
     public interface IHostsService
     {
-        static string HostsFilePath { get; }
+        string HostsFilePath { get; }
 
         event EventHandler FileChanged;