IReportProgress

Author: sergey-shandar

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

@@ -69,6 +69,7 @@
   <ItemGroup>
     <Compile Include="Compute\Image.cs" />
     <Compile Include="Compute\Images.cs" />
+    <Compile Include="IReportProgress.cs" />
     <Compile Include="IResourceConfig.cs" />
     <Compile Include="IResourceConfigVisitor.cs" />
     <Compile Include="IShouldProcess.cs" />

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

@@ -0,0 +1,8 @@
+namespace Microsoft.Azure.Commands.Common.Strategies
+{
+    public interface IProgressReport
+    {
+        void Report<TModel>(ResourceConfig<TModel> config, double progress)
+                where TModel : class;
+    }
+}

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

@@ -12,11 +12,15 @@ public static async Task<IState> UpdateStateAsync<TModel>(
             IClient client,
             IState target,
             CancellationToken cancellationToken,
-            IShouldProcess shouldProcess)
+            IShouldProcess shouldProcess,
+            IProgressReport progressReport)
             where TModel : class
         {
             var context = new Context(
-                new StateOperationContext(client, cancellationToken), target, shouldProcess);
+                new StateOperationContext(client, cancellationToken),
+                target,
+                shouldProcess,
+                progressReport);
             await context.UpdateStateAsync(config);
             return context.Result;
         }
@@ -31,12 +35,18 @@ sealed class Context
 
             readonly IShouldProcess _ShouldProcess;
 
+            readonly IProgressReport _ProgressReport;
+
             public Context(
-                StateOperationContext operationContext, IState target, IShouldProcess shouldProcess)
+                StateOperationContext operationContext,
+                IState target,
+                IShouldProcess shouldProcess,
+                IProgressReport progressReport)
             {
                 _OperationContext = operationContext;
                 _Target = target;
                 _ShouldProcess = shouldProcess;
+                _ProgressReport = progressReport;
             }
 
             public async Task UpdateStateAsync<TModel>(ResourceConfig<TModel> config)
@@ -57,10 +67,13 @@ await _OperationContext.GetOrAdd(
                             // call the CreateOrUpdateAsync function for the resource.
                             if (_ShouldProcess.ShouldCreate(config, model))
                             {
-                                return await config.CreateOrUpdateAsync(
+                                _ProgressReport.Report(config, 0.0);
+                                var result = await config.CreateOrUpdateAsync(
                                     _OperationContext.Client,
                                     model,
                                     _OperationContext.CancellationToken);
+                                _ProgressReport.Report(config, 1.0);
+                                return result;
                             }
                             else
                             {

src/ResourceManager/Compute/Commands.Compute/VirtualMachine/Operation/NewAzureVMCommand.cs

@@ -195,7 +195,28 @@ public ShouldProcessType(Cmdlet cmdlet)
 
             public bool ShouldCreate<TModel>(ResourceConfig<TModel> config, TModel model)
                 where TModel : class
-                => _Cmdlet.ShouldProcess(config.Name + " " + config.Strategy.Type, VerbsCommon.New);
+                => _Cmdlet.ShouldProcess(config.Name, VerbsCommon.New + " " + config.Strategy.Type);
+        }
+
+        private sealed class ProgressReportType : IProgressReport
+        {
+            readonly Cmdlet _Cmdlet;
+
+            public ProgressReportType(Cmdlet cmdlet)
+            {
+                _Cmdlet = cmdlet;
+            }
+
+            public void Report<TModel>(ResourceConfig<TModel> config, double progress) 
+                where TModel : class
+            {
+
+            }
+            /*
+                => _Cmdlet.WriteVerbose(
+                    progress == 0 ? "Creating " + config.Name + " " + config.Strategy.Type + "..."
+                    : config.Name + " " + config.Strategy.Type + " is created.");
+                    */
         }
 
         public void StrategyExecuteCmdlet()
@@ -256,7 +277,11 @@ public void StrategyExecuteCmdlet()
 
             var result = virtualMachine
                 .UpdateStateAsync(
-                    client, target, new CancellationToken(), new ShouldProcessType(this))
+                    client,
+                    target,
+                    new CancellationToken(),
+                    new ShouldProcessType(this),
+                    new ProgressReportType(this))
                 .GetAwaiter()
                 .GetResult();
             WriteObject(result);