Index: Riskeer/ClosingStructures/test/Riskeer.ClosingStructures.Forms.Test/Views/ClosingStructuresFailureMechanismSectionResultRowTest.cs =================================================================== diff -u -rcdf449f6832d8dc9122ae1452dccbb66c1d6c596 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/ClosingStructures/test/Riskeer.ClosingStructures.Forms.Test/Views/ClosingStructuresFailureMechanismSectionResultRowTest.cs (.../ClosingStructuresFailureMechanismSectionResultRowTest.cs) (revision cdf449f6832d8dc9122ae1452dccbb66c1d6c596) +++ Riskeer/ClosingStructures/test/Riskeer.ClosingStructures.Forms.Test/Views/ClosingStructuresFailureMechanismSectionResultRowTest.cs (.../ClosingStructuresFailureMechanismSectionResultRowTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -750,7 +750,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -790,7 +790,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -834,7 +834,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -871,7 +871,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -915,7 +915,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } Index: Riskeer/Common/src/Riskeer.Common.Forms/Helpers/FailureMechanismSectionResultRowHelper.cs =================================================================== diff -u -r822617da9d315fb1ee30a0a3861f53be154d6371 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/Common/src/Riskeer.Common.Forms/Helpers/FailureMechanismSectionResultRowHelper.cs (.../FailureMechanismSectionResultRowHelper.cs) (revision 822617da9d315fb1ee30a0a3861f53be154d6371) +++ Riskeer/Common/src/Riskeer.Common.Forms/Helpers/FailureMechanismSectionResultRowHelper.cs (.../FailureMechanismSectionResultRowHelper.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -40,34 +40,6 @@ public static class FailureMechanismSectionResultRowHelper { /// - /// Gets the error text to display when the detailed assessment fails. - /// - /// The value representing the probability of the detailed assessment. - /// The set for the - /// section result. May be null if the section result does not have a calculation set. - /// The error message when an error is present; otherwise. - public static string GetDetailedAssessmentError(double detailedAssessmentProbability, - ICalculation normativeCalculation) - { - if (normativeCalculation == null) - { - return Resources.FailureMechanismResultView_DataGridViewCellFormatting_Calculation_not_set; - } - - if (!normativeCalculation.HasOutput) - { - return Resources.FailureMechanismResultView_DataGridViewCellFormatting_Calculation_not_calculated; - } - - if (double.IsNaN(detailedAssessmentProbability)) - { - return Resources.FailureMechanismResultView_DataGridViewCellFormatting_Calculation_must_have_valid_output; - } - - return string.Empty; - } - - /// /// Gets the error text to display when the detailed assessment probability fails. /// /// The type of . Index: Riskeer/Common/src/Riskeer.Common.Forms/Properties/Resources.Designer.cs =================================================================== diff -u -rba47735ba43eefc2a0d69d7cef9432614d05b7a6 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/Common/src/Riskeer.Common.Forms/Properties/Resources.Designer.cs (.../Resources.Designer.cs) (revision ba47735ba43eefc2a0d69d7cef9432614d05b7a6) +++ Riskeer/Common/src/Riskeer.Common.Forms/Properties/Resources.Designer.cs (.../Resources.Designer.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -1736,7 +1736,7 @@ } /// - /// Looks up a localized string similar to Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.. + /// Looks up a localized string similar to Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.. /// public static string FailureMechanismResultView_DataGridViewCellFormatting_All_calculations_must_have_valid_output { get { @@ -1756,26 +1756,8 @@ } /// - /// Looks up a localized string similar to De maatgevende berekening voor dit vak moet nog worden uitgevoerd.. + /// Looks up a localized string similar to Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.. /// - public static string FailureMechanismResultView_DataGridViewCellFormatting_Calculation_not_calculated { - get { - return ResourceManager.GetString("FailureMechanismResultView_DataGridViewCellFormatting_Calculation_not_calculated", resourceCulture); - } - } - - /// - /// Looks up a localized string similar to Er moet een maatgevende berekening voor dit vak worden geselecteerd.. - /// - public static string FailureMechanismResultView_DataGridViewCellFormatting_Calculation_not_set { - get { - return ResourceManager.GetString("FailureMechanismResultView_DataGridViewCellFormatting_Calculation_not_set", resourceCulture); - } - } - - /// - /// Looks up a localized string similar to Alle berekeningen voor dit vak moeten uitgevoerd zijn.. - /// public static string FailureMechanismResultView_DataGridViewCellFormatting_Not_all_calculations_have_been_executed { get { return ResourceManager.GetString("FailureMechanismResultView_DataGridViewCellFormatting_Not_all_calculations_have_b" + @@ -1784,7 +1766,7 @@ } /// - /// Looks up a localized string similar to Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.. + /// Looks up a localized string similar to Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.. /// public static string FailureMechanismResultView_DataGridViewCellFormatting_Not_any_calculation_set { get { @@ -1793,7 +1775,7 @@ } /// - /// Looks up a localized string similar to Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.. + /// Looks up a localized string similar to De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.. /// public static string FailureMechanismResultView_DataGridViewCellFormatting_Scenario_contribution_for_this_section_not_100 { get { Index: Riskeer/Common/src/Riskeer.Common.Forms/Properties/Resources.resx =================================================================== diff -u -rba47735ba43eefc2a0d69d7cef9432614d05b7a6 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/Common/src/Riskeer.Common.Forms/Properties/Resources.resx (.../Resources.resx) (revision ba47735ba43eefc2a0d69d7cef9432614d05b7a6) +++ Riskeer/Common/src/Riskeer.Common.Forms/Properties/Resources.resx (.../Resources.resx) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -285,13 +285,13 @@ ..\Resources\document-task.png;System.Drawing.Bitmap, System.Drawing, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a - Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%. + De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%. - Alle berekeningen voor dit vak moeten uitgevoerd zijn. + Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn. - Alle berekeningen voor dit vak moeten een geldige uitkomst hebben. + Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben. - @@ -863,17 +863,11 @@ Dijkprofielen - - De maatgevende berekening voor dit vak moet nog worden uitgevoerd. - - - Er moet een maatgevende berekening voor dit vak worden geselecteerd. - De maatgevende berekening voor dit vak moet een geldige uitkomst hebben. - Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd. + Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd. Koppelingsafstand Index: Riskeer/Common/test/Riskeer.Common.Forms.Test/Helpers/FailureMechanismSectionResultRowHelperTest.cs =================================================================== diff -u -r7ca9e70221e813bd346b89788b8ea9ab4c7c5c35 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/Common/test/Riskeer.Common.Forms.Test/Helpers/FailureMechanismSectionResultRowHelperTest.cs (.../FailureMechanismSectionResultRowHelperTest.cs) (revision 7ca9e70221e813bd346b89788b8ea9ab4c7c5c35) +++ Riskeer/Common/test/Riskeer.Common.Forms.Test/Helpers/FailureMechanismSectionResultRowHelperTest.cs (.../FailureMechanismSectionResultRowHelperTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -20,7 +20,6 @@ // All rights reserved. using System; -using System.Collections.Generic; using System.ComponentModel; using System.Drawing; using Core.Common.Controls.DataGrid; @@ -29,7 +28,6 @@ using Rhino.Mocks; using Riskeer.AssemblyTool.Data; using Riskeer.Common.Data.Calculation; -using Riskeer.Common.Data.TestUtil; using Riskeer.Common.Forms.Helpers; using Riskeer.Common.Forms.TestUtil; using Riskeer.Common.Primitives; @@ -40,22 +38,6 @@ public class FailureMechanismSectionResultRowHelperTest { [Test] - [TestCaseSource(nameof(GetCalculationNullConfigurations))] - [TestCaseSource(nameof(GetCalculationWithoutOutputConfigurations))] - [TestCaseSource(nameof(GetCalculationWithOutputConfigurations))] - public void GetAssessmentDetailedAssessmentError_WithVariousConfigurations_GetsExpectedErrorText(double detailedAssessmentResult, - ICalculation normativeCalculation, - string expectedErrorText) - { - // Call - string errorText = FailureMechanismSectionResultRowHelper.GetDetailedAssessmentError(detailedAssessmentResult, - normativeCalculation); - - // Assert - Assert.AreEqual(expectedErrorText, errorText); - } - - [Test] public void GetDetailedAssessmentProbabilityError_RelevantScenariosNull_ThrowsArgumentNullException() { // Call @@ -102,7 +84,7 @@ new ICalculationScenario[0], scenarios => random.NextDouble(), scenarios => random.NextDouble()); // Assert - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", errorMessage); + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", errorMessage); } [Test] @@ -124,7 +106,7 @@ calculationScenarios, scenarios => random.NextDouble(0, 0.99), scenarios => random.NextDouble()); // Assert - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", errorMessage); + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", errorMessage); } [Test] @@ -149,7 +131,7 @@ calculationScenarios, scenarios => 1, scenarios => random.NextDouble()); // Assert - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", errorMessage); + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", errorMessage); } [Test] @@ -171,7 +153,7 @@ calculationScenarios, scenarios => 1, scenarios => double.NaN); // Assert - Assert.AreEqual("Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.", errorMessage); + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.", errorMessage); } [Test] @@ -275,12 +257,11 @@ public void SetAssemblyCategoryGroupStyle_DataGridViewColumnStateDefinitionNull_ThrowsArgumentNullException() { // Call - TestDelegate test = () => FailureMechanismSectionResultRowHelper.SetAssemblyCategoryGroupStyle( - null, - new Random(39).NextEnumValue()); + void Call() => FailureMechanismSectionResultRowHelper.SetAssemblyCategoryGroupStyle( + null, new Random(39).NextEnumValue()); // Assert - var exception = Assert.Throws(test); + var exception = Assert.Throws(Call); Assert.AreEqual("columnStateDefinition", exception.ParamName); } @@ -294,12 +275,12 @@ }; // Call - TestDelegate test = () => FailureMechanismSectionResultRowHelper.SetAssemblyCategoryGroupStyle(columnStateDefinition, - (FailureMechanismSectionAssemblyCategoryGroup) 99); + void Call() => FailureMechanismSectionResultRowHelper.SetAssemblyCategoryGroupStyle( + columnStateDefinition, (FailureMechanismSectionAssemblyCategoryGroup) 99); // Assert const string expectedMessage = "The value of argument 'assemblyCategoryGroup' (99) is invalid for Enum type 'FailureMechanismSectionAssemblyCategoryGroup'."; - TestHelper.AssertThrowsArgumentExceptionAndTestMessage(test, expectedMessage); + TestHelper.AssertThrowsArgumentExceptionAndTestMessage(Call, expectedMessage); } [Test] @@ -317,51 +298,5 @@ Assert.AreEqual(expectedColor, columnStateDefinition.Style.BackgroundColor); Assert.AreEqual(Color.FromKnownColor(KnownColor.ControlText), columnStateDefinition.Style.TextColor); } - - #region Test cases - - private static IEnumerable GetCalculationNullConfigurations() - { - const string expectedErrorMessage = "Er moet een maatgevende berekening voor dit vak worden geselecteerd."; - - yield return new TestCaseData(double.NaN, null, - expectedErrorMessage) - .SetName("InvalidDetailedAssessmentAndNoCalculation"); - yield return new TestCaseData(0.0, null, - expectedErrorMessage) - .SetName("ValidDetailedAssessmentAndNoCalculation"); - } - - private static IEnumerable GetCalculationWithoutOutputConfigurations() - { - const string expectedErrorMessage = "De maatgevende berekening voor dit vak moet nog worden uitgevoerd."; - - yield return new TestCaseData(double.NaN, - CalculationTestDataFactory.CreateCalculationWithoutOutput(), - expectedErrorMessage) - .SetName("InvalidDetailedAssessmentAndCalculationWithoutOutput"); - yield return new TestCaseData(0.0, - CalculationTestDataFactory.CreateCalculationWithoutOutput(), - expectedErrorMessage) - .SetName("ValidDetailedAssessmentAndCalculationWithoutOutput"); - } - - private static IEnumerable GetCalculationWithOutputConfigurations() - { - const string expectedErrorMessageOutputInvalid = "De maatgevende berekening voor dit vak moet een geldige uitkomst hebben."; - string expectedEmptyErrorMessage = string.Empty; - - yield return new TestCaseData(double.NaN, - CalculationTestDataFactory.CreateCalculationWithOutput(), - expectedErrorMessageOutputInvalid) - .SetName("InvalidDetailedAssessmentAndCalculationWithOutput"); - - yield return new TestCaseData(0.0, - CalculationTestDataFactory.CreateCalculationWithOutput(), - expectedEmptyErrorMessage) - .SetName("ValidDetailedAssessmentAndCalculationWithOutput"); - } - - #endregion } } \ No newline at end of file Index: Riskeer/GrassCoverErosionInwards/test/Riskeer.GrassCoverErosionInwards.Forms.Test/Views/GrassCoverErosionInwardsFailureMechanismSectionResultRowTest.cs =================================================================== diff -u -rcdf449f6832d8dc9122ae1452dccbb66c1d6c596 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/GrassCoverErosionInwards/test/Riskeer.GrassCoverErosionInwards.Forms.Test/Views/GrassCoverErosionInwardsFailureMechanismSectionResultRowTest.cs (.../GrassCoverErosionInwardsFailureMechanismSectionResultRowTest.cs) (revision cdf449f6832d8dc9122ae1452dccbb66c1d6c596) +++ Riskeer/GrassCoverErosionInwards/test/Riskeer.GrassCoverErosionInwards.Forms.Test/Views/GrassCoverErosionInwardsFailureMechanismSectionResultRowTest.cs (.../GrassCoverErosionInwardsFailureMechanismSectionResultRowTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -754,7 +754,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -794,7 +794,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -838,7 +838,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -875,7 +875,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -918,7 +918,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } Index: Riskeer/HeightStructures/test/Riskeer.HeightStructures.Forms.Test/Views/HeightStructuresFailureMechanismSectionResultRowTest.cs =================================================================== diff -u -rcdf449f6832d8dc9122ae1452dccbb66c1d6c596 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/HeightStructures/test/Riskeer.HeightStructures.Forms.Test/Views/HeightStructuresFailureMechanismSectionResultRowTest.cs (.../HeightStructuresFailureMechanismSectionResultRowTest.cs) (revision cdf449f6832d8dc9122ae1452dccbb66c1d6c596) +++ Riskeer/HeightStructures/test/Riskeer.HeightStructures.Forms.Test/Views/HeightStructuresFailureMechanismSectionResultRowTest.cs (.../HeightStructuresFailureMechanismSectionResultRowTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -707,7 +707,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -747,7 +747,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -791,7 +791,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -828,7 +828,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -872,7 +872,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } Index: Riskeer/MacroStabilityInwards/test/Riskeer.MacroStabilityInwards.Forms.Test/Views/MacroStabilityInwardsFailureMechanismSectionResultRowTest.cs =================================================================== diff -u -rcdf449f6832d8dc9122ae1452dccbb66c1d6c596 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/MacroStabilityInwards/test/Riskeer.MacroStabilityInwards.Forms.Test/Views/MacroStabilityInwardsFailureMechanismSectionResultRowTest.cs (.../MacroStabilityInwardsFailureMechanismSectionResultRowTest.cs) (revision cdf449f6832d8dc9122ae1452dccbb66c1d6c596) +++ Riskeer/MacroStabilityInwards/test/Riskeer.MacroStabilityInwards.Forms.Test/Views/MacroStabilityInwardsFailureMechanismSectionResultRowTest.cs (.../MacroStabilityInwardsFailureMechanismSectionResultRowTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -753,7 +753,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -793,7 +793,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -837,7 +837,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -874,7 +874,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -914,7 +914,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } Index: Riskeer/MacroStabilityInwards/test/Riskeer.MacroStabilityInwards.Integration.Test/MacroStabilityInwardsFailureMechanismResultViewIntegrationTest.cs =================================================================== diff -u -r77f595651b934a75a5625828a9027d7c48e8e5f9 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/MacroStabilityInwards/test/Riskeer.MacroStabilityInwards.Integration.Test/MacroStabilityInwardsFailureMechanismResultViewIntegrationTest.cs (.../MacroStabilityInwardsFailureMechanismResultViewIntegrationTest.cs) (revision 77f595651b934a75a5625828a9027d7c48e8e5f9) +++ Riskeer/MacroStabilityInwards/test/Riskeer.MacroStabilityInwards.Integration.Test/MacroStabilityInwardsFailureMechanismResultViewIntegrationTest.cs (.../MacroStabilityInwardsFailureMechanismResultViewIntegrationTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -65,7 +65,7 @@ DataImportHelper.ImportFailureMechanismSections(assessmentSection, failureMechanism); Assert.AreEqual(283, dataGridView.Rows.Count); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Import surface lines @@ -93,29 +93,29 @@ assessmentSection.MacroStabilityInwards.CalculationsGroup.Children.Add(calculation1); assessmentSection.MacroStabilityInwards.CalculationsGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Add group and ensure the data grid view is not changed var nestedCalculationGroup = new CalculationGroup(); assessmentSection.MacroStabilityInwards.CalculationsGroup.Children.Add(nestedCalculationGroup); assessmentSection.MacroStabilityInwards.CalculationsGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Add another, nested calculation and ensure the data grid view is updated nestedCalculationGroup.Children.Add(calculation2); nestedCalculationGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Set the second calculation to not relevant and ensure the data grid view is updated calculation2.IsRelevant = false; calculation2.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Execute the first calculation and ensure the data grid view is updated @@ -144,27 +144,27 @@ sl => sl.Name == "PK001_0001"); calculation3.InputParameters.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Change the contribution of the calculation and make sure the data grid view is updated calculation3.Contribution = (RoundedDouble) 0.3; calculation3.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); calculation1.Contribution = (RoundedDouble) 0.7; calculation1.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Remove a calculation and make sure the data grid view is updated nestedCalculationGroup.Children.Remove(calculation3); nestedCalculationGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Set contribution again so we have a probability. @@ -178,7 +178,7 @@ MacroStabilityInwardsDataSynchronizationService.ClearCalculationOutput(calculation1); calculation1.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); } } Index: Riskeer/Piping/test/Riskeer.Piping.Forms.Test/Views/PipingFailureMechanismSectionResultRowTest.cs =================================================================== diff -u -rcdf449f6832d8dc9122ae1452dccbb66c1d6c596 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/Piping/test/Riskeer.Piping.Forms.Test/Views/PipingFailureMechanismSectionResultRowTest.cs (.../PipingFailureMechanismSectionResultRowTest.cs) (revision cdf449f6832d8dc9122ae1452dccbb66c1d6c596) +++ Riskeer/Piping/test/Riskeer.Piping.Forms.Test/Views/PipingFailureMechanismSectionResultRowTest.cs (.../PipingFailureMechanismSectionResultRowTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -752,7 +752,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -792,7 +792,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -836,7 +836,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -873,7 +873,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -913,7 +913,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } Index: Riskeer/Piping/test/Riskeer.Piping.Integration.Test/PipingFailureMechanismResultViewIntegrationTest.cs =================================================================== diff -u -r9c297dba6739032bcaddfb5ceb3452082c54db4a -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/Piping/test/Riskeer.Piping.Integration.Test/PipingFailureMechanismResultViewIntegrationTest.cs (.../PipingFailureMechanismResultViewIntegrationTest.cs) (revision 9c297dba6739032bcaddfb5ceb3452082c54db4a) +++ Riskeer/Piping/test/Riskeer.Piping.Integration.Test/PipingFailureMechanismResultViewIntegrationTest.cs (.../PipingFailureMechanismResultViewIntegrationTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -67,7 +67,7 @@ DataImportHelper.ImportFailureMechanismSections(assessmentSection, failureMechanism); Assert.AreEqual(283, dataGridView.Rows.Count); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Import surface lines @@ -95,29 +95,29 @@ assessmentSection.Piping.CalculationsGroup.Children.Add(pipingCalculation1); assessmentSection.Piping.CalculationsGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Add group and ensure the data grid view is not changed var nestedPipingCalculationGroup = new CalculationGroup(); assessmentSection.Piping.CalculationsGroup.Children.Add(nestedPipingCalculationGroup); assessmentSection.Piping.CalculationsGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Add another, nested calculation and ensure the data grid view is updated nestedPipingCalculationGroup.Children.Add(pipingCalculation2); nestedPipingCalculationGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Set the second calculation to not relevant and ensure the data grid view is updated pipingCalculation2.IsRelevant = false; pipingCalculation2.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Execute the first calculation and ensure the data grid view is updated @@ -139,27 +139,27 @@ sl => sl.Name == "PK001_0001"); pipingCalculation3.InputParameters.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Change the contribution of the calculation and make sure the data grid view is updated pipingCalculation3.Contribution = (RoundedDouble) 0.3; pipingCalculation3.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); pipingCalculation1.Contribution = (RoundedDouble) 0.7; pipingCalculation1.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Remove a calculation and make sure the data grid view is updated nestedPipingCalculationGroup.Children.Remove(pipingCalculation3); nestedPipingCalculationGroup.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); // Set contribution again so we have a probability. @@ -173,7 +173,7 @@ PipingDataSynchronizationService.ClearCalculationOutput(pipingCalculation1); pipingCalculation1.NotifyObservers(); Assert.AreEqual("-", dataGridView.Rows[22].Cells[detailedAssessmentIndex].FormattedValue); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", dataGridView.Rows[22].Cells[detailedAssessmentIndex].ErrorText); } } Index: Riskeer/StabilityPointStructures/test/Riskeer.StabilityPointStructures.Forms.Test/Views/StabilityPointStructuresFailureMechanismSectionResultRowTest.cs =================================================================== diff -u -rcdf449f6832d8dc9122ae1452dccbb66c1d6c596 -r8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b --- Riskeer/StabilityPointStructures/test/Riskeer.StabilityPointStructures.Forms.Test/Views/StabilityPointStructuresFailureMechanismSectionResultRowTest.cs (.../StabilityPointStructuresFailureMechanismSectionResultRowTest.cs) (revision cdf449f6832d8dc9122ae1452dccbb66c1d6c596) +++ Riskeer/StabilityPointStructures/test/Riskeer.StabilityPointStructures.Forms.Test/Views/StabilityPointStructuresFailureMechanismSectionResultRowTest.cs (.../StabilityPointStructuresFailureMechanismSectionResultRowTest.cs) (revision 8a9b9f84b0709f99bf5d8589c0ca41eda5f1b47b) @@ -751,7 +751,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Bijdrage van de geselecteerde scenario's voor dit vak moet opgeteld gelijk zijn aan 100%.", + Assert.AreEqual("De bijdragen van de maatgevende scenario's voor dit vak moeten opgeteld gelijk zijn aan 100%.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -791,7 +791,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten uitgevoerd zijn.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten uitgevoerd zijn.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -835,7 +835,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Alle berekeningen voor dit vak moeten een geldige uitkomst hebben.", + Assert.AreEqual("Alle maatgevende berekeningen voor dit vak moeten een geldige uitkomst hebben.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -872,7 +872,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); } @@ -916,7 +916,7 @@ // Assert Assert.IsNaN(resultRow.DetailedAssessmentProbability); - Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden geselecteerd.", + Assert.AreEqual("Er moet minimaal één maatgevende berekening voor dit vak worden gedefinieerd.", resultRow.ColumnStateDefinitions[ConstructionProperties.DetailedAssessmentProbabilityIndex].ErrorText); mocks.VerifyAll(); }