Index: Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingCalculationActivityTest.cs =================================================================== diff -u -rbcd6928813d32119c3b5d8c5c4fa65455e5b4fb8 -r8650820cbc2da7dcd34964bc92046fe66d14ec23 --- Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingCalculationActivityTest.cs (.../PipingCalculationActivityTest.cs) (revision bcd6928813d32119c3b5d8c5c4fa65455e5b4fb8) +++ Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingCalculationActivityTest.cs (.../PipingCalculationActivityTest.cs) (revision 8650820cbc2da7dcd34964bc92046fe66d14ec23) @@ -82,7 +82,7 @@ // Setup var originalOutput = new TestPipingOutput(); - var invalidPipingCalculation = PipingCalculationFactory.CreateCalculationWithInvalidData(); + PipingCalculationScenario invalidPipingCalculation = PipingCalculationFactory.CreateCalculationWithInvalidData(); invalidPipingCalculation.Output = originalOutput; var activity = new PipingCalculationActivity(invalidPipingCalculation, new PipingProbabilityAssessmentInput(), int.MinValue, double.NaN); @@ -111,10 +111,10 @@ public void Run_ValidPipingCalculation_PerformPipingValidationAndCalculationAndLogStartAndEnd() { // Setup - var validPipingCalculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + PipingCalculationScenario validPipingCalculation = PipingCalculationFactory.CreateCalculationWithValidInput(); validPipingCalculation.Output = null; - var norm = new Random(21).Next(100, 300000); + int norm = new Random(21).Next(100, 300000); var activity = new PipingCalculationActivity(validPipingCalculation, new PipingProbabilityAssessmentInput(), norm, double.NaN); activity.Run(); @@ -144,7 +144,7 @@ observerMock.Expect(o => o.UpdateObserver()); mocks.ReplayAll(); - var validPipingCalculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + PipingCalculationScenario validPipingCalculation = PipingCalculationFactory.CreateCalculationWithValidInput(); validPipingCalculation.Output = null; validPipingCalculation.Attach(observerMock); Index: Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingCalculationServiceTest.cs =================================================================== diff -u -r2beed1614ecded5cc316f6deeb8d4b2e7fc73d28 -r8650820cbc2da7dcd34964bc92046fe66d14ec23 --- Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingCalculationServiceTest.cs (.../PipingCalculationServiceTest.cs) (revision 2beed1614ecded5cc316f6deeb8d4b2e7fc73d28) +++ Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingCalculationServiceTest.cs (.../PipingCalculationServiceTest.cs) (revision 8650820cbc2da7dcd34964bc92046fe66d14ec23) @@ -67,7 +67,7 @@ invalidPipingCalculation.Output = output; // Call - var isValid = PipingCalculationService.Validate(invalidPipingCalculation); + bool isValid = PipingCalculationService.Validate(invalidPipingCalculation); // Assert Assert.IsFalse(isValid); @@ -80,8 +80,10 @@ // Setup const string name = ""; - var calculation = new PipingCalculation(new GeneralPipingInput()); - calculation.Name = name; + var calculation = new PipingCalculation(new GeneralPipingInput()) + { + Name = name + }; // Call bool isValid = false; @@ -108,7 +110,7 @@ { // Setup const string name = ""; - + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); calculation.InputParameters.HydraulicBoundaryLocation = null; calculation.Name = name; @@ -161,7 +163,7 @@ { // Setup const string name = ""; - + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); calculation.InputParameters.EntryPointL = (RoundedDouble) double.NaN; calculation.Name = name; @@ -189,7 +191,7 @@ const string name = ""; PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); - calculation.InputParameters.ExitPointL = (RoundedDouble)double.NaN; + calculation.InputParameters.ExitPointL = (RoundedDouble) double.NaN; calculation.Name = name; // Call @@ -213,7 +215,7 @@ { // Setup const string name = ""; - + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); calculation.InputParameters.SurfaceLine = null; calculation.InputParameters.ExitPointL = (RoundedDouble) 0.9; @@ -241,7 +243,7 @@ { // Setup const string name = ""; - + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); calculation.InputParameters.StochasticSoilProfile = null; calculation.Name = name; @@ -268,20 +270,19 @@ { // Setup const string name = ""; - + + var profile = new PipingSoilProfile(string.Empty, 0.0, + new[] + { + new PipingSoilLayer(2.0) + { + IsAquifer = false + } + }, + SoilProfileType.SoilProfile1D, -1); + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); - calculation.InputParameters.StochasticSoilProfile.SoilProfile = new PipingSoilProfile( - string.Empty, - 0.0, - new[] - { - new PipingSoilLayer(2.0) - { - IsAquifer = false - }, - }, - SoilProfileType.SoilProfile1D, - -1); + calculation.InputParameters.StochasticSoilProfile.SoilProfile = profile; calculation.Name = name; bool isValid = false; @@ -309,25 +310,24 @@ { // Setup const string name = ""; - - PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + var random = new Random(21); - calculation.InputParameters.StochasticSoilProfile.SoilProfile = new PipingSoilProfile( - string.Empty, - 0.0, - new[] - { - new PipingSoilLayer(2.0) - { - IsAquifer = true, - DiameterD70Deviation = random.NextDouble(), - DiameterD70Mean = 0.1 + random.NextDouble(), - PermeabilityDeviation = random.NextDouble(), - PermeabilityMean = 0.1 + random.NextDouble() - }, - }, - SoilProfileType.SoilProfile1D, - -1); + var profile = new PipingSoilProfile(string.Empty, 0.0, + new[] + { + new PipingSoilLayer(2.0) + { + IsAquifer = true, + DiameterD70Deviation = random.NextDouble(), + DiameterD70Mean = 0.1 + random.NextDouble(), + PermeabilityDeviation = random.NextDouble(), + PermeabilityMean = 0.1 + random.NextDouble() + } + }, + SoilProfileType.SoilProfile1D, -1); + + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + calculation.InputParameters.StochasticSoilProfile.SoilProfile = profile; calculation.Name = name; bool isValid = false; @@ -354,28 +354,27 @@ // Setup const string name = ""; - PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); var random = new Random(21); - calculation.InputParameters.StochasticSoilProfile.SoilProfile = new PipingSoilProfile( - string.Empty, - 0.0, - new[] - { - new PipingSoilLayer(13.0) - { - IsAquifer = false - }, - new PipingSoilLayer(11.0) - { - IsAquifer = true, - DiameterD70Deviation = random.NextDouble(), - DiameterD70Mean = 0.1 + random.NextDouble(), - PermeabilityDeviation = random.NextDouble(), - PermeabilityMean = 0.1 + random.NextDouble() - }, - }, - SoilProfileType.SoilProfile1D, - -1); + var profile = new PipingSoilProfile(string.Empty, 0.0, + new[] + { + new PipingSoilLayer(13.0) + { + IsAquifer = false + }, + new PipingSoilLayer(11.0) + { + IsAquifer = true, + DiameterD70Deviation = random.NextDouble(), + DiameterD70Mean = 0.1 + random.NextDouble(), + PermeabilityDeviation = random.NextDouble(), + PermeabilityMean = 0.1 + random.NextDouble() + } + }, + SoilProfileType.SoilProfile1D, -1); + + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + calculation.InputParameters.StochasticSoilProfile.SoilProfile = profile; calculation.Name = name; bool isValid = false; @@ -397,16 +396,14 @@ } [Test] - [TestCase(false,false)] - [TestCase(false,true)] - [TestCase(true,false)] + [TestCase(false, false)] + [TestCase(false, true)] + [TestCase(true, false)] public void Validate_CalculationWithIncompleteDiameterD70Definition_LogsErrorAndReturnsFalse(bool meanSet, bool deviationSet) { // Setup const string name = ""; - PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); - var random = new Random(21); var incompletePipingSoilLayer = new PipingSoilLayer(5.0) { @@ -423,22 +420,22 @@ incompletePipingSoilLayer.DiameterD70Deviation = random.NextDouble(); } - calculation.InputParameters.StochasticSoilProfile.SoilProfile = new PipingSoilProfile( - string.Empty, - 0.0, - new[] - { - new PipingSoilLayer(10.5) - { - IsAquifer = false, - BelowPhreaticLevelDeviation = GetRandomDoubleFromRange(random, 1e-6, 999.999), - BelowPhreaticLevelMean = GetRandomDoubleFromRange(random, 10.0, 999.999), - BelowPhreaticLevelShift = GetRandomDoubleFromRange(random, 1e-6, 10.0) - }, - incompletePipingSoilLayer - }, - SoilProfileType.SoilProfile1D, - -1); + var profile = new PipingSoilProfile(string.Empty, 0.0, + new[] + { + new PipingSoilLayer(10.5) + { + IsAquifer = false, + BelowPhreaticLevelDeviation = GetRandomDoubleFromRange(random, 1e-6, 999.999), + BelowPhreaticLevelMean = GetRandomDoubleFromRange(random, 10.0, 999.999), + BelowPhreaticLevelShift = GetRandomDoubleFromRange(random, 1e-6, 10.0) + }, + incompletePipingSoilLayer + }, + SoilProfileType.SoilProfile1D, -1); + + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + calculation.InputParameters.StochasticSoilProfile.SoilProfile = profile; calculation.Name = name; bool isValid = false; @@ -459,16 +456,14 @@ } [Test] - [TestCase(false,false)] - [TestCase(false,true)] - [TestCase(true,false)] + [TestCase(false, false)] + [TestCase(false, true)] + [TestCase(true, false)] public void Validate_CalculationWithIncompletePermeabilityDefinition_LogsErrorAndReturnsFalse(bool meanSet, bool deviationSet) { // Setup const string name = ""; - PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); - var random = new Random(21); var incompletePipingSoilLayer = new PipingSoilLayer(5.0) { @@ -485,22 +480,22 @@ incompletePipingSoilLayer.PermeabilityDeviation = random.NextDouble(); } - calculation.InputParameters.StochasticSoilProfile.SoilProfile = new PipingSoilProfile( - string.Empty, - 0.0, - new[] - { - new PipingSoilLayer(10.5) - { - IsAquifer = false, - BelowPhreaticLevelDeviation = GetRandomDoubleFromRange(random, 1e-6, 999.999), - BelowPhreaticLevelMean = GetRandomDoubleFromRange(random, 10.0, 999.999), - BelowPhreaticLevelShift = GetRandomDoubleFromRange(random, 1e-6, 10.0) - }, - incompletePipingSoilLayer - }, - SoilProfileType.SoilProfile1D, - -1); + var profile = new PipingSoilProfile(string.Empty, 0.0, + new[] + { + new PipingSoilLayer(10.5) + { + IsAquifer = false, + BelowPhreaticLevelDeviation = GetRandomDoubleFromRange(random, 1e-6, 999.999), + BelowPhreaticLevelMean = GetRandomDoubleFromRange(random, 10.0, 999.999), + BelowPhreaticLevelShift = GetRandomDoubleFromRange(random, 1e-6, 10.0) + }, + incompletePipingSoilLayer + }, + SoilProfileType.SoilProfile1D, -1); + + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + calculation.InputParameters.StochasticSoilProfile.SoilProfile = profile; calculation.Name = name; bool isValid = false; @@ -521,17 +516,15 @@ } [Test] - [TestCase(false,false,false)] - [TestCase(false,true,true)] - [TestCase(true,false,true)] - [TestCase(true,true,false)] + [TestCase(false, false, false)] + [TestCase(false, true, true)] + [TestCase(true, false, true)] + [TestCase(true, true, false)] public void Validate_CalculationWithIncompletSaturatedVolumicWeightDefinition_LogsErrorAndReturnsFalse(bool meanSet, bool deviationSet, bool shiftSet) { // Setup const string name = ""; - PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); - var random = new Random(21); var incompletePipingSoilLayer = new PipingSoilLayer(10.5) { @@ -550,23 +543,23 @@ incompletePipingSoilLayer.BelowPhreaticLevelMean = 0.1 + incompletePipingSoilLayer.BelowPhreaticLevelShift + random.NextDouble(); } - calculation.InputParameters.StochasticSoilProfile.SoilProfile = new PipingSoilProfile( - string.Empty, - 0.0, - new[] - { - incompletePipingSoilLayer, - new PipingSoilLayer(5.0) - { - IsAquifer = true, - PermeabilityDeviation = random.NextDouble(), - PermeabilityMean = 0.1 + random.NextDouble(), - DiameterD70Deviation = random.NextDouble(), - DiameterD70Mean = 0.1 + random.NextDouble() - } - }, - SoilProfileType.SoilProfile1D, - -1); + var profile = new PipingSoilProfile(string.Empty, 0.0, + new[] + { + incompletePipingSoilLayer, + new PipingSoilLayer(5.0) + { + IsAquifer = true, + PermeabilityDeviation = random.NextDouble(), + PermeabilityMean = 0.1 + random.NextDouble(), + DiameterD70Deviation = random.NextDouble(), + DiameterD70Mean = 0.1 + random.NextDouble() + } + }, + SoilProfileType.SoilProfile1D, -1); + + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + calculation.InputParameters.StochasticSoilProfile.SoilProfile = profile; calculation.Name = name; bool isValid = false; @@ -592,41 +585,39 @@ // Setup const string name = ""; - PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); - var random = new Random(21); var belowPhreaticLevelDeviation = random.NextDouble(); var belowPhreaticLevelShift = random.NextDouble(); - calculation.InputParameters.StochasticSoilProfile.SoilProfile = new PipingSoilProfile( - string.Empty, - 0.0, - new[] - { - new PipingSoilLayer(10.5) - { - IsAquifer = false, - BelowPhreaticLevelDeviation = belowPhreaticLevelDeviation, - BelowPhreaticLevelShift = belowPhreaticLevelShift, - BelowPhreaticLevelMean = 0.1 + belowPhreaticLevelShift + random.NextDouble() - }, - new PipingSoilLayer(8.5) - { - IsAquifer = false, - BelowPhreaticLevelDeviation = belowPhreaticLevelDeviation, - BelowPhreaticLevelShift = belowPhreaticLevelShift, - BelowPhreaticLevelMean = 0.1 + belowPhreaticLevelShift + random.NextDouble() - }, - new PipingSoilLayer(5.0) - { - IsAquifer = true, - PermeabilityDeviation = random.NextDouble(), - PermeabilityMean = 0.1 + random.NextDouble(), - DiameterD70Deviation = random.NextDouble(), - DiameterD70Mean = 0.1 + random.NextDouble() - } - }, - SoilProfileType.SoilProfile1D, - -1); + var profile = new PipingSoilProfile(string.Empty, 0.0, + new[] + { + new PipingSoilLayer(10.5) + { + IsAquifer = false, + BelowPhreaticLevelDeviation = belowPhreaticLevelDeviation, + BelowPhreaticLevelShift = belowPhreaticLevelShift, + BelowPhreaticLevelMean = 0.1 + belowPhreaticLevelShift + random.NextDouble() + }, + new PipingSoilLayer(8.5) + { + IsAquifer = false, + BelowPhreaticLevelDeviation = belowPhreaticLevelDeviation, + BelowPhreaticLevelShift = belowPhreaticLevelShift, + BelowPhreaticLevelMean = 0.1 + belowPhreaticLevelShift + random.NextDouble() + }, + new PipingSoilLayer(5.0) + { + IsAquifer = true, + PermeabilityDeviation = random.NextDouble(), + PermeabilityMean = 0.1 + random.NextDouble(), + DiameterD70Deviation = random.NextDouble(), + DiameterD70Mean = 0.1 + random.NextDouble() + } + }, + SoilProfileType.SoilProfile1D, -1); + + PipingCalculation calculation = PipingCalculationFactory.CreateCalculationWithValidInput(); + calculation.InputParameters.StochasticSoilProfile.SoilProfile = profile; calculation.Name = name; bool isValid = false; @@ -753,35 +744,44 @@ var upliftCalculator = testFactory.LastCreatedUpliftCalculator; var sellmeijerCalculator = testFactory.LastCreatedSellmeijerCalculator; - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetThicknessCoverageLayer(input).GetDesignValue(), heaveCalculator.DTotal, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetThicknessCoverageLayer(input).GetDesignValue(), + heaveCalculator.DTotal, input.ThicknessCoverageLayer.GetAccuracy()); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), heaveCalculator.HExit, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), + heaveCalculator.HExit, input.PhreaticLevelExit.GetAccuracy()); Assert.AreEqual(input.CriticalHeaveGradient, heaveCalculator.Ich); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), heaveCalculator.PhiPolder, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), + heaveCalculator.PhiPolder, input.PhreaticLevelExit.GetAccuracy()); Assert.AreEqual(input.PiezometricHeadExit.Value, heaveCalculator.PhiExit); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDampingFactorExit(input).GetDesignValue(), heaveCalculator.RExit, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDampingFactorExit(input).GetDesignValue(), + heaveCalculator.RExit, input.DampingFactorExit.GetAccuracy()); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), upliftCalculator.HExit, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), + upliftCalculator.HExit, input.PhreaticLevelExit.GetAccuracy()); Assert.AreEqual(input.AssessmentLevel.Value, upliftCalculator.HRiver); Assert.AreEqual(input.UpliftModelFactor, upliftCalculator.ModelFactorUplift); Assert.AreEqual(input.PiezometricHeadExit.Value, upliftCalculator.PhiExit); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), upliftCalculator.PhiPolder, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), + upliftCalculator.PhiPolder, input.PhreaticLevelExit.GetAccuracy()); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDampingFactorExit(input).GetDesignValue(), upliftCalculator.RExit, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDampingFactorExit(input).GetDesignValue(), + upliftCalculator.RExit, input.DampingFactorExit.GetAccuracy()); Assert.AreEqual(input.WaterVolumetricWeight, upliftCalculator.VolumetricWeightOfWater); RoundedDouble effectiveStress = PipingSemiProbabilisticDesignValueFactory.GetThicknessCoverageLayer(input).GetDesignValue()* (PipingSemiProbabilisticDesignValueFactory.GetSaturatedVolumicWeightOfCoverageLayer(input).GetDesignValue() - input.WaterVolumetricWeight); Assert.AreEqual(effectiveStress, upliftCalculator.EffectiveStress, effectiveStress.GetAccuracy()); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetSeepageLength(input).GetDesignValue(), sellmeijerCalculator.SeepageLength, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetSeepageLength(input).GetDesignValue(), + sellmeijerCalculator.SeepageLength, input.SeepageLength.GetAccuracy()); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), sellmeijerCalculator.HExit, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetPhreaticLevelExit(input).GetDesignValue(), + sellmeijerCalculator.HExit, input.PhreaticLevelExit.GetAccuracy()); Assert.AreEqual(input.AssessmentLevel.Value, sellmeijerCalculator.HRiver); Assert.AreEqual(input.WaterKinematicViscosity, sellmeijerCalculator.KinematicViscosityWater); @@ -790,24 +790,27 @@ Assert.AreEqual(input.WaterVolumetricWeight, sellmeijerCalculator.VolumetricWeightOfWater); Assert.AreEqual(input.WhitesDragCoefficient, sellmeijerCalculator.WhitesDragCoefficient); Assert.AreEqual(input.BeddingAngle, sellmeijerCalculator.BeddingAngle); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetThicknessCoverageLayer(input).GetDesignValue(), sellmeijerCalculator.DTotal, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetThicknessCoverageLayer(input).GetDesignValue(), + sellmeijerCalculator.DTotal, input.ThicknessCoverageLayer.GetAccuracy()); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDiameter70(input).GetDesignValue(), sellmeijerCalculator.D70, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDiameter70(input).GetDesignValue(), + sellmeijerCalculator.D70, input.Diameter70.GetAccuracy()); Assert.AreEqual(input.MeanDiameter70, sellmeijerCalculator.D70Mean); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetThicknessAquiferLayer(input).GetDesignValue(), sellmeijerCalculator.DAquifer, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetThicknessAquiferLayer(input).GetDesignValue(), + sellmeijerCalculator.DAquifer, input.ThicknessAquiferLayer.GetAccuracy()); - Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDarcyPermeability(input).GetDesignValue(), sellmeijerCalculator.DarcyPermeability, + Assert.AreEqual(PipingSemiProbabilisticDesignValueFactory.GetDarcyPermeability(input).GetDesignValue(), + sellmeijerCalculator.DarcyPermeability, input.DarcyPermeability.GetAccuracy()); Assert.AreEqual(input.SandParticlesVolumicWeight, sellmeijerCalculator.GammaSubParticles); Assert.AreEqual(input.Gravity, sellmeijerCalculator.Gravity); } - private double GetRandomDoubleFromRange(Random random, double lowerLimit, double upperLimit) { double difference = upperLimit - lowerLimit; - return lowerLimit + random.NextDouble() * difference; + return lowerLimit + random.NextDouble()*difference; } } } \ No newline at end of file Index: Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingDataSynchronizationServiceTest.cs =================================================================== diff -u -rf7b5ed585321bbc2249ec9c8ecd8af7f9eb1808a -r8650820cbc2da7dcd34964bc92046fe66d14ec23 --- Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingDataSynchronizationServiceTest.cs (.../PipingDataSynchronizationServiceTest.cs) (revision f7b5ed585321bbc2249ec9c8ecd8af7f9eb1808a) +++ Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingDataSynchronizationServiceTest.cs (.../PipingDataSynchronizationServiceTest.cs) (revision 8650820cbc2da7dcd34964bc92046fe66d14ec23) @@ -48,17 +48,17 @@ { // Setup PipingFailureMechanism failureMechanism = new PipingFailureMechanism(); - PipingCalculation calculation1 = new PipingCalculation(new GeneralPipingInput()) + var calculation1 = new PipingCalculation(new GeneralPipingInput()) { Output = new TestPipingOutput() }; - - PipingCalculation calculation2 = new PipingCalculation(new GeneralPipingInput()) + + var calculation2 = new PipingCalculation(new GeneralPipingInput()) { Output = new TestPipingOutput() }; - PipingCalculation calculation3 = new PipingCalculation(new GeneralPipingInput()); + var calculation3 = new PipingCalculation(new GeneralPipingInput()); failureMechanism.CalculationsGroup.Children.Add(calculation1); failureMechanism.CalculationsGroup.Children.Add(calculation2); @@ -72,7 +72,11 @@ { Assert.IsNull(calculation.Output); } - CollectionAssert.AreEqual(new[] { calculation1, calculation2 }, affectedItems); + CollectionAssert.AreEqual(new[] + { + calculation1, + calculation2 + }, affectedItems); } [Test] @@ -90,7 +94,7 @@ public void ClearCalculationOutput_WithCalculation_ClearsOutput() { // Setup - PipingCalculation calculation = new PipingCalculation(new GeneralPipingInput()) + var calculation = new PipingCalculation(new GeneralPipingInput()) { Output = new TestPipingOutput() }; @@ -117,26 +121,26 @@ public void ClearHydraulicBoundaryLocations_WithHydraulicBoundaryLocation_ClearsHydraulicBoundaryLocationAndReturnsAffectedCalculations() { // Setup - PipingFailureMechanism failureMechanism = new PipingFailureMechanism(); - HydraulicBoundaryLocation hydraulicBoundaryLocation = new HydraulicBoundaryLocation(1, string.Empty, 0, 0); - - PipingCalculation calculation1 = new PipingCalculation(new GeneralPipingInput()) + var failureMechanism = new PipingFailureMechanism(); + var hydraulicBoundaryLocation = new HydraulicBoundaryLocation(1, string.Empty, 0, 0); + + var calculation1 = new PipingCalculation(new GeneralPipingInput()) { InputParameters = { HydraulicBoundaryLocation = hydraulicBoundaryLocation } }; - PipingCalculation calculation2 = new PipingCalculation(new GeneralPipingInput()) + var calculation2 = new PipingCalculation(new GeneralPipingInput()) { InputParameters = { HydraulicBoundaryLocation = hydraulicBoundaryLocation } }; - PipingCalculation calculation3 = new PipingCalculation(new GeneralPipingInput()); + var calculation3 = new PipingCalculation(new GeneralPipingInput()); failureMechanism.CalculationsGroup.Children.Add(calculation1); failureMechanism.CalculationsGroup.Children.Add(calculation2); @@ -172,10 +176,10 @@ public void ClearAllCalculationOutputAndHydraulicBoundaryLocations_CalculationsWithHydraulicBoundaryLocationAndOutput_ClearsHydraulicBoundaryLocationAndCalculationsAndReturnsAffectedCalculations() { // Setup - PipingFailureMechanism failureMechanism = new PipingFailureMechanism(); - HydraulicBoundaryLocation hydraulicBoundaryLocation = new HydraulicBoundaryLocation(1, string.Empty, 0, 0); + var failureMechanism = new PipingFailureMechanism(); + var hydraulicBoundaryLocation = new HydraulicBoundaryLocation(1, string.Empty, 0, 0); - PipingCalculation calculation1 = new PipingCalculation(new GeneralPipingInput()) + var calculation1 = new PipingCalculation(new GeneralPipingInput()) { InputParameters = { @@ -184,7 +188,7 @@ Output = new TestPipingOutput() }; - PipingCalculation calculation2 = new PipingCalculation(new GeneralPipingInput()) + var calculation2 = new PipingCalculation(new GeneralPipingInput()) { InputParameters = { @@ -193,7 +197,7 @@ Output = new TestPipingOutput() }; - PipingCalculation calculation3 = new PipingCalculation(new GeneralPipingInput()); + var calculation3 = new PipingCalculation(new GeneralPipingInput()); failureMechanism.CalculationsGroup.Children.Add(calculation1); failureMechanism.CalculationsGroup.Children.Add(calculation2); @@ -219,26 +223,26 @@ public void ClearAllCalculationOutputAndHydraulicBoundaryLocations_CalculationsWithHydraulicBoundaryLocationNoOutput_ClearsHydraulicBoundaryLocationAndReturnsAffectedCalculations() { // Setup - PipingFailureMechanism failureMechanism = new PipingFailureMechanism(); - HydraulicBoundaryLocation hydraulicBoundaryLocation = new HydraulicBoundaryLocation(1, string.Empty, 0, 0); + var failureMechanism = new PipingFailureMechanism(); + var hydraulicBoundaryLocation = new HydraulicBoundaryLocation(1, string.Empty, 0, 0); - PipingCalculation calculation1 = new PipingCalculation(new GeneralPipingInput()) + var calculation1 = new PipingCalculation(new GeneralPipingInput()) { InputParameters = { HydraulicBoundaryLocation = hydraulicBoundaryLocation } }; - PipingCalculation calculation2 = new PipingCalculation(new GeneralPipingInput()) + var calculation2 = new PipingCalculation(new GeneralPipingInput()) { InputParameters = { HydraulicBoundaryLocation = hydraulicBoundaryLocation } }; - PipingCalculation calculation3 = new PipingCalculation(new GeneralPipingInput()); + var calculation3 = new PipingCalculation(new GeneralPipingInput()); failureMechanism.CalculationsGroup.Children.Add(calculation1); failureMechanism.CalculationsGroup.Children.Add(calculation2); @@ -263,19 +267,19 @@ public void ClearAllCalculationOutputAndHydraulicBoundaryLocations_CalculationsWithOutputAndNoHydraulicBoundaryLocation_ClearsOuputAndReturnsAffectedCalculations() { // Setup - PipingFailureMechanism failureMechanism = new PipingFailureMechanism(); + var failureMechanism = new PipingFailureMechanism(); - PipingCalculation calculation1 = new PipingCalculation(new GeneralPipingInput()) + var calculation1 = new PipingCalculation(new GeneralPipingInput()) { Output = new TestPipingOutput() }; - PipingCalculation calculation2 = new PipingCalculation(new GeneralPipingInput()) + var calculation2 = new PipingCalculation(new GeneralPipingInput()) { Output = new TestPipingOutput() }; - PipingCalculation calculation3 = new PipingCalculation(new GeneralPipingInput()); + var calculation3 = new PipingCalculation(new GeneralPipingInput()); failureMechanism.CalculationsGroup.Children.Add(calculation1); failureMechanism.CalculationsGroup.Children.Add(calculation2); Index: Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingInputServiceTest.cs =================================================================== diff -u -r1781a4b8579c585777b5fb0097c20527f781d7de -r8650820cbc2da7dcd34964bc92046fe66d14ec23 --- Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingInputServiceTest.cs (.../PipingInputServiceTest.cs) (revision 1781a4b8579c585777b5fb0097c20527f781d7de) +++ Ringtoets/Piping/test/Ringtoets.Piping.Service.Test/PipingInputServiceTest.cs (.../PipingInputServiceTest.cs) (revision 8650820cbc2da7dcd34964bc92046fe66d14ec23) @@ -137,8 +137,10 @@ var soilModel = new StochasticSoilModel(1, "A", "B"); soilModel.StochasticSoilProfiles.Add(soilProfile); - var pipingInput = new PipingInput(new GeneralPipingInput()); - pipingInput.StochasticSoilModel = soilModel; + var pipingInput = new PipingInput(new GeneralPipingInput()) + { + StochasticSoilModel = soilModel + }; // Call PipingInputService.SyncStochasticSoilProfileWithStochasticSoilModel(pipingInput); @@ -157,8 +159,10 @@ new StochasticSoilProfile(0.0, SoilProfileType.SoilProfile1D, 1), new StochasticSoilProfile(1.1, SoilProfileType.SoilProfile1D, 2) }); - var pipingInput = new PipingInput(new GeneralPipingInput()); - pipingInput.StochasticSoilModel = soilModel; + var pipingInput = new PipingInput(new GeneralPipingInput()) + { + StochasticSoilModel = soilModel + }; // Call PipingInputService.SyncStochasticSoilProfileWithStochasticSoilModel(pipingInput); @@ -176,9 +180,11 @@ var soilModel = new StochasticSoilModel(1, "A", "B"); soilModel.StochasticSoilProfiles.Add(soilProfile); - var pipingInput = new PipingInput(new GeneralPipingInput()); - pipingInput.StochasticSoilModel = soilModel; - pipingInput.StochasticSoilProfile = soilProfile; + var pipingInput = new PipingInput(new GeneralPipingInput()) + { + StochasticSoilModel = soilModel, + StochasticSoilProfile = soilProfile + }; // Call PipingInputService.SyncStochasticSoilProfileWithStochasticSoilModel(pipingInput);