Index: Ringtoets/GrassCoverErosionInwards/test/Ringtoets.GrassCoverErosionInwards.Forms.Test/TreeNodeInfos/GrassCoverErosionInwardsCalculationGroupContextTreeNodeInfoTest.cs =================================================================== diff -u -r82136d8f4a3bb2c3a7669e788b62a1578e82c1f2 -r4e6c8ac0eb663ddfe4565745fb4829a8432019e2 --- Ringtoets/GrassCoverErosionInwards/test/Ringtoets.GrassCoverErosionInwards.Forms.Test/TreeNodeInfos/GrassCoverErosionInwardsCalculationGroupContextTreeNodeInfoTest.cs (.../GrassCoverErosionInwardsCalculationGroupContextTreeNodeInfoTest.cs) (revision 82136d8f4a3bb2c3a7669e788b62a1578e82c1f2) +++ Ringtoets/GrassCoverErosionInwards/test/Ringtoets.GrassCoverErosionInwards.Forms.Test/TreeNodeInfos/GrassCoverErosionInwardsCalculationGroupContextTreeNodeInfoTest.cs (.../GrassCoverErosionInwardsCalculationGroupContextTreeNodeInfoTest.cs) (revision 4e6c8ac0eb663ddfe4565745fb4829a8432019e2) @@ -165,7 +165,7 @@ var groupContext = new GrassCoverErosionInwardsCalculationGroupContext(group, failureMechanism, assessmentSectionMock); - var treeViewControlMock = mocks.StrictMock(); + var menuBuilderMock = mocks.StrictMock(); menuBuilderMock.Expect(mb => mb.AddOpenItem()).Return(menuBuilderMock); @@ -186,12 +186,15 @@ menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); menuBuilderMock.Expect(mb => mb.AddPropertiesItem()).Return(menuBuilderMock); menuBuilderMock.Expect(mb => mb.Build()).Return(null); - guiMock.Expect(cmp => cmp.Get(groupContext, treeViewControlMock)).Return(menuBuilderMock); - mocks.ReplayAll(); + using (var treeViewControl = new TreeViewControl()) + { + guiMock.Expect(cmp => cmp.Get(groupContext, treeViewControl)).Return(menuBuilderMock); + mocks.ReplayAll(); - // Call - info.ContextMenuStrip(groupContext, null, treeViewControlMock); + // Call + info.ContextMenuStrip(groupContext, null, treeViewControl); + } // Assert mocks.VerifyAll(); @@ -207,37 +210,38 @@ var groupContext = new GrassCoverErosionInwardsCalculationGroupContext(group, failureMechanism, assessmentSectionMock); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); - guiMock.Expect(g => g.Get(groupContext, treeViewControlMock)).Return(menuBuilder); + guiMock.Expect(g => g.Get(groupContext, treeViewControl)).Return(menuBuilder); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - ContextMenuStrip menu = info.ContextMenuStrip(groupContext, null, treeViewControlMock); + // Call + ContextMenuStrip menu = info.ContextMenuStrip(groupContext, null, treeViewControl); - // Assert - Assert.AreEqual(11, menu.Items.Count); - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationGroupIndexRootGroup, - RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup, - RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup_Tooltip, - RingtoetsFormsResources.AddFolderIcon); - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationIndexRootGroup, - RingtoetsFormsResources.CalculationGroup_Add_Calculation, - RingtoetsFormsResources.CalculationGroup_Add_Calculation_Tooltip, - RingtoetsFormsResources.FailureMechanismIcon); - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuCalculateAllIndexRootGroup, - RingtoetsFormsResources.Calculate_all, - RingtoetsFormsResources.CalculationGroup_CalculateAll_No_calculations_to_run, - RingtoetsFormsResources.CalculateIcon, - false); - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexRootGroup, - RingtoetsFormsResources.Clear_all_output, - RingtoetsFormsResources.CalculationGroup_ClearOutput_No_calculation_with_output_to_clear, - RingtoetsFormsResources.ClearIcon, - false); - + // Assert + Assert.AreEqual(11, menu.Items.Count); + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationGroupIndexRootGroup, + RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup, + RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup_Tooltip, + RingtoetsFormsResources.AddFolderIcon); + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationIndexRootGroup, + RingtoetsFormsResources.CalculationGroup_Add_Calculation, + RingtoetsFormsResources.CalculationGroup_Add_Calculation_Tooltip, + RingtoetsFormsResources.FailureMechanismIcon); + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuCalculateAllIndexRootGroup, + RingtoetsFormsResources.Calculate_all, + RingtoetsFormsResources.CalculationGroup_CalculateAll_No_calculations_to_run, + RingtoetsFormsResources.CalculateIcon, + false); + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexRootGroup, + RingtoetsFormsResources.Clear_all_output, + RingtoetsFormsResources.CalculationGroup_ClearOutput_No_calculation_with_output_to_clear, + RingtoetsFormsResources.ClearIcon, + false); + } mocks.VerifyAll(); } @@ -255,33 +259,34 @@ var parentGroupContext = new GrassCoverErosionInwardsCalculationGroupContext(parentGroup, failureMechanism, assessmentSectionMock); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilderMock = mocks.StrictMock(); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilderMock = mocks.StrictMock(); - menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddRenameItem()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddDeleteItem()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddImportItem()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddExportItem()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddExpandAllItem()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddCollapseAllItem()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.AddPropertiesItem()).Return(menuBuilderMock); - menuBuilderMock.Expect(mb => mb.Build()).Return(null); - guiMock.Expect(cmp => cmp.Get(groupContext, treeViewControlMock)).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddCustomItem(null)).IgnoreArguments().Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddRenameItem()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddDeleteItem()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddImportItem()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddExportItem()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddExpandAllItem()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddCollapseAllItem()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddSeparator()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.AddPropertiesItem()).Return(menuBuilderMock); + menuBuilderMock.Expect(mb => mb.Build()).Return(null); + guiMock.Expect(cmp => cmp.Get(groupContext, treeViewControl)).Return(menuBuilderMock); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - info.ContextMenuStrip(groupContext, parentGroupContext, treeViewControlMock); - + // Call + info.ContextMenuStrip(groupContext, parentGroupContext, treeViewControl); + } // Assert mocks.VerifyAll(); } @@ -300,39 +305,39 @@ var parentGroupContext = new GrassCoverErosionInwardsCalculationGroupContext(parentGroup, failureMechanism, assessmentSectionMock); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); + guiMock.Expect(g => g.Get(groupContext, treeViewControl)).Return(menuBuilder); - guiMock.Expect(g => g.Get(groupContext, treeViewControlMock)).Return(menuBuilder); + mocks.ReplayAll(); - mocks.ReplayAll(); + // Call + ContextMenuStrip menu = info.ContextMenuStrip(groupContext, parentGroupContext, treeViewControl); - // Call - ContextMenuStrip menu = info.ContextMenuStrip(groupContext, parentGroupContext, treeViewControlMock); + // Assert + Assert.AreEqual(9, menu.Items.Count); - // Assert - Assert.AreEqual(9, menu.Items.Count); + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationGroupIndexNestedGroup, + RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup, + RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup_Tooltip, + RingtoetsFormsResources.AddFolderIcon); + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationIndexNestedGroup, + RingtoetsFormsResources.CalculationGroup_Add_Calculation, + RingtoetsFormsResources.CalculationGroup_Add_Calculation_Tooltip, + RingtoetsFormsResources.FailureMechanismIcon); - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationGroupIndexNestedGroup, - RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup, - RingtoetsFormsResources.CalculationGroup_Add_CalculationGroup_Tooltip, - RingtoetsFormsResources.AddFolderIcon); - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuAddCalculationIndexNestedGroup, - RingtoetsFormsResources.CalculationGroup_Add_Calculation, - RingtoetsFormsResources.CalculationGroup_Add_Calculation_Tooltip, - RingtoetsFormsResources.FailureMechanismIcon); - - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuCalculateAllIndexNestedGroup, - RingtoetsFormsResources.Calculate_all, - RingtoetsFormsResources.CalculationGroup_CalculateAll_No_calculations_to_run, - RingtoetsFormsResources.CalculateIcon, - false); - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexNestedGroup, - RingtoetsFormsResources.Clear_all_output, - RingtoetsFormsResources.CalculationGroup_ClearOutput_No_calculation_with_output_to_clear, - RingtoetsFormsResources.ClearIcon, - false); - + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuCalculateAllIndexNestedGroup, + RingtoetsFormsResources.Calculate_all, + RingtoetsFormsResources.CalculationGroup_CalculateAll_No_calculations_to_run, + RingtoetsFormsResources.CalculateIcon, + false); + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexNestedGroup, + RingtoetsFormsResources.Clear_all_output, + RingtoetsFormsResources.CalculationGroup_ClearOutput_No_calculation_with_output_to_clear, + RingtoetsFormsResources.ClearIcon, + false); + } mocks.VerifyAll(); } @@ -358,25 +363,26 @@ var applicationFeatureCommandHandlerStub = mocks.Stub(); var exportImportHandlerStub = mocks.Stub(); var viewCommandsHandlerMock = mocks.StrictMock(); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControlMock); - guiMock.Expect(g => g.Get(nodeData, treeViewControlMock)).Return(menuBuilder); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControl); + guiMock.Expect(g => g.Get(nodeData, treeViewControl)).Return(menuBuilder); - treeViewControlMock.Expect(tvc => tvc.CanExpandOrCollapseForData(nodeData)).Repeat.Twice().Return(false); - viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); + viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - ContextMenuStrip menu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + // Call + ContextMenuStrip menu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Assert - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexRootGroup, - RingtoetsFormsResources.Calculate_all, - RingtoetsFormsResources.GuiPlugin_AllDataAvailable_No_failure_mechanism_sections_imported, - RingtoetsFormsResources.CalculateAllIcon, - false); + // Assert + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexRootGroup, + RingtoetsFormsResources.Calculate_all, + RingtoetsFormsResources.GuiPlugin_AllDataAvailable_No_failure_mechanism_sections_imported, + RingtoetsFormsResources.CalculateAllIcon, + false); + } mocks.VerifyAll(); } @@ -409,26 +415,26 @@ var applicationFeatureCommandHandlerStub = mocks.Stub(); var exportImportHandlerStub = mocks.Stub(); var viewCommandsHandlerMock = mocks.StrictMock(); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControlMock); - guiMock.Expect(g => g.Get(nodeData, treeViewControlMock)).Return(menuBuilder); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControl); + guiMock.Expect(g => g.Get(nodeData, treeViewControl)).Return(menuBuilder); - treeViewControlMock.Expect(tvc => tvc.CanExpandOrCollapseForData(nodeData)).Repeat.Twice().Return(false); - viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); + viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + // Call + var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Assert - TestHelper.AssertContextMenuStripContainsItem(contextMenu, contextMenuClearAllIndexRootGroup, - RingtoetsFormsResources.Calculate_all, - RingtoetsFormsResources.GuiPlugin_AllDataAvailable_No_hydraulic_boundary_database_imported, - RingtoetsFormsResources.CalculateAllIcon, - false); - + // Assert + TestHelper.AssertContextMenuStripContainsItem(contextMenu, contextMenuClearAllIndexRootGroup, + RingtoetsFormsResources.Calculate_all, + RingtoetsFormsResources.GuiPlugin_AllDataAvailable_No_hydraulic_boundary_database_imported, + RingtoetsFormsResources.CalculateAllIcon, + false); + } mocks.VerifyAll(); } @@ -461,27 +467,27 @@ var applicationFeatureCommandHandlerStub = mocks.Stub(); var exportImportHandlerStub = mocks.Stub(); var viewCommandsHandlerMock = mocks.StrictMock(); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControlMock); - guiMock.Expect(g => g.Get(nodeData, treeViewControlMock)).Return(menuBuilder); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControl); + guiMock.Expect(g => g.Get(nodeData, treeViewControl)).Return(menuBuilder); - treeViewControlMock.Expect(tvc => tvc.CanExpandOrCollapseForData(nodeData)).Repeat.Twice().Return(false); - viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); + viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - ContextMenuStrip contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + // Call + ContextMenuStrip contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Assert - ToolStripItem contextMenuItem = contextMenu.Items[contextMenuClearAllIndexRootGroup]; + // Assert + ToolStripItem contextMenuItem = contextMenu.Items[contextMenuClearAllIndexRootGroup]; - Assert.AreEqual(RingtoetsFormsResources.Calculate_all, contextMenuItem.Text); - StringAssert.Contains(string.Format(RingtoetsFormsResources.Hydraulic_boundary_database_connection_failed_0_, ""), contextMenuItem.ToolTipText); - TestHelper.AssertImagesAreEqual(RingtoetsFormsResources.CalculateAllIcon, contextMenuItem.Image); - Assert.IsFalse(contextMenuItem.Enabled); - + Assert.AreEqual(RingtoetsFormsResources.Calculate_all, contextMenuItem.Text); + StringAssert.Contains(string.Format(RingtoetsFormsResources.Hydraulic_boundary_database_connection_failed_0_, ""), contextMenuItem.ToolTipText); + TestHelper.AssertImagesAreEqual(RingtoetsFormsResources.CalculateAllIcon, contextMenuItem.Image); + Assert.IsFalse(contextMenuItem.Enabled); + } mocks.VerifyAll(); } @@ -522,27 +528,28 @@ var applicationFeatureCommandHandlerStub = mocks.Stub(); var exportImportHandlerStub = mocks.Stub(); var viewCommandsHandlerMock = mocks.StrictMock(); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControlMock); - guiMock.Expect(g => g.Get(nodeData, treeViewControlMock)).Return(menuBuilder); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new ContextMenuBuilder(applicationFeatureCommandHandlerStub, exportImportHandlerStub, viewCommandsHandlerMock, nodeData, treeViewControl); + guiMock.Expect(g => g.Get(nodeData, treeViewControl)).Return(menuBuilder); - treeViewControlMock.Expect(tvc => tvc.CanExpandOrCollapseForData(nodeData)).Repeat.Twice().Return(false); - viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); + viewCommandsHandlerMock.Expect(vc => vc.CanOpenViewFor(nodeData)).Return(false); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - ContextMenuStrip menu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + // Call + ContextMenuStrip menu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Assert - TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexRootGroup, - RingtoetsFormsResources.Calculate_all, - RingtoetsFormsResources.CalculationGroup_CalculateAll_ToolTip, - RingtoetsFormsResources.CalculateIcon); + // Assert + TestHelper.AssertContextMenuStripContainsItem(menu, contextMenuClearAllIndexRootGroup, + RingtoetsFormsResources.Calculate_all, + RingtoetsFormsResources.CalculationGroup_CalculateAll_ToolTip, + RingtoetsFormsResources.CalculateIcon); + } mocks.VerifyAll(); } - + [Test] public void ContextMenuStrip_FailureMechanismWithoutDikeProfiles_ContextMenuItemGenerateCalculationsDisabledAndTooltipSet() { @@ -556,31 +563,32 @@ failureMechanism, assessmentSectionMock); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); - guiMock.Expect(g => g.Get(nodeData, treeViewControlMock)).Return(menuBuilder); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); + guiMock.Expect(g => g.Get(nodeData, treeViewControl)).Return(menuBuilder); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + // Call + var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Assert - TestHelper.AssertContextMenuStripContainsItem(contextMenu, contextMenuGenerateCalculationsIndexRootGroup, - RingtoetsFormsResources.CalculationGroup_Generate_Scenarios, - GrassCoverErosionInwardsPluginResources.GrassCoverErosionInwardsGuiPlugin_CreateGenerateCalculationsItem_NoDikeLocations_ToolTip, - RingtoetsFormsResources.GenerateScenariosIcon, - false); - + // Assert + TestHelper.AssertContextMenuStripContainsItem(contextMenu, contextMenuGenerateCalculationsIndexRootGroup, + RingtoetsFormsResources.CalculationGroup_Generate_Scenarios, + GrassCoverErosionInwardsPluginResources.GrassCoverErosionInwardsGuiPlugin_CreateGenerateCalculationsItem_NoDikeLocations_ToolTip, + RingtoetsFormsResources.GenerateScenariosIcon, + false); + } mocks.VerifyAll(); } - + [Test] public void ContextMenuStrip_FailureMechanismWithDikeProfiles_ContextMenuItemGenerateCalculationsDisabledAndTooltipSet() { // Setup var failureMechanism = new GrassCoverErosionInwardsFailureMechanism(); - failureMechanism.DikeProfiles.Add(new DikeProfile(new Point2D(1,2))); + failureMechanism.DikeProfiles.Add(new DikeProfile(new Point2D(1, 2))); var assessmentSectionMock = mocks.StrictMock(); assessmentSectionMock.Stub(asm => asm.HydraulicBoundaryDatabase).Return(null); @@ -589,21 +597,22 @@ failureMechanism, assessmentSectionMock); - var treeViewControlMock = mocks.StrictMock(); - var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); - guiMock.Expect(g => g.Get(nodeData, treeViewControlMock)).Return(menuBuilder); + using (var treeViewControl = new TreeViewControl()) + { + var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); + guiMock.Expect(g => g.Get(nodeData, treeViewControl)).Return(menuBuilder); - mocks.ReplayAll(); + mocks.ReplayAll(); - // Call - var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + // Call + var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Assert - TestHelper.AssertContextMenuStripContainsItem(contextMenu, contextMenuGenerateCalculationsIndexRootGroup, - RingtoetsFormsResources.CalculationGroup_Generate_Scenarios, - GrassCoverErosionInwardsPluginResources.GrassCoverErosionInwardsGuiPlugin_CreateGenerateCalculationsItem_ToolTip, - RingtoetsFormsResources.GenerateScenariosIcon); - + // Assert + TestHelper.AssertContextMenuStripContainsItem(contextMenu, contextMenuGenerateCalculationsIndexRootGroup, + RingtoetsFormsResources.CalculationGroup_Generate_Scenarios, + GrassCoverErosionInwardsPluginResources.GrassCoverErosionInwardsGuiPlugin_CreateGenerateCalculationsItem_ToolTip, + RingtoetsFormsResources.GenerateScenariosIcon); + } mocks.VerifyAll(); } @@ -612,7 +621,6 @@ { // Setup var mainWindowStub = mocks.Stub(); - var treeViewControlMock = mocks.StrictMock(); var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); var failureMechanism = new GrassCoverErosionInwardsFailureMechanism(); @@ -650,43 +658,45 @@ assessmentSection.HydraulicBoundaryDatabase = hydraulicBoundaryDatabaseStub; var groupContext = new GrassCoverErosionInwardsCalculationGroupContext(failureMechanism.CalculationsGroup, - failureMechanism, - assessmentSection); + failureMechanism, + assessmentSection); - guiMock.Expect(g => g.Get(groupContext, treeViewControlMock)).Return(menuBuilder); - guiMock.Expect(g => g.MainWindow).Return(mainWindowStub); + using (var treeViewControl = new TreeViewControl()) + { + guiMock.Expect(g => g.Get(groupContext, treeViewControl)).Return(menuBuilder); + guiMock.Expect(g => g.MainWindow).Return(mainWindowStub); - mocks.ReplayAll(); + mocks.ReplayAll(); - plugin.Gui = guiMock; - var contextMenu = info.ContextMenuStrip(groupContext, null, treeViewControlMock); + plugin.Gui = guiMock; + var contextMenu = info.ContextMenuStrip(groupContext, null, treeViewControl); - DialogBoxHandler = (name, wnd) => - { - // Expect an activity dialog which is automatically closed - }; + DialogBoxHandler = (name, wnd) => + { + // Expect an activity dialog which is automatically closed + }; - // Call - TestHelper.AssertLogMessages(() => contextMenu.Items[contextMenuCalculateAllIndexRootGroup].PerformClick(), messages => - { - var messageList = messages.ToList(); + // Call + TestHelper.AssertLogMessages(() => contextMenu.Items[contextMenuCalculateAllIndexRootGroup].PerformClick(), messages => + { + var messageList = messages.ToList(); - // Assert - Assert.AreEqual(12, messageList.Count); - StringAssert.StartsWith("Validatie van 'A' gestart om: ", messageList[0]); - StringAssert.StartsWith("Validatie van 'A' beëindigd om: ", messageList[1]); - StringAssert.StartsWith("Berekening van 'A' gestart om: ", messageList[2]); - Assert.AreEqual("De berekening voor grasbekleding erosie kruin en binnentalud 'A' is niet gelukt.", messageList[3]); - StringAssert.StartsWith("Berekening van 'A' beëindigd om: ", messageList[4]); - StringAssert.StartsWith("Validatie van 'B' gestart om: ", messageList[5]); - StringAssert.StartsWith("Validatie van 'B' beëindigd om: ", messageList[6]); - StringAssert.StartsWith("Berekening van 'B' gestart om: ", messageList[7]); - Assert.AreEqual("De berekening voor grasbekleding erosie kruin en binnentalud 'B' is niet gelukt.", messageList[8]); - StringAssert.StartsWith("Berekening van 'B' beëindigd om: ", messageList[9]); - Assert.AreEqual("Uitvoeren van 'A' is mislukt.", messageList[10]); - Assert.AreEqual("Uitvoeren van 'B' is mislukt.", messageList[11]); - }); - + // Assert + Assert.AreEqual(12, messageList.Count); + StringAssert.StartsWith("Validatie van 'A' gestart om: ", messageList[0]); + StringAssert.StartsWith("Validatie van 'A' beëindigd om: ", messageList[1]); + StringAssert.StartsWith("Berekening van 'A' gestart om: ", messageList[2]); + Assert.AreEqual("De berekening voor grasbekleding erosie kruin en binnentalud 'A' is niet gelukt.", messageList[3]); + StringAssert.StartsWith("Berekening van 'A' beëindigd om: ", messageList[4]); + StringAssert.StartsWith("Validatie van 'B' gestart om: ", messageList[5]); + StringAssert.StartsWith("Validatie van 'B' beëindigd om: ", messageList[6]); + StringAssert.StartsWith("Berekening van 'B' gestart om: ", messageList[7]); + Assert.AreEqual("De berekening voor grasbekleding erosie kruin en binnentalud 'B' is niet gelukt.", messageList[8]); + StringAssert.StartsWith("Berekening van 'B' beëindigd om: ", messageList[9]); + Assert.AreEqual("Uitvoeren van 'A' is mislukt.", messageList[10]); + Assert.AreEqual("Uitvoeren van 'B' is mislukt.", messageList[11]); + }); + } mocks.VerifyAll(); } @@ -706,31 +716,32 @@ Name = "Nieuwe map" }; var observerMock = mocks.StrictMock(); - var treeViewControlMock = mocks.StrictMock(); - observerMock.Expect(o => o.UpdateObserver()); - guiMock.Expect(cmp => cmp.Get(nodeData, treeViewControlMock)).Return(menuBuilder); - mocks.ReplayAll(); + using (var treeViewControl = new TreeViewControl()) + { + guiMock.Expect(cmp => cmp.Get(nodeData, treeViewControl)).Return(menuBuilder); - group.Children.Add(calculationGroup); - nodeData.Attach(observerMock); + mocks.ReplayAll(); - ContextMenuStrip contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + group.Children.Add(calculationGroup); + nodeData.Attach(observerMock); - // Precondition - Assert.AreEqual(1, group.Children.Count); + ContextMenuStrip contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Call - contextMenu.Items[contextMenuAddCalculationGroupIndexRootGroup].PerformClick(); + // Precondition + Assert.AreEqual(1, group.Children.Count); - // Assert - Assert.AreEqual(2, group.Children.Count); - var newlyAddedItem = group.Children.Last(); - Assert.IsInstanceOf(newlyAddedItem); - Assert.AreEqual("Nieuwe map (1)", newlyAddedItem.Name, - "An item with the same name default name already exists, therefore '(1)' needs to be appended."); + // Call + contextMenu.Items[contextMenuAddCalculationGroupIndexRootGroup].PerformClick(); + // Assert + Assert.AreEqual(2, group.Children.Count); + var newlyAddedItem = group.Children.Last(); + Assert.IsInstanceOf(newlyAddedItem); + Assert.AreEqual("Nieuwe map (1)", newlyAddedItem.Name, + "An item with the same name default name already exists, therefore '(1)' needs to be appended."); + } mocks.VerifyAll(); } @@ -750,31 +761,32 @@ Name = "Nieuwe berekening" }; var observerMock = mocks.StrictMock(); - var treeViewControlMock = mocks.StrictMock(); - observerMock.Expect(o => o.UpdateObserver()); - guiMock.Expect(cmp => cmp.Get(nodeData, treeViewControlMock)).Return(menuBuilder); - mocks.ReplayAll(); + using (var treeViewControl = new TreeViewControl()) + { + guiMock.Expect(cmp => cmp.Get(nodeData, treeViewControl)).Return(menuBuilder); - group.Children.Add(calculationItem); - nodeData.Attach(observerMock); + mocks.ReplayAll(); - var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControlMock); + group.Children.Add(calculationItem); + nodeData.Attach(observerMock); - // Precondition - Assert.AreEqual(1, group.Children.Count); + var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); - // Call - contextMenu.Items[contextMenuAddCalculationIndexRootGroup].PerformClick(); + // Precondition + Assert.AreEqual(1, group.Children.Count); - // Assert - Assert.AreEqual(2, group.Children.Count); - var newlyAddedItem = group.Children.Last(); - Assert.IsInstanceOf(newlyAddedItem); - Assert.AreEqual("Nieuwe berekening (1)", newlyAddedItem.Name, - "An item with the same name default name already exists, therefore '(1)' needs to be appended."); + // Call + contextMenu.Items[contextMenuAddCalculationIndexRootGroup].PerformClick(); + // Assert + Assert.AreEqual(2, group.Children.Count); + var newlyAddedItem = group.Children.Last(); + Assert.IsInstanceOf(newlyAddedItem); + Assert.AreEqual("Nieuwe berekening (1)", newlyAddedItem.Name, + "An item with the same name default name already exists, therefore '(1)' needs to be appended."); + } mocks.VerifyAll(); } @@ -806,8 +818,8 @@ }; var nodeData = new GrassCoverErosionInwardsCalculationGroupContext(failureMechanism.CalculationsGroup, - failureMechanism, - assessmentSectionMock); + failureMechanism, + assessmentSectionMock); var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); var mainWindow = mocks.Stub(); @@ -822,8 +834,8 @@ DialogBoxHandler = (name, wnd) => { - var selectionDialog = (GrassCoverErosionInwardsDikeProfileSelectionDialog)new FormTester(name).TheObject; - var grid = (DataGridView)new ControlTester("DikeProfileDataGrid", selectionDialog).TheObject; + var selectionDialog = (GrassCoverErosionInwardsDikeProfileSelectionDialog) new FormTester(name).TheObject; + var grid = (DataGridView) new ControlTester("DikeProfileDataGrid", selectionDialog).TheObject; grid.Rows[0].Cells[0].Value = true; @@ -870,8 +882,8 @@ }; var nodeData = new GrassCoverErosionInwardsCalculationGroupContext(failureMechanism.CalculationsGroup, - failureMechanism, - assessmentSectionMock); + failureMechanism, + assessmentSectionMock); var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); var mainWindow = mocks.Stub(); @@ -883,11 +895,11 @@ mocks.ReplayAll(); plugin.Gui = gui; - + DialogBoxHandler = (name, wnd) => { - var selectionDialog = (GrassCoverErosionInwardsDikeProfileSelectionDialog)new FormTester(name).TheObject; - var grid = (DataGridView)new ControlTester("DikeProfileDataGrid", selectionDialog).TheObject; + var selectionDialog = (GrassCoverErosionInwardsDikeProfileSelectionDialog) new FormTester(name).TheObject; + var grid = (DataGridView) new ControlTester("DikeProfileDataGrid", selectionDialog).TheObject; grid.Rows[0].Cells[0].Value = true; @@ -900,8 +912,7 @@ contextMenu.Items[contextMenuGenerateCalculationsIndexRootGroup].PerformClick(); // Then - var grassCoverErosionInwardsCalculations = failureMechanism.Calculations.OfType().ToArray(); - Assert.AreEqual(0, grassCoverErosionInwardsCalculations.Length); + Assert.AreEqual(0, failureMechanism.Calculations.OfType().Count()); } mocks.VerifyAll(); } @@ -939,8 +950,8 @@ }; var nodeData = new GrassCoverErosionInwardsCalculationGroupContext(failureMechanism.CalculationsGroup, - failureMechanism, - assessmentSectionMock); + failureMechanism, + assessmentSectionMock); var menuBuilder = new CustomItemsOnlyContextMenuBuilder(); var mainWindow = mocks.Stub(); @@ -952,20 +963,20 @@ mocks.ReplayAll(); plugin.Gui = gui; - + DialogBoxHandler = (name, wnd) => { - var selectionDialog = (GrassCoverErosionInwardsDikeProfileSelectionDialog)new FormTester(name).TheObject; - var grid = (DataGridView)new ControlTester("DikeProfileDataGrid", selectionDialog).TheObject; + var selectionDialog = (GrassCoverErosionInwardsDikeProfileSelectionDialog) new FormTester(name).TheObject; + var grid = (DataGridView) new ControlTester("DikeProfileDataGrid", selectionDialog).TheObject; grid.Rows[0].Cells[0].Value = true; new ButtonTester("OkButton", selectionDialog).Click(); }; - var contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); + ContextMenuStrip contextMenu = info.ContextMenuStrip(nodeData, null, treeViewControl); - var expectedNewName = NamingHelper.GetUniqueName(failureMechanism.CalculationsGroup.Children, existingCalculationName, c => c.Name); + string expectedNewName = NamingHelper.GetUniqueName(failureMechanism.CalculationsGroup.Children, existingCalculationName, c => c.Name); // When contextMenu.Items[contextMenuGenerateCalculationsIndexRootGroup].PerformClick();