Merge pull request Azure#143 from Azure/nowait

Do not run AsJob with NoWait giving NoWait priority

Author: niander

src/Common/Extensions/CmdletExtensions.cs

@@ -121,7 +121,8 @@ public static void ExecuteSynchronouslyOrAsJob<T>(this T cmdlet, Action<T> execu
                 throw new ArgumentNullException(nameof(executor));
             }
 
-            if (cmdlet.AsJobPresent())
+            // Since right now NoWait and AsJob are not in different parameter sets this check is necessary
+            if (cmdlet.IsBound("AsJob") && !cmdlet.IsBound("NoWait"))
             {
                 cmdlet.WriteObject(cmdlet.ExecuteAsJob(cmdlet.ImplementationBackgroundJobDescription, executor));
             }
@@ -212,7 +213,7 @@ public static void SafeCopyParameterSet<T>(this T source, T target) where T : Az
         /// <returns>true if the parameter was provided by the user, otherwise false</returns>
         public static bool IsBound(this PSCmdlet cmdlet, string parameterName) 
         {
-            return cmdlet.MyInvocation.BoundParameters.ContainsKey(parameterName);
+            return cmdlet.MyInvocation?.BoundParameters.ContainsKey(parameterName) ?? false;
         }
 
         public static string AsAbsoluteLocation(this string realtivePath)
@@ -358,7 +359,7 @@ public static List<T> ExecuteScript<T>(this PSCmdlet cmdlet, string contents)
         public static bool IsParameterBound<TPSCmdlet, TProp>(this TPSCmdlet cmdlet, Expression<Func<TPSCmdlet, TProp>> propertySelector) where TPSCmdlet : PSCmdlet
         {
             var propName = ((MemberExpression)propertySelector.Body).Member.Name;
-            return cmdlet.MyInvocation.BoundParameters.ContainsKey(propName);
+            return cmdlet.IsBound(propName);
         }
 
         #region PowerShell Commands