Fix refresh argument arity (#1313)

John Luo · web-flow · commit 31009f6de1fe · 2021-06-03T10:05:57.000+12:00
diff --git a/src/dotnet-grpc/Commands/RefreshCommand.cs b/src/dotnet-grpc/Commands/RefreshCommand.cs
@@ -48,7 +48,7 @@ public static Command Create()
             {
                 Name = "references",
                 Description = CoreStrings.RefreshCommandArgumentDescription,
-                Arity = ArgumentArity.OneOrMore
+                Arity = ArgumentArity.ZeroOrMore
             });
             command.AddOption(CommonOptions.ProjectOption());
             command.AddOption(new Option(
@@ -79,7 +79,7 @@ public static Command Create()
 
         public async Task RefreshAsync(bool dryRun, string[] references)
         {
-            var refsToRefresh = references.Length == 0 ? Project.GetItems(ProtobufElement).Where(p => p.HasMetadata(SourceUrlElement)) : ResolveReferences(references);
+            var refsToRefresh = references == null || references.Length == 0 ? Project.GetItems(ProtobufElement).Where(p => p.HasMetadata(SourceUrlElement)) : ResolveReferences(references);
 
             foreach (var reference in refsToRefresh)
             {

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public static Command Create()`
`48`	`48`	`{`
`49`	`49`	`Name = "references",`
`50`	`50`	`Description = CoreStrings.RefreshCommandArgumentDescription,`
`51`		`- Arity = ArgumentArity.OneOrMore`
	`51`	`+ Arity = ArgumentArity.ZeroOrMore`
`52`	`52`	`});`
`53`	`53`	`command.AddOption(CommonOptions.ProjectOption());`
`54`	`54`	`command.AddOption(new Option(`
`@@ -79,7 +79,7 @@ public static Command Create()`
`79`	`79`
`80`	`80`	`public async Task RefreshAsync(bool dryRun, string[] references)`
`81`	`81`	`{`
`82`		`- var refsToRefresh = references.Length == 0 ? Project.GetItems(ProtobufElement).Where(p => p.HasMetadata(SourceUrlElement)) : ResolveReferences(references);`
	`82`	`+ var refsToRefresh = references == null \|\| references.Length == 0 ? Project.GetItems(ProtobufElement).Where(p => p.HasMetadata(SourceUrlElement)) : ResolveReferences(references);`
`83`	`83`
`84`	`84`	`foreach (var reference in refsToRefresh)`
`85`	`85`	`{`