Index: Ringtoets/Piping/src/Ringtoets.Piping.Plugin/PipingPlugin.cs =================================================================== diff -u -rbc771ec1743c16b85b12a768f1fa10431be9948d -r429c5e44a8ade2b6261e42c334335fed974a222e --- Ringtoets/Piping/src/Ringtoets.Piping.Plugin/PipingPlugin.cs (.../PipingPlugin.cs) (revision bc771ec1743c16b85b12a768f1fa10431be9948d) +++ Ringtoets/Piping/src/Ringtoets.Piping.Plugin/PipingPlugin.cs (.../PipingPlugin.cs) (revision 429c5e44a8ade2b6261e42c334335fed974a222e) @@ -120,7 +120,7 @@ Category = RingtoetsCommonFormsResources.Ringtoets_Category, Image = PipingFormsResources.PipingSurfaceLineIcon, FileFilterGenerator = PipingSurfaceLineFileFilter, - IsEnabled = context => context.AssessmentSection.ReferenceLine != null, + IsEnabled = context => HasReferenceLine(context.AssessmentSection), CreateFileImporter = (context, filePath) => new SurfaceLinesCsvImporter( context.WrappedData, filePath, @@ -135,7 +135,7 @@ Category = RingtoetsCommonFormsResources.Ringtoets_Category, Image = PipingFormsResources.PipingSoilProfileIcon, FileFilterGenerator = StochasticSoilModelFileFilter, - IsEnabled = context => context.AssessmentSection.ReferenceLine != null, + IsEnabled = context => HasReferenceLine(context.AssessmentSection), CreateFileImporter = (context, filePath) => new StochasticSoilModelImporter( context.WrappedData, filePath, @@ -414,6 +414,11 @@ return !changeHandler.RequireConfirmation() || changeHandler.InquireConfirmation(); } + private static bool HasReferenceLine(IAssessmentSection assessmentSection) + { + return assessmentSection.ReferenceLine.Points.Any(); + } + #region PipingFailureMechanismView ViewInfo private static bool ClosePipingFailureMechanismViewForData(PipingFailureMechanismView view, object o) Index: Ringtoets/Piping/test/Ringtoets.Piping.Plugin.Test/ImportInfos/PipingStochasticSoilModelCollectionContextImportInfoTest.cs =================================================================== diff -u -r0cb75aafe2dae01db4edda194e0b27da3a0f382f -r429c5e44a8ade2b6261e42c334335fed974a222e --- Ringtoets/Piping/test/Ringtoets.Piping.Plugin.Test/ImportInfos/PipingStochasticSoilModelCollectionContextImportInfoTest.cs (.../PipingStochasticSoilModelCollectionContextImportInfoTest.cs) (revision 0cb75aafe2dae01db4edda194e0b27da3a0f382f) +++ Ringtoets/Piping/test/Ringtoets.Piping.Plugin.Test/ImportInfos/PipingStochasticSoilModelCollectionContextImportInfoTest.cs (.../PipingStochasticSoilModelCollectionContextImportInfoTest.cs) (revision 429c5e44a8ade2b6261e42c334335fed974a222e) @@ -32,6 +32,7 @@ using NUnit.Framework; using Rhino.Mocks; using Ringtoets.Common.Data.AssessmentSection; +using Ringtoets.Common.Data.TestUtil; using Ringtoets.Common.IO.SoilProfile; using Ringtoets.Piping.Data; using Ringtoets.Piping.Data.SoilProfile; @@ -78,11 +79,12 @@ } [Test] - public void IsEnabled_ReferenceLineNull_ReturnFalse() + public void IsEnabled_ReferenceLineWithoutGeometry_ReturnFalse() { // Setup var mocks = new MockRepository(); var assessmentSection = mocks.Stub(); + assessmentSection.Stub(a => a.ReferenceLine).Return(new ReferenceLine()); mocks.ReplayAll(); var failureMechanism = new PipingFailureMechanism(); @@ -98,12 +100,12 @@ } [Test] - public void IsEnabled_ReferenceLineSet_ReturnTrue() + public void IsEnabled_ReferenceLineWithGeometry_ReturnTrue() { // Setup var mocks = new MockRepository(); var assessmentSection = mocks.Stub(); - assessmentSection.Stub(a => a.ReferenceLine).Return(new ReferenceLine()); + assessmentSection.Stub(a => a.ReferenceLine).Return(ReferenceLineTestFactory.CreateReferenceLineWithGeometry()); mocks.ReplayAll(); var failureMechanism = new PipingFailureMechanism(); Index: Ringtoets/Piping/test/Ringtoets.Piping.Plugin.Test/ImportInfos/PipingSurfaceLinesContextImportInfoTest.cs =================================================================== diff -u -r0cb75aafe2dae01db4edda194e0b27da3a0f382f -r429c5e44a8ade2b6261e42c334335fed974a222e --- Ringtoets/Piping/test/Ringtoets.Piping.Plugin.Test/ImportInfos/PipingSurfaceLinesContextImportInfoTest.cs (.../PipingSurfaceLinesContextImportInfoTest.cs) (revision 0cb75aafe2dae01db4edda194e0b27da3a0f382f) +++ Ringtoets/Piping/test/Ringtoets.Piping.Plugin.Test/ImportInfos/PipingSurfaceLinesContextImportInfoTest.cs (.../PipingSurfaceLinesContextImportInfoTest.cs) (revision 429c5e44a8ade2b6261e42c334335fed974a222e) @@ -32,6 +32,7 @@ using NUnit.Framework; using Rhino.Mocks; using Ringtoets.Common.Data.AssessmentSection; +using Ringtoets.Common.Data.TestUtil; using Ringtoets.Common.IO.SurfaceLines; using Ringtoets.Piping.Data; using Ringtoets.Piping.Data.TestUtil; @@ -78,11 +79,12 @@ } [Test] - public void IsEnabled_ReferenceLineNull_ReturnFalse() + public void IsEnabled_ReferenceLineWithoutGeometry_ReturnFalse() { // Setup var mocks = new MockRepository(); var assessmentSection = mocks.Stub(); + assessmentSection.Stub(a => a.ReferenceLine).Return(new ReferenceLine()); mocks.ReplayAll(); var failureMechanism = new PipingFailureMechanism(); @@ -99,12 +101,12 @@ } [Test] - public void IsEnabled_ReferenceLineSet_ReturnTrue() + public void IsEnabled_ReferenceLineWithGeometry_ReturnTrue() { // Setup var mocks = new MockRepository(); var assessmentSection = mocks.Stub(); - assessmentSection.Stub(a => a.ReferenceLine).Return(new ReferenceLine()); + assessmentSection.Stub(a => a.ReferenceLine).Return(ReferenceLineTestFactory.CreateReferenceLineWithGeometry()); mocks.ReplayAll(); var failureMechanism = new PipingFailureMechanism();