Index: Riskeer/Migration/src/Riskeer.Migration.Core/EmbeddedResources/Migration_23.1_23.2.sql =================================================================== diff -u -r6f9cd6be760c019f10a36465f0b860edc6e0c947 -r87d347be1c4afa0bfd1311c193364c49cfa73b60 --- Riskeer/Migration/src/Riskeer.Migration.Core/EmbeddedResources/Migration_23.1_23.2.sql (.../Migration_23.1_23.2.sql) (revision 6f9cd6be760c019f10a36465f0b860edc6e0c947) +++ Riskeer/Migration/src/Riskeer.Migration.Core/EmbeddedResources/Migration_23.1_23.2.sql (.../Migration_23.1_23.2.sql) (revision 87d347be1c4afa0bfd1311c193364c49cfa73b60) @@ -614,6 +614,17 @@ WHERE [NrDeleted] > 0 LIMIT 1; +INSERT INTO [LOGDATABASE].MigrationLogEntity ( + [FromVersion], + [ToVersion], +[LogMessage]) +VALUES ("23.1", + "23.2", + "* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast: + + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.' + + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.' + + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'"); + CREATE TEMP TABLE TempFailureMechanisms ( 'FailureMechanismType' INTEGER NOT NULL, Index: Riskeer/Migration/test/Riskeer.Migration.Integration.Test/MigrationIntegrationTest.cs =================================================================== diff -u -r786a8396c99eedb03a0f98445f9378cb7e4b6234 -r87d347be1c4afa0bfd1311c193364c49cfa73b60 --- Riskeer/Migration/test/Riskeer.Migration.Integration.Test/MigrationIntegrationTest.cs (.../MigrationIntegrationTest.cs) (revision 786a8396c99eedb03a0f98445f9378cb7e4b6234) +++ Riskeer/Migration/test/Riskeer.Migration.Integration.Test/MigrationIntegrationTest.cs (.../MigrationIntegrationTest.cs) (revision 87d347be1c4afa0bfd1311c193364c49cfa73b60) @@ -19,6 +19,7 @@ // Stichting Deltares and remain full property of Stichting Deltares at all times. // All rights reserved. +using System; using System.Collections.Generic; using System.Collections.ObjectModel; using System.IO; @@ -64,10 +65,10 @@ using (new FileDisposeHelper(targetFilePath)) { // When - TestDelegate call = () => migrator.Migrate(fromVersionedFile, newVersion, targetFilePath); + void Call() => migrator.Migrate(fromVersionedFile, newVersion, targetFilePath); // Then - Assert.DoesNotThrow(call); + Assert.DoesNotThrow(Call); } } finally @@ -143,10 +144,10 @@ new MigrationLogMessage("21.1", "22.1", "Gevolgen van de migratie van versie 21.1 naar versie 22.1:"), messages[12]); MigrationLogTestHelper.AssertMigrationLogMessageEqual( - new MigrationLogMessage("21.1", "22.1", "* De oorspronkelijke faalmechanismen zijn omgezet naar het nieuwe formaat.\r\n* Alle toetsoordelen zijn verwijderd."), + new MigrationLogMessage("21.1", "22.1", $"* De oorspronkelijke faalmechanismen zijn omgezet naar het nieuwe formaat.{Environment.NewLine}* Alle toetsoordelen zijn verwijderd."), messages[13]); MigrationLogTestHelper.AssertMigrationLogMessageEqual( - new MigrationLogMessage("22.1", "23.1", $"Gevolgen van de migratie van versie 22.1 naar versie 23.1:"), + new MigrationLogMessage("22.1", "23.1", "Gevolgen van de migratie van versie 22.1 naar versie 23.1:"), messages[14]); MigrationLogTestHelper.AssertMigrationLogMessageEqual( new MigrationLogMessage("22.1", "23.1", "* Geen aanpassingen."), @@ -155,7 +156,11 @@ new MigrationLogMessage("23.1", $"{latestVersion}", $"Gevolgen van de migratie van versie 23.1 naar versie {latestVersion}:"), messages[16]); MigrationLogTestHelper.AssertMigrationLogMessageEqual( - new MigrationLogMessage("23.1", $"{latestVersion}", "* Geen aanpassingen."), + new MigrationLogMessage("23.1", $"{latestVersion}", + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'"), messages[17]); } } Index: Riskeer/Migration/test/Riskeer.Migration.Integration.Test/MigrationTo232IntegrationTest.cs =================================================================== diff -u -r5eb941c2618bdd156d06a21d605a74df9b1332ec -r87d347be1c4afa0bfd1311c193364c49cfa73b60 --- Riskeer/Migration/test/Riskeer.Migration.Integration.Test/MigrationTo232IntegrationTest.cs (.../MigrationTo232IntegrationTest.cs) (revision 5eb941c2618bdd156d06a21d605a74df9b1332ec) +++ Riskeer/Migration/test/Riskeer.Migration.Integration.Test/MigrationTo232IntegrationTest.cs (.../MigrationTo232IntegrationTest.cs) (revision 87d347be1c4afa0bfd1311c193364c49cfa73b60) @@ -19,6 +19,7 @@ // Stichting Deltares and remain full property of Stichting Deltares at all times. // All rights reserved. +using System; using System.Collections.Generic; using System.Collections.ObjectModel; using System.Linq; @@ -108,44 +109,73 @@ { yield return new TestCaseData("MigrationTestProject231NoOutput.risk", new[] { - "* Geen aanpassingen." - }); - + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" + }); + yield return new TestCaseData("MigrationTestProject231VariousFailureMechanismSectionResultConfigurations.risk", new[] { - "* Geen aanpassingen." + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" }); yield return new TestCaseData("MigrationTestProject231MacroStabilityInwardsNoManualAssessmentLevels.risk", new[] { - "* Alle berekende resultaten zijn verwijderd." + "* Alle berekende resultaten zijn verwijderd.", + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" }); yield return new TestCaseData("MigrationTestProject231PipingNoManualAssessmentLevels.risk", new[] { - "* Alle berekende resultaten zijn verwijderd." + "* Alle berekende resultaten zijn verwijderd.", + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" }); // This file contains all configured failure mechanisms (except Dunes and MacroStabilityInwards) with output. // The mechanisms Dunes and MacroStabilityInwards have different assessment sections, and are therefore put in different test files. yield return new TestCaseData("MigrationTestProject231WithOutput.risk", new[] { - "* Alle berekende resultaten zijn verwijderd, behalve die van het faalmechanisme 'Piping' en/of 'Macrostabiliteit binnenwaarts' waarbij de waterstand handmatig is ingevuld." + "* Alle berekende resultaten zijn verwijderd, behalve die van het faalmechanisme 'Piping' en/of 'Macrostabiliteit binnenwaarts' waarbij de waterstand handmatig is ingevuld.", + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" }); yield return new TestCaseData("MigrationTestProject231DunesWithOutput.risk", new[] { - "* Alle berekende resultaten zijn verwijderd." + "* Alle berekende resultaten zijn verwijderd.", + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" }); yield return new TestCaseData("MigrationTestProject231MacroStabilityInwardsWithOutput.risk", new[] { - "* Alle berekende resultaten zijn verwijderd, behalve die van het faalmechanisme 'Piping' en/of 'Macrostabiliteit binnenwaarts' waarbij de waterstand handmatig is ingevuld." + "* Alle berekende resultaten zijn verwijderd, behalve die van het faalmechanisme 'Piping' en/of 'Macrostabiliteit binnenwaarts' waarbij de waterstand handmatig is ingevuld.", + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" }); yield return new TestCaseData("MigrationTestProject231RevetmentCalculations.risk", new[] { - "* Geen aanpassingen." + $"* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:{Environment.NewLine}" + + $" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'{Environment.NewLine}" + + $" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'{Environment.NewLine}" + + " + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'" }); } Index: Riskeer/Migration/test/Riskeer.Migration.Test/ProjectMigratorTest.cs =================================================================== diff -u -rdf2eb00ad2eb3d4356b50e738ac81b76ba03c8de -r87d347be1c4afa0bfd1311c193364c49cfa73b60 --- Riskeer/Migration/test/Riskeer.Migration.Test/ProjectMigratorTest.cs (.../ProjectMigratorTest.cs) (revision df2eb00ad2eb3d4356b50e738ac81b76ba03c8de) +++ Riskeer/Migration/test/Riskeer.Migration.Test/ProjectMigratorTest.cs (.../ProjectMigratorTest.cs) (revision 87d347be1c4afa0bfd1311c193364c49cfa73b60) @@ -606,9 +606,11 @@ migrationLog.AppendLine(" + Faalmechanisme: 'Technische innovaties'"); migrationLog.AppendLine(" - De automatisch berekende faalkans van het faalmechanisme is verwijderd."); migrationLog.AppendLine("Gevolgen van de migratie van versie 23.1 naar versie 23.2:"); - migrationLog.AppendLine("* Geen aanpassingen."); + migrationLog.AppendLine("* Omdat alleen faalkansen op vakniveau een rol spelen in de assemblage, zijn de assemblageresultaten voor de faalmechanismen aangepast:"); + migrationLog.AppendLine(" + De initiële faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Handmatig invullen.'"); + migrationLog.AppendLine(" + De aangescherpte faalkansen per doorsnede zijn verwijderd in het geval van de optie 'Per doorsnede' of 'Beide.'"); + migrationLog.AppendLine(" + De assemblagemethode 'Automatisch berekenen o.b.v. slechtste doorsnede of vak' is vervangen door de methode 'Automatisch berekenen o.b.v. slechtste vak.'"); - Tuple[] expectedLogMessagesAndLevel = { Tuple.Create(expectedMessage, LogLevelConstant.Info),