Index: Ringtoets/Piping/test/Ringtoets.Piping.Forms.Test/PropertyClasses/PipingOutputContextPropertiesTest.cs =================================================================== diff -u -r3fe99efe7580ec1d9eb850908c001c990dc43fe5 -rbbb75989e9785f5082835556cc30e643e6eb5890 --- Ringtoets/Piping/test/Ringtoets.Piping.Forms.Test/PropertyClasses/PipingOutputContextPropertiesTest.cs (.../PipingOutputContextPropertiesTest.cs) (revision 3fe99efe7580ec1d9eb850908c001c990dc43fe5) +++ Ringtoets/Piping/test/Ringtoets.Piping.Forms.Test/PropertyClasses/PipingOutputContextPropertiesTest.cs (.../PipingOutputContextPropertiesTest.cs) (revision bbb75989e9785f5082835556cc30e643e6eb5890) @@ -27,7 +27,6 @@ using Ringtoets.Common.Data.TestUtil; using Ringtoets.Piping.Data; using Ringtoets.Piping.Data.TestUtil; -using Ringtoets.Piping.Forms.PresentationObjects; using Ringtoets.Piping.Forms.PropertyClasses; namespace Ringtoets.Piping.Forms.Test.PropertyClasses @@ -36,17 +35,42 @@ public class PipingOutputContextPropertiesTest { [Test] - public void DefaultConstructor_ExpectedValues() + public void Constructor_OutputNull_ThrowsArgumentNullException() { // Call - var properties = new PipingOutputContextProperties(); + TestDelegate call = () => new PipingOutputContextProperties(null, new TestPipingSemiProbabilisticOutput()); // Assert - Assert.IsInstanceOf>(properties); - Assert.IsNull(properties.Data); + var exception = Assert.Throws(call); + Assert.AreEqual("output", exception.ParamName); } [Test] + public void Constructor_SemiProbabilisticOutputNull_ThrowsArgumentNullException() + { + // Call + TestDelegate call = () => new PipingOutputContextProperties(new TestPipingOutput(), null); + + // Assert + var exception = Assert.Throws(call); + Assert.AreEqual("semiProbabilisticOutput", exception.ParamName); + } + + [Test] + public void Constructor_ExpectedValues() + { + // Setup + var output = new TestPipingOutput(); + + // Call + var properties = new PipingOutputContextProperties(output, new TestPipingSemiProbabilisticOutput()); + + // Assert + Assert.IsInstanceOf>(properties); + Assert.AreSame(output, properties.Data); + } + + [Test] public void GetProperties_WithData_ReturnExpectedValues() { // Setup @@ -98,12 +122,9 @@ }); // Call - var properties = new PipingOutputContextProperties - { - Data = new PipingOutputContext(output, semiProbabilisticOutput) - }; + var properties = new PipingOutputContextProperties(output, semiProbabilisticOutput); - // Call & Assert + // Assert const string probabilityFormat = "1/{0:n0}"; Assert.AreEqual(upliftFactorOfSafety, properties.UpliftFactorOfSafety, properties.UpliftFactorOfSafety.GetAccuracy()); Assert.AreEqual(upliftReliability, properties.UpliftReliability, properties.UpliftReliability.GetAccuracy()); @@ -165,12 +186,9 @@ pipingReliability, pipingFactorOfSafety); - var properties = new PipingOutputContextProperties - { - Data = new PipingOutputContext(new TestPipingOutput(), semiProbabilisticOutput) - }; + var properties = new PipingOutputContextProperties(new TestPipingOutput(), semiProbabilisticOutput); - // Call & Assert + // Assert const string probability = "1/Oneindig"; Assert.AreEqual(probability, properties.UpliftProbability); Assert.AreEqual(probability, properties.HeaveProbability); @@ -216,10 +234,7 @@ pipingFactorOfSafety); // Call - var properties = new PipingOutputContextProperties - { - Data = new PipingOutputContext(new TestPipingOutput(), semiProbabilisticOutput) - }; + var properties = new PipingOutputContextProperties(new TestPipingOutput(), semiProbabilisticOutput); // Assert PropertyDescriptorCollection dynamicProperties = PropertiesTestHelper.GetAllVisiblePropertyDescriptors(properties);