Browse Source

PackageManagement.Test: Ignore PropertyChanged

pull/374/head
Dragan 12 years ago
parent
commit
5e6e8f22fc
  1. 15
      src/AddIns/Misc/PackageManagement/Test/Src/PackageViewModelTests.cs
  2. 2
      src/AddIns/Misc/PackageManagement/Test/Src/RecentPackagesViewModelTests.cs

15
src/AddIns/Misc/PackageManagement/Test/Src/PackageViewModelTests.cs

@ -220,8 +220,11 @@ namespace PackageManagement.Tests @@ -220,8 +220,11 @@ namespace PackageManagement.Tests
}
[Test]
[Ignore("Property is not changed. Event OnParentPackageInstalled is fired")]
public void AddPackage_PackageAddedSuccessfully_PropertyNotifyChangedFiredForIsAddedProperty()
{
Assert.Ignore("OnParentPackageInstalled event, parent ([derived]PackagesViewModel) will call ReadPackages() or OnPackageChanged");
CreateViewModel();
viewModel.AddOneFakeInstallPackageOperationForViewModelPackage();
@ -233,8 +236,11 @@ namespace PackageManagement.Tests @@ -233,8 +236,11 @@ namespace PackageManagement.Tests
}
[Test]
[Ignore("Property is not changed. Event OnParentPackageInstalled is fired")]
public void AddPackage_PackageAddedSuccessfully_PropertyNotifyChangedFiredAfterPackageInstalled()
{
Assert.Ignore("OnParentPackageInstalled event, parent ([derived]PackagesViewModel) will call ReadPackages() or OnPackageChanged");
CreateViewModel();
IPackage packagePassedToInstallPackageWhenPropertyNameChanged = null;
viewModel.PropertyChanged += (sender, e) => {
@ -337,8 +343,11 @@ namespace PackageManagement.Tests @@ -337,8 +343,11 @@ namespace PackageManagement.Tests
}
[Test]
[Ignore("Property is not changed. Event OnParentPackageUninstalled is fired")]
public void RemovePackage_PackageRemovedSuccessfully_PropertyNotifyChangedFiredForIsAddedProperty()
{
Assert.Ignore("OnParentPackageUninstalled event, parent ([derived]PackagesViewModel) will call ReadPackages() or OnPackageChanged");
CreateViewModel();
string propertyChangedName = null;
viewModel.PropertyChanged += (sender, e) => propertyChangedName = e.PropertyName;
@ -348,8 +357,11 @@ namespace PackageManagement.Tests @@ -348,8 +357,11 @@ namespace PackageManagement.Tests
}
[Test]
[Ignore("Property is not changed. Event OnParentPackageUninstalled is fired")]
public void RemovePackage_PackageRemovedSuccessfully_PropertyNotifyChangedFiredAfterPackageUninstalled()
{
Assert.Ignore("OnParentPackageUninstalled event, parent ([derived]PackagesViewModel) will call ReadPackages() or OnPackageChanged");
CreateViewModel();
IPackage packagePassedToUninstallPackageWhenPropertyNameChanged = null;
viewModel.PropertyChanged += (sender, e) => {
@ -898,8 +910,11 @@ namespace PackageManagement.Tests @@ -898,8 +910,11 @@ namespace PackageManagement.Tests
}
[Test]
[Ignore("Property is not changed. Event OnParentPackageInstalled is fired")]
public void ManagePackage_TwoProjectsSelectedAndUserAcceptsSelectedProjects_IsAddedPropertyChanged()
{
Assert.Ignore("OnParentPackageInstalled event, parent ([derived]PackagesViewModel) will call ReadPackages() or OnPackageChanged");
CreateViewModelWithTwoProjectsSelected("Project A", "Project B");
viewModel.FakePackageManagementEvents.ProjectsToSelect.Add("Project A");
viewModel.FakePackageManagementEvents.ProjectsToSelect.Add("Project B");

2
src/AddIns/Misc/PackageManagement/Test/Src/RecentPackagesViewModelTests.cs

@ -90,7 +90,7 @@ namespace PackageManagement.Tests @@ -90,7 +90,7 @@ namespace PackageManagement.Tests
viewModel.ReadPackages();
CompleteReadPackagesTask();
//RecentPackagesViewModel will not re-ReadPackages OnParentPackageUninstalled
// RecentPackagesViewModel will not re-ReadPackages OnParentPackageUninstalled
// because OnParentPackageUninstalled that package is not removed from recent packages
packageManagementEvents.OnParentPackageUninstalled(new FakePackage());

Loading…
Cancel
Save