Rename IStateManager methods to get and set

rinrab · Jan 2, 2024 · 86d6714 · 86d6714
1 parent 5600c49
commit 86d6714
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 16 deletions.
diff --git a/AOVpnManager.Tests/StateManagerTests.cs b/AOVpnManager.Tests/StateManagerTests.cs
@@ -13,17 +13,17 @@ public void LastConnectionName()
 
             try
             {
-                stateManager.UpdateLastConnectionName("Contosa Vpn");
-                Assert.AreEqual("Contosa Vpn", stateManager.ReadLastConnectionName());
+                stateManager.SetLastConnectionName("Contosa Vpn");
+                Assert.AreEqual("Contosa Vpn", stateManager.GetLastConnectionName());
 
-                stateManager.UpdateLastConnectionName("Contosa Always On Vpn");
-                Assert.AreEqual("Contosa Always On Vpn", stateManager.ReadLastConnectionName());
+                stateManager.SetLastConnectionName("Contosa Always On Vpn");
+                Assert.AreEqual("Contosa Always On Vpn", stateManager.GetLastConnectionName());
 
-                stateManager.UpdateLastConnectionName(null);
-                Assert.AreEqual(null, stateManager.ReadLastConnectionName());
+                stateManager.SetLastConnectionName(null);
+                Assert.AreEqual(null, stateManager.GetLastConnectionName());
 
-                stateManager.UpdateLastConnectionName("Contosa Vpn v2");
-                Assert.AreEqual("Contosa Vpn v2", stateManager.ReadLastConnectionName());
+                stateManager.SetLastConnectionName("Contosa Vpn v2");
+                Assert.AreEqual("Contosa Vpn v2", stateManager.GetLastConnectionName());
             }
             finally
             {

diff --git a/AOVpnManager/IStateManager.cs b/AOVpnManager/IStateManager.cs
@@ -2,7 +2,7 @@
 {
     public interface IStateManager
     {
-        string ReadLastConnectionName();
-        void UpdateLastConnectionName(string connectionName);
+        string GetLastConnectionName();
+        void SetLastConnectionName(string connectionName);
     }
 }
diff --git a/AOVpnManager/Program.cs b/AOVpnManager/Program.cs
@@ -19,7 +19,7 @@ static int Main(string[] args)
             try
             {
                 GroupPolicySettings settings = policyProvider.ReadSettings();
-                string lastConnectionName = stateManager.ReadLastConnectionName();
+                string lastConnectionName = stateManager.GetLastConnectionName();
 
                 using (IVpnManager vpnManager = VpnManager.Create())
                 {
@@ -29,7 +29,7 @@ static int Main(string[] args)
                         {
                             vpnManager.DeleteVpnConnection(lastConnectionName);
                             logger.VpnConnectionDeleted(lastConnectionName);
-                            stateManager.UpdateLastConnectionName(null);
+                            stateManager.SetLastConnectionName(null);
                         }
                     }
                     else
@@ -38,7 +38,7 @@ static int Main(string[] args)
                         {
                             vpnManager.DeleteVpnConnection(lastConnectionName);
                             logger.VpnConnectionDeleted(lastConnectionName);
-                            stateManager.UpdateLastConnectionName(null);
+                            stateManager.SetLastConnectionName(null);
                         }
 
                         VpnConnectionInfo oldConnection = FindVpnConnection(vpnManager, settings.VpnConnectionName);
@@ -56,7 +56,7 @@ static int Main(string[] args)
                             logger.VpnConnectionUpdated(settings.VpnConnectionName);
                         }
 
-                        stateManager.UpdateLastConnectionName(settings.VpnConnectionName);
+                        stateManager.SetLastConnectionName(settings.VpnConnectionName);
                     }
                 }
             }

diff --git a/AOVpnManager/StateManager.cs b/AOVpnManager/StateManager.cs
@@ -15,15 +15,15 @@ public StateManager(RegistryKey root, string path)
             this.path = path;
         }
 
-        public string ReadLastConnectionName()
+        public string GetLastConnectionName()
         {
             using (RegistryKey key = root.OpenSubKey(path))
             {
                 return key?.GetValue<string>(LastConnectionName, null);
             }
         }
 
-        public void UpdateLastConnectionName(string connectionName)
+        public void SetLastConnectionName(string connectionName)
         {
             using (RegistryKey key = OpenOrCreateKey())
             {