Index: Ringtoets/Common/test/Ringtoets.Common.Forms.Test/Views/HydraulicBoundaryLocationsViewTest.cs =================================================================== diff -u -rf1876d31d910814fdf1d327c325fc1dfd6846c5e -rb7267258b1ff8fbdd11cb3a9561eb7d8a71d2935 --- Ringtoets/Common/test/Ringtoets.Common.Forms.Test/Views/HydraulicBoundaryLocationsViewTest.cs (.../HydraulicBoundaryLocationsViewTest.cs) (revision f1876d31d910814fdf1d327c325fc1dfd6846c5e) +++ Ringtoets/Common/test/Ringtoets.Common.Forms.Test/Views/HydraulicBoundaryLocationsViewTest.cs (.../HydraulicBoundaryLocationsViewTest.cs) (revision b7267258b1ff8fbdd11cb3a9561eb7d8a71d2935) @@ -23,10 +23,10 @@ using System.Collections.Generic; using System.Linq; using System.Windows.Forms; -using Core.Common.Base; using Core.Common.Base.Geometry; using NUnit.Extensions.Forms; using NUnit.Framework; +using Ringtoets.Common.Data.AssessmentSection; using Ringtoets.Common.Data.Hydraulics; using Ringtoets.Common.Data.IllustrationPoints; using Ringtoets.Common.Data.TestUtil; @@ -59,15 +59,14 @@ } [Test] - public void DefaultConstructor_DefaultValues() + public void Constructor_ExpectedValues() { - // Call - using (var view = new TestHydraulicBoundaryLocationsView()) - { - // Assert - Assert.IsInstanceOf>(view); - Assert.IsNull(view.Data); - } + // Setup & Call + TestHydraulicBoundaryLocationsView view = ShowFullyConfiguredTestHydraulicBoundaryLocationsView(); + + // Assert + Assert.IsInstanceOf>(view); + Assert.IsNull(view.Data); } [Test] @@ -100,38 +99,6 @@ } [Test] - public void Data_HydraulicBoundaryLocations_DataSet() - { - // Setup - using (var view = new TestHydraulicBoundaryLocationsView()) - { - IEnumerable hydraulicBoundaryLocations = Enumerable.Empty(); - - // Call - view.Data = hydraulicBoundaryLocations; - - // Assert - Assert.AreSame(hydraulicBoundaryLocations, view.Data); - } - } - - [Test] - public void Data_OtherThanHydraulicBoundaryLocations_DataNull() - { - // Setup - using (var view = new TestHydraulicBoundaryLocationsView()) - { - var data = new object(); - - // Call - view.Data = data; - - // Assert - Assert.IsNull(view.Data); - } - } - - [Test] public void HydraulicBoundaryLocationsView_AssessmentSectionWithData_DataGridViewCorrectlyInitialized() { // Setup & Call @@ -216,12 +183,10 @@ }; view.ItemToCreate = calculation; - var hydraulicBoundaryLocations = new ObservableList(); var hydraulicBoundaryLocation = new TestHydraulicBoundaryLocation(); - hydraulicBoundaryLocations.Add(hydraulicBoundaryLocation); + view.AssessmentSection.HydraulicBoundaryDatabase.Locations.Add(hydraulicBoundaryLocation); + view.AssessmentSection.HydraulicBoundaryDatabase.Locations.NotifyObservers(); - view.Data = hydraulicBoundaryLocations; - // Call IEnumerable actualControlItems = view.PublicGetIllustrationPointControlItems(); @@ -260,11 +225,10 @@ return view; } - private void ShowFullyConfiguredTestHydraulicBoundaryLocationsView() + private TestHydraulicBoundaryLocationsView ShowFullyConfiguredTestHydraulicBoundaryLocationsView() { - TestHydraulicBoundaryLocationsView view = ShowTestHydraulicBoundaryLocationsView(); - - view.Data = new[] + var assessmentSection = new ObservableTestAssessmentSectionStub(); + assessmentSection.HydraulicBoundaryDatabase.Locations.AddRange(new[] { new HydraulicBoundaryLocation(1, "1", 1.0, 1.0), new HydraulicBoundaryLocation(2, "2", 2.0, 2.0) @@ -291,13 +255,22 @@ } } } - }; + }); + + var view = new TestHydraulicBoundaryLocationsView(assessmentSection); + + testForm.Controls.Add(view); + testForm.Show(); + + return view; } private sealed class TestHydraulicBoundaryLocationsView : HydraulicBoundaryLocationsView { - public TestHydraulicBoundaryLocationsView() : base(new ObservableTestAssessmentSectionStub()) {} + public TestHydraulicBoundaryLocationsView() : this(new ObservableTestAssessmentSectionStub()) {} + public TestHydraulicBoundaryLocationsView(IAssessmentSection assessmentSection) : base(assessmentSection) {} + public HydraulicBoundaryLocation GetCalculationsCallArgument { get; private set; } public HydraulicBoundaryLocationCalculation ItemToCreate { private get; set; }