Index: Riskeer/Piping/src/Riskeer.Piping.Plugin/PipingPlugin.cs =================================================================== diff -u -ra13bd1b93f177cf300337fd36b3a6ac3b948ecce -r40aef93935efc6585533d5885a20a6f3524e9875 --- Riskeer/Piping/src/Riskeer.Piping.Plugin/PipingPlugin.cs (.../PipingPlugin.cs) (revision a13bd1b93f177cf300337fd36b3a6ac3b948ecce) +++ Riskeer/Piping/src/Riskeer.Piping.Plugin/PipingPlugin.cs (.../PipingPlugin.cs) (revision 40aef93935efc6585533d5885a20a6f3524e9875) @@ -1314,7 +1314,7 @@ /// The calculations to validate. /// The general input to use during the validation. /// The assessment section the belong to. - /// The failure mechanism to use during the validation. + /// The failure mechanism the belong to. /// Thrown when any of the provided calculations is of a type that is not supported. private static void ValidateAll(IEnumerable> pipingCalculations, GeneralPipingInput generalPipingInput, Index: Riskeer/Piping/test/Riskeer.Piping.Service.Test/Probabilistic/ProbabilisticPipingCalculationServiceTest.cs =================================================================== diff -u -r50df1781e18d8f7077e4247e8b71c2e0c9e6d55d -r40aef93935efc6585533d5885a20a6f3524e9875 --- Riskeer/Piping/test/Riskeer.Piping.Service.Test/Probabilistic/ProbabilisticPipingCalculationServiceTest.cs (.../ProbabilisticPipingCalculationServiceTest.cs) (revision 50df1781e18d8f7077e4247e8b71c2e0c9e6d55d) +++ Riskeer/Piping/test/Riskeer.Piping.Service.Test/Probabilistic/ProbabilisticPipingCalculationServiceTest.cs (.../ProbabilisticPipingCalculationServiceTest.cs) (revision 40aef93935efc6585533d5885a20a6f3524e9875) @@ -116,37 +116,37 @@ } [Test] - public void Validate_AssessmentSectionNull_ThrowsArgumentNullException() + public void Validate_FailureMechanismNull_ThrowsArgumentNullException() { + // Setup + var mocks = new MockRepository(); + var assessmentSection = mocks.Stub(); + mocks.ReplayAll(); + // Call void Call() => ProbabilisticPipingCalculationService.Validate(new TestProbabilisticPipingCalculation(), new GeneralPipingInput(), - null, - new PipingFailureMechanism()); + assessmentSection, + null); // Assert var exception = Assert.Throws(Call); - Assert.AreEqual("assessmentSection", exception.ParamName); + Assert.AreEqual("failureMechanism", exception.ParamName); + mocks.VerifyAll(); } [Test] - public void ValidateFailureMechanismNull_ThrowsArgumentNullException() + public void Validate_AssessmentSectionNull_ThrowsArgumentNullException() { - // Setup - var mocks = new MockRepository(); - var assessmentSection = mocks.Stub(); - mocks.ReplayAll(); - // Call void Call() => ProbabilisticPipingCalculationService.Validate(new TestProbabilisticPipingCalculation(), new GeneralPipingInput(), - assessmentSection, - null); + null, + new PipingFailureMechanism()); // Assert var exception = Assert.Throws(Call); - Assert.AreEqual("failureMechanism", exception.ParamName); - mocks.VerifyAll(); + Assert.AreEqual("assessmentSection", exception.ParamName); } [Test] @@ -1012,7 +1012,7 @@ } [Test] - public void Validate_CalculationWithoutSection_LogsMessage() + public void Validate_FailureMechanismWithoutSections_LogsMessageAndReturnsFalse() { // Setup var failureMechanism = new PipingFailureMechanism(); @@ -1044,7 +1044,7 @@ } [Test] - public void Validate_CalculationInMultipleSections_LogsMessageOnlyForDuplicateSection() + public void Validate_CalculationInMultipleSections_LogsMessageAndReturnsFalse() { // Setup var failureMechanism = new PipingFailureMechanism();