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();