Rename to add source

Author: William Li

src/dotnet/commands/dotnet-tool/install/ToolInstallCommand.cs

@@ -55,7 +55,7 @@ public ToolInstallCommand(
             _packageVersion = appliedCommand.ValueOrDefault<string>("version");
             _configFilePath = appliedCommand.ValueOrDefault<string>("configfile");
             _framework = appliedCommand.ValueOrDefault<string>("framework");
-            _source = appliedCommand.ValueOrDefault<string[]>("source-feed");
+            _source = appliedCommand.ValueOrDefault<string[]>("add-source");
             _global = appliedCommand.ValueOrDefault<bool>("global");
             _verbosity = appliedCommand.SingleArgumentOrDefault("verbosity");
             _toolPath = appliedCommand.SingleArgumentOrDefault("tool-path");

src/dotnet/commands/dotnet-tool/install/ToolInstallCommandParser.cs

@@ -32,7 +32,7 @@ public static Command ToolInstall()
                     LocalizableStrings.ConfigFileOptionDescription,
                     Accept.ExactlyOneArgument()),
                 Create.Option(
-                    "--source-feed",
+                    "--add-source",
                     LocalizableStrings.SourceFeedOptionDescription,
                     Accept.OneOrMoreArguments()
                         .With(name: LocalizableStrings.SourceFeedOptionName)),

src/dotnet/commands/dotnet-tool/update/ToolUpdateCommand.cs

@@ -52,7 +52,7 @@ public ToolUpdateCommand(AppliedOption appliedCommand,
             _packageId = new PackageId(appliedCommand.Arguments.Single());
             _configFilePath = appliedCommand.ValueOrDefault<string>("configfile");
             _framework = appliedCommand.ValueOrDefault<string>("framework");
-            _additionalFeeds = appliedCommand.ValueOrDefault<string[]>("source-feed");
+            _additionalFeeds = appliedCommand.ValueOrDefault<string[]>("add-source");
             _global = appliedCommand.ValueOrDefault<bool>("global");
             _verbosity = appliedCommand.SingleArgumentOrDefault("verbosity");
             _toolPath = appliedCommand.SingleArgumentOrDefault("tool-path");

src/dotnet/commands/dotnet-tool/update/ToolUpdateCommandParser.cs

@@ -28,7 +28,7 @@ public static Command ToolUpdate()
                     LocalizableStrings.ConfigFileOptionDescription,
                     Accept.ExactlyOneArgument()),
                 Create.Option(
-                    "--source-feed",
+                    "--add-source",
                     LocalizableStrings.SourceFeedOptionDescription,
                     Accept.OneOrMoreArguments()
                         .With(name: LocalizableStrings.SourceFeedOptionName)),

test/dotnet.Tests/CommandTests/ToolInstallCommandTests.cs

@@ -85,10 +85,10 @@ public void WhenRunWithPackageIdItShouldCreateValidShim()
         public void WhenRunWithPackageIdWithSourceItShouldCreateValidShim()
         {
             const string sourcePath = "http://mysouce.com";
-            ParseResult result = Parser.Instance.Parse($"dotnet tool install -g {PackageId} --source-feed {sourcePath}");
+            ParseResult result = Parser.Instance.Parse($"dotnet tool install -g {PackageId} --add-source {sourcePath}");
             AppliedOption appliedCommand = result["dotnet"]["tool"]["install"];
             ParseResult parseResult =
-                Parser.Instance.ParseFrom("dotnet tool", new[] { "install", "-g", PackageId, "--source-feed", sourcePath });
+                Parser.Instance.ParseFrom("dotnet tool", new[] { "install", "-g", PackageId, "--add-source", sourcePath });
 
 
             var toolToolPackageInstaller = CreateToolPackageInstaller(

test/dotnet.Tests/ParserTests/InstallToolParserTests.cs

@@ -55,10 +55,10 @@ public void InstallToolParserCanParseSourceOption()
             const string expectedSourceValue = "TestSourceValue";
 
             var result =
-                Parser.Instance.Parse($"dotnet tool install -g --source-feed {expectedSourceValue} console.test.app");
+                Parser.Instance.Parse($"dotnet tool install -g --add-source {expectedSourceValue} console.test.app");
 
             var appliedOptions = result["dotnet"]["tool"]["install"];
-            appliedOptions.ValueOrDefault<string[]>("source-feed").First().Should().Be(expectedSourceValue);
+            appliedOptions.ValueOrDefault<string[]>("add-source").First().Should().Be(expectedSourceValue);
         }
 
         [Fact]
@@ -70,13 +70,13 @@ public void InstallToolParserCanParseMultipleSourceOption()
             var result =
                 Parser.Instance.Parse(
                     $"dotnet tool install -g " +
-                    $"--source-feed {expectedSourceValue1} " +
-                    $"--source-feed {expectedSourceValue2} console.test.app");
+                    $"--add-source {expectedSourceValue1} " +
+                    $"--add-source {expectedSourceValue2} console.test.app");
 
             var appliedOptions = result["dotnet"]["tool"]["install"];
 
-            appliedOptions.ValueOrDefault<string[]>("source-feed")[0].Should().Be(expectedSourceValue1);
-            appliedOptions.ValueOrDefault<string[]>("source-feed")[1].Should().Be(expectedSourceValue2);
+            appliedOptions.ValueOrDefault<string[]>("add-source")[0].Should().Be(expectedSourceValue1);
+            appliedOptions.ValueOrDefault<string[]>("add-source")[1].Should().Be(expectedSourceValue2);
         }
 
         [Fact]

test/dotnet.Tests/ParserTests/UpdateToolParserTests.cs

@@ -62,10 +62,10 @@ public void UpdateToolParserCanParseSourceOption()
             const string expectedSourceValue = "TestSourceValue";
 
             var result =
-                Parser.Instance.Parse($"dotnet tool update -g --source-feed {expectedSourceValue} console.test.app");
+                Parser.Instance.Parse($"dotnet tool update -g --add-source {expectedSourceValue} console.test.app");
 
             var appliedOptions = result["dotnet"]["tool"]["update"];
-            appliedOptions.ValueOrDefault<string[]>("source-feed").First().Should().Be(expectedSourceValue);
+            appliedOptions.ValueOrDefault<string[]>("add-source").First().Should().Be(expectedSourceValue);
         }
 
         [Fact]
@@ -77,13 +77,13 @@ public void UpdateToolParserCanParseMultipleSourceOption()
             var result =
                 Parser.Instance.Parse(
                     $"dotnet tool update -g " +
-                    $"--source-feed {expectedSourceValue1} " +
-                    $"--source-feed {expectedSourceValue2} console.test.app");
+                    $"--add-source {expectedSourceValue1} " +
+                    $"--add-source {expectedSourceValue2} console.test.app");
 
             var appliedOptions = result["dotnet"]["tool"]["update"];
 
-            appliedOptions.ValueOrDefault<string[]>("source-feed")[0].Should().Be(expectedSourceValue1);
-            appliedOptions.ValueOrDefault<string[]>("source-feed")[1].Should().Be(expectedSourceValue2);
+            appliedOptions.ValueOrDefault<string[]>("add-source")[0].Should().Be(expectedSourceValue1);
+            appliedOptions.ValueOrDefault<string[]>("add-source")[1].Should().Be(expectedSourceValue2);
         }
 
         [Fact]