Comments.

Author: sergey-shandar

src/ResourceManager/Common/Commands.Common.Strategies/Commands.Common.Strategies.csproj

@@ -79,10 +79,9 @@
     <Compile Include="NestedResourceConfigExtensions.cs" />
     <Compile Include="ResourceConfigExtensions.cs" />
     <Compile Include="StateExtensions.cs" />
-    <Compile Include="TargetDependencies.cs" />
     <Compile Include="NestedResourceConfig.cs" />
     <Compile Include="NestedResourceStrategy.cs" />
-    <Compile Include="TargetState.cs" />
+    <Compile Include="TargetStateExtensions.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="ResourceConfig.cs" />
     <Compile Include="ResourceStrategy.cs" />

src/ResourceManager/Common/Commands.Common.Strategies/IEntityStrategy.cs

@@ -14,6 +14,9 @@
 
 namespace Microsoft.Azure.Commands.Common.Strategies
 {
+    /// <summary>
+    /// Base interface for ResourceStrategy and NestedResourceStrategy classes.
+    /// </summary>
     public interface IEntityStrategy
     {
     }

src/ResourceManager/Common/Commands.Common.Strategies/IResourceConfig.cs

@@ -16,6 +16,9 @@
 
 namespace Microsoft.Azure.Commands.Common.Strategies
 {
+    /// <summary>
+    /// Base interface for ResourceConfig[].
+    /// </summary>
     public interface IResourceConfig : IEntityConfig
     {
         new IResourceStrategy Strategy { get; }

src/ResourceManager/Common/Commands.Common.Strategies/IResourceStrategy.cs

@@ -1,5 +1,8 @@
 namespace Microsoft.Azure.Commands.Common.Strategies
 {
+    /// <summary>
+    /// Base interface for ResourceStrategy[].
+    /// </summary>
     public interface IResourceStrategy : IEntityStrategy
     {
         string Type { get; }

src/ResourceManager/Common/Commands.Common.Strategies/NestedResourceConfig.cs

@@ -18,6 +18,11 @@
 
 namespace Microsoft.Azure.Commands.Common.Strategies
 {
+    /// <summary>
+    /// Nested resource configuration. Fro example, Subnet.
+    /// </summary>
+    /// <typeparam name="TModel"></typeparam>
+    /// <typeparam name="TParenModel"></typeparam>
     public sealed class NestedResourceConfig<TModel, TParenModel> : IEntityConfig<TModel>
         where TModel : class
         where TParenModel : class
@@ -26,6 +31,9 @@ public sealed class NestedResourceConfig<TModel, TParenModel> : IEntityConfig<TM
 
         public string Name { get; }
 
+        /// <summary>
+        /// Parent. For example, VirtualNetwork is a parent of Subnet.
+        /// </summary>
         public IEntityConfig<TParenModel> Parent { get; }
 
         public Func<TModel> CreateModel { get; }

src/ResourceManager/Common/Commands.Common.Strategies/ProgressMap.cs

@@ -31,8 +31,8 @@ public ProgressMap(Dictionary<IResourceConfig, Tuple<TimeSlot, int>> map, int du
         }
 
         /// <summary>
-        /// Returns a value [0..1] which is used to increment a progress bar when the resource is 
-        /// created.
+        /// Returns a value of [0..1] range which is used to increment a progress bar when the 
+        /// resource is created.
         /// </summary>
         /// <param name="config">a resource configuration.</param>
         /// <returns></returns>

src/ResourceManager/Common/Commands.Common.Strategies/ResourceConfig.cs

@@ -18,6 +18,11 @@
 
 namespace Microsoft.Azure.Commands.Common.Strategies
 {
+    /// <summary>
+    /// Resource configuration. It contains information to create a resource,
+    /// including name, resource group name, dependencies, model creation function, etc.
+    /// </summary>
+    /// <typeparam name="TModel"></typeparam>
     public sealed class ResourceConfig<TModel> : IEntityConfig<TModel>, IResourceConfig
         where TModel : class
     {

src/ResourceManager/Common/Commands.Common.Strategies/State.cs

@@ -17,6 +17,9 @@
 
 namespace Microsoft.Azure.Commands.Common.Strategies
 {
+    /// <summary>
+    /// Azure State. It contains information (models) of Azure resources.
+    /// </summary>
     sealed class State : IState
     {
         readonly ConcurrentDictionary<string, object> _Map

src/ResourceManager/Common/Commands.Common.Strategies/TargetDependencies.cs

@@ -12,10 +12,17 @@
 // limitations under the License.
 // ----------------------------------------------------------------------------------
 
+using System.Collections.Generic;
+using System.Linq;
+
 namespace Microsoft.Azure.Commands.Common.Strategies
 {
-    public static class TargetState
+    public static class TargetStateExtensions
     {
+        public static IEnumerable<IResourceConfig> GetTargetDependencies(
+            this IResourceConfig config, IState target)
+            => config.GetResourceDependencies().Where(target.Contains);
+
         public static IState GetTargetState<TModel>(
             this ResourceConfig<TModel> config,
             IState current,

src/ResourceManager/Common/Commands.Common.Strategies/TargetState.cs renamed to src/ResourceManager/Common/Commands.Common.Strategies/TargetStateExtensions.cs

@@ -16,7 +16,7 @@ namespace Microsoft.Azure.Commands.Common.Strategies
 {
     /// <summary>
     /// TimeSlot is a node of a singly linked list of TimeSlots.
-    /// The last node of the list is always an empty time slot
+    /// The last node of the list is always an empty time slot:
     /// - Duration = 0.
     /// - Next = null.
     /// - TaskCount = 0.