Browse Source

Change Get-Package cmdlet parameters to match those in NuGet 1.2

pull/15/head
Matt Ward 15 years ago
parent
commit
51bdbe6243
  1. 13
      src/AddIns/Misc/PackageManagement/Cmdlets/Project/Src/GetPackageCmdlet.cs
  2. 14
      src/AddIns/Misc/PackageManagement/Cmdlets/Test/Src/GetPackageCmdletTests.cs

13
src/AddIns/Misc/PackageManagement/Cmdlets/Project/Src/GetPackageCmdlet.cs

@ -17,7 +17,7 @@ namespace ICSharpCode.PackageManagement.Cmdlets
public class GetPackageCmdlet : PackageManagementCmdlet public class GetPackageCmdlet : PackageManagementCmdlet
{ {
int? skip; int? skip;
int? take; int? first;
public GetPackageCmdlet() public GetPackageCmdlet()
: this( : this(
@ -59,11 +59,12 @@ namespace ICSharpCode.PackageManagement.Cmdlets
set { skip = value; } set { skip = value; }
} }
[Alias("Take")]
[Parameter] [Parameter]
[ValidateRange(0, Int32.MaxValue)] [ValidateRange(0, Int32.MaxValue)]
public int Take { public int First {
get { return take.GetValueOrDefault(); } get { return first.GetValueOrDefault(); }
set { take = value; } set { first = value; }
} }
protected override void ProcessRecord() protected override void ProcessRecord()
@ -118,8 +119,8 @@ namespace ICSharpCode.PackageManagement.Cmdlets
if (skip.HasValue) { if (skip.HasValue) {
packages = packages.Skip(skip.Value); packages = packages.Skip(skip.Value);
} }
if (take.HasValue) { if (first.HasValue) {
packages = packages.Take(take.Value); packages = packages.Take(first.Value);
} }
return packages; return packages;
} }

14
src/AddIns/Misc/PackageManagement/Cmdlets/Test/Src/GetPackageCmdletTests.cs

@ -88,9 +88,9 @@ namespace PackageManagement.Cmdlets.Tests
cmdlet.Skip = skip; cmdlet.Skip = skip;
} }
void SetTakeParameter(int take) void SetFirstParameter(int first)
{ {
cmdlet.Take = take; cmdlet.First = first;
} }
[Test] [Test]
@ -501,7 +501,7 @@ namespace PackageManagement.Cmdlets.Tests
} }
[Test] [Test]
public void ProcessRecord_ListAvailableAndTakeTwo_ReturnsFirstTwoPackages() public void ProcessRecord_ListAvailableAndFirstTwo_ReturnsFirstTwoPackages()
{ {
CreateCmdlet(); CreateCmdlet();
var repository = fakePackageManagementService.FakePackageRepositoryToReturnFromCreatePackageRepository; var repository = fakePackageManagementService.FakePackageRepositoryToReturnFromCreatePackageRepository;
@ -510,7 +510,7 @@ namespace PackageManagement.Cmdlets.Tests
repository.AddOneFakePackage("C"); repository.AddOneFakePackage("C");
EnableListAvailableParameter(); EnableListAvailableParameter();
SetTakeParameter(2); SetFirstParameter(2);
RunCmdlet(); RunCmdlet();
var actualPackages = fakeCommandRuntime.ObjectsPassedToWriteObject; var actualPackages = fakeCommandRuntime.ObjectsPassedToWriteObject;
@ -523,12 +523,12 @@ namespace PackageManagement.Cmdlets.Tests
} }
[Test] [Test]
public void Take_NewGetPackageCmdletInstance_ReturnsZero() public void First_NewGetPackageCmdletInstance_ReturnsZero()
{ {
CreateCmdlet(); CreateCmdlet();
int take = cmdlet.Take; int first = cmdlet.First;
Assert.AreEqual(0, take); Assert.AreEqual(0, first);
} }
} }
} }

Loading…
Cancel
Save