Separeted policies.

hyonholee · Nov 9, 2017 · 157f96b · 157f96b
1 parent 0b84a5e
commit 157f96b
Show file tree

Hide file tree

Showing 8 changed files with 101 additions and 57 deletions.
diff --git a/experiments/Azure.Experiments/Azure.Experiments/Compute/ComputePolicy.cs b/experiments/Azure.Experiments/Azure.Experiments/Compute/ComputePolicy.cs
@@ -19,23 +19,5 @@ public static ResourcePolicy<ResourceName, Info> Create<Operations, Info>(
                 .Create(getAsync, createOrUpdateAsync)
                 .Transform(getOperations)
                 .CreateResourcePolicy(i => i.Location, (i, location) => i.Location = location);
-
-        public static ResourcePolicy<ResourceName, VirtualMachine> VirtualMachine
-        { get; }
-            = Create(
-                client => client.VirtualMachines,
-                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
-                (operations, name, info)
-                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
-
-        public static ResourceConfig<ResourceName, VirtualMachine> CreateVirtualMachineConfig(
-            this ResourceConfig<string, ResourceGroup> resourceGroup,
-            string name,
-            ResourceConfig<ResourceName, NetworkInterface> networkInterface)
-            => resourceGroup.CreateResourceConfig(
-                VirtualMachine,
-                name,
-                new VirtualMachine(),
-                new [] { networkInterface });
     }
 }
diff --git a/experiments/Azure.Experiments/Azure.Experiments/Compute/VirtualMachinePolicy.cs b/experiments/Azure.Experiments/Azure.Experiments/Compute/VirtualMachinePolicy.cs
@@ -0,0 +1,28 @@
+using Microsoft.Azure.Experiments.ResourceManager;
+using Microsoft.Azure.Management.Compute;
+using Microsoft.Azure.Management.Compute.Models;
+using Microsoft.Azure.Management.Network.Models;
+using Microsoft.Azure.Management.ResourceManager.Models;
+
+namespace Microsoft.Azure.Experiments.Compute
+{
+    public static class VirtualMachinePolicy
+    {
+        public static ResourcePolicy<ResourceName, VirtualMachine> Policy { get; }
+            = ComputePolicy.Create(
+                client => client.VirtualMachines,
+                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
+                (operations, name, info)
+                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
+
+        public static ResourceConfig<ResourceName, VirtualMachine> CreateVirtualMachineConfig(
+            this ResourceConfig<string, ResourceGroup> resourceGroup,
+            string name,
+            ResourceConfig<ResourceName, NetworkInterface> networkInterface)
+            => resourceGroup.CreateResourceConfig(
+                Policy,
+                name,
+                new VirtualMachine(),
+                new[] { networkInterface });
+    }
+}
diff --git a/experiments/Azure.Experiments/Azure.Experiments/Network/NetworkPolicy.cs b/experiments/Azure.Experiments/Azure.Experiments/Network/NetworkPolicy.cs
@@ -19,42 +19,6 @@ public static ResourcePolicy<ResourceName, Info> Create<Operations, Info>(
                 .Transform(getOperations)
                 .CreateResourcePolicy(i => i.Location, (i, location) => i.Location = location);
 
-        public static ResourcePolicy<ResourceName, NetworkSecurityGroup> NetworkSecurityGroup { get; }
-            = Create(
-                client => client.NetworkSecurityGroups,
-                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
-                (operations, name, info)
-                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
-
-        public static ResourceConfig<ResourceName, NetworkSecurityGroup> CreateNetworkSecurityGroupConfig(
-            this ResourceConfig<string, ResourceGroup> resourceGroup,
-            string name)
-            => resourceGroup.CreateResourceConfig(NetworkSecurityGroup, name, new NetworkSecurityGroup());
-
-        public static ResourcePolicy<ResourceName, PublicIPAddress> PublicIPAddress { get; }
-            = Create(
-                client => client.PublicIPAddresses,
-                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
-                (operations, name, info)
-                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
-
-        public static ResourceConfig<ResourceName, PublicIPAddress> CreatePublicIPAddressConfig(
-            this ResourceConfig<string, ResourceGroup> resourceGroup,
-            string name)
-            => resourceGroup.CreateResourceConfig(PublicIPAddress, name, new PublicIPAddress());
-
-        public static ResourcePolicy<ResourceName, VirtualNetwork> VirtualNetwork { get; }
-            = Create(
-                client => client.VirtualNetworks,
-                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
-                (operations, name, info)
-                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
-
-        public static ResourceConfig<ResourceName, VirtualNetwork> CreateVirtualNetworkConfig(
-            this ResourceConfig<string, ResourceGroup> resourceGroup,
-            string name)
-            => resourceGroup.CreateResourceConfig(VirtualNetwork, name, new VirtualNetwork());
-
         public static ResourcePolicy<ResourceName, NetworkInterface> NetworkInterface { get; }
             = Create(
                 client => client.NetworkInterfaces,

diff --git a/experiments/Azure.Experiments/Azure.Experiments/Network/NetworkSecurityGroupPolicy.cs b/experiments/Azure.Experiments/Azure.Experiments/Network/NetworkSecurityGroupPolicy.cs
@@ -0,0 +1,22 @@
+using Microsoft.Azure.Experiments.ResourceManager;
+using Microsoft.Azure.Management.Network;
+using Microsoft.Azure.Management.Network.Models;
+using Microsoft.Azure.Management.ResourceManager.Models;
+
+namespace Microsoft.Azure.Experiments.Network
+{
+    public static class NetworkSecurityGroupPolicy
+    {
+        public static ResourcePolicy<ResourceName, NetworkSecurityGroup> Policy { get; }
+            = NetworkPolicy.Create(
+                client => client.NetworkSecurityGroups,
+                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
+                (operations, name, info)
+                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
+
+        public static ResourceConfig<ResourceName, NetworkSecurityGroup> CreateNetworkSecurityGroupConfig(
+            this ResourceConfig<string, ResourceGroup> resourceGroup,
+            string name)
+            => resourceGroup.CreateResourceConfig(Policy, name, new NetworkSecurityGroup());
+    }
+}
diff --git a/experiments/Azure.Experiments/Azure.Experiments/Network/PublicIPAddressPolicy.cs b/experiments/Azure.Experiments/Azure.Experiments/Network/PublicIPAddressPolicy.cs
@@ -0,0 +1,22 @@
+using Microsoft.Azure.Experiments.ResourceManager;
+using Microsoft.Azure.Management.Network;
+using Microsoft.Azure.Management.Network.Models;
+using Microsoft.Azure.Management.ResourceManager.Models;
+
+namespace Microsoft.Azure.Experiments.Network
+{
+    public static class PublicIPAddressPolicy
+    {
+        public static ResourcePolicy<ResourceName, PublicIPAddress> Policy { get; }
+            = NetworkPolicy.Create(
+                client => client.PublicIPAddresses,
+                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
+                (operations, name, info)
+                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
+
+        public static ResourceConfig<ResourceName, PublicIPAddress> CreatePublicIPAddressConfig(
+            this ResourceConfig<string, ResourceGroup> resourceGroup,
+            string name)
+            => resourceGroup.CreateResourceConfig(Policy, name, new PublicIPAddress());
+    }
+}
diff --git a/experiments/Azure.Experiments/Azure.Experiments/Network/VirtualNetworkPolicy.cs b/experiments/Azure.Experiments/Azure.Experiments/Network/VirtualNetworkPolicy.cs
@@ -0,0 +1,22 @@
+using Microsoft.Azure.Experiments.ResourceManager;
+using Microsoft.Azure.Management.Network;
+using Microsoft.Azure.Management.Network.Models;
+using Microsoft.Azure.Management.ResourceManager.Models;
+
+namespace Microsoft.Azure.Experiments.Network
+{
+    public static class VirtualNetworkPolicy
+    {
+        public static ResourcePolicy<ResourceName, VirtualNetwork> Policy { get; }
+            = NetworkPolicy.Create(
+                client => client.VirtualNetworks,
+                (operations, name) => operations.GetAsync(name.ResourceGroupName, name.Name),
+                (operations, name, info)
+                    => operations.CreateOrUpdateAsync(name.ResourceGroupName, name.Name, info));
+
+        public static ResourceConfig<ResourceName, VirtualNetwork> CreateVirtualNetworkConfig(
+            this ResourceConfig<string, ResourceGroup> resourceGroup,
+            string name)
+            => resourceGroup.CreateResourceConfig(Policy, name, new VirtualNetwork());
+    }
+}
diff --git a/experiments/Azure.Experiments/Azure.Experiments/ResourceConfig.cs b/experiments/Azure.Experiments/Azure.Experiments/ResourceConfig.cs
@@ -6,6 +6,10 @@ public interface IResourceConfig
     {
     }
 
+    public interface IResourceConfig<Info>
+    {
+    }
+
     public static class ResourceConfig
     {
         public static ResourceConfig<Name, Info> CreateResourceConfig<Name, Info>(

diff --git a/.../ResourceManager/ResourceManagerPolicy.cs → ...ts/ResourceManager/ResourceGroupPolicy.cs b/.../ResourceManager/ResourceManagerPolicy.cs → ...ts/ResourceManager/ResourceGroupPolicy.cs
@@ -5,9 +5,9 @@
 
 namespace Microsoft.Azure.Experiments.ResourceManager
 {
-    public static class ResourceManagerPolicy
+    public static class ResourceGroupPolicy
     {
-        public static ResourcePolicy<string, ResourceGroup> ResourceGroup { get; }
+        public static ResourcePolicy<string, ResourceGroup> Policy { get; }
             = OperationsPolicy
                 .Create<IResourceGroupsOperations, string, ResourceGroup>(
                     (operations, name) => operations.GetAsync(name),
@@ -16,7 +16,7 @@ public static class ResourceManagerPolicy
                 .CreateResourcePolicy(i => i.Location, (i, location) => i.Location = location);
 
         public static ResourceConfig<string, ResourceGroup> CreateResourceGroupConfig(string name)
-            => ResourceGroup.CreateResourceConfig(name, new ResourceGroup());
+            => Policy.CreateResourceConfig(name, new ResourceGroup());
 
         public static ResourceConfig<ResourceName, Info> CreateResourceConfig<Info>(
             this ResourceConfig<string, ResourceGroup> resourceGroup,