Browse Source

Hide disabled NuGet package sources in manage packages dialog.

pull/28/head
Matt Ward 13 years ago
parent
commit
1dfa232205
  1. 2
      src/AddIns/Misc/PackageManagement/Project/Src/PackagesViewModel.cs
  2. 26
      src/AddIns/Misc/PackageManagement/Test/Src/AvailablePackagesViewModelTests.cs

2
src/AddIns/Misc/PackageManagement/Project/Src/PackagesViewModel.cs

@ -342,7 +342,7 @@ namespace ICSharpCode.PackageManagement @@ -342,7 +342,7 @@ namespace ICSharpCode.PackageManagement
public IEnumerable<PackageSource> PackageSources {
get {
foreach (PackageSource packageSource in registeredPackageRepositories.PackageSources) {
foreach (PackageSource packageSource in registeredPackageRepositories.PackageSources.GetEnabledPackageSources()) {
yield return packageSource;
}
if (registeredPackageRepositories.PackageSources.HasMultiplePackageSources) {

26
src/AddIns/Misc/PackageManagement/Test/Src/AvailablePackagesViewModelTests.cs

@ -76,6 +76,17 @@ namespace PackageManagement.Tests @@ -76,6 +76,17 @@ namespace PackageManagement.Tests
registeredPackageRepositories.AddPackageSources(sources);
}
PackageSource AddTwoPackageSourcesToRegisteredSourcesWithFirstOneDisabled()
{
var expectedPackageSources = new PackageSource[] {
new PackageSource("http://first.com", "First") { IsEnabled = false },
new PackageSource("http://second.com", "Second") { IsEnabled = true }
};
AddPackageSourcesToRegisteredSources(expectedPackageSources);
registeredPackageRepositories.HasMultiplePackageSources = true;
return expectedPackageSources[0];
}
void CreateNewActiveRepositoryWithDifferentPackages()
{
var package = new FakePackage("NewRepositoryPackageId");
@ -429,5 +440,20 @@ namespace PackageManagement.Tests @@ -429,5 +440,20 @@ namespace PackageManagement.Tests
};
PackageCollectionAssert.AreEqual(expectedPackages, allPackages);
}
[Test]
public void PackageSources_TwoPackageSourcesButFirstIsDisabled_DoesNotReturnDisabledPackageSource()
{
CreateRegisteredPackageRepositories();
AddTwoPackageSourcesToRegisteredSourcesWithFirstOneDisabled();
CreateViewModel(registeredPackageRepositories);
IEnumerable<PackageSource> packageSources = viewModel.PackageSources;
bool containsDisabledPackageSource = packageSources.Contains(registeredPackageRepositories.PackageSources[0]);
bool containsEnabledPackageSource = packageSources.Contains(registeredPackageRepositories.PackageSources[1]);
Assert.IsFalse(containsDisabledPackageSource);
Assert.IsTrue(containsEnabledPackageSource);
}
}
}

Loading…
Cancel
Save