Index: Ringtoets/Piping/src/Ringtoets.Piping.IO/Exporters/PipingConfigurationExporter.cs =================================================================== diff -u -r36b1ab8fe4c395ade52f057cb2ce07960d04098c -raf27351920210528ed4f8e121bcca04f6400434d --- Ringtoets/Piping/src/Ringtoets.Piping.IO/Exporters/PipingConfigurationExporter.cs (.../PipingConfigurationExporter.cs) (revision 36b1ab8fe4c395ade52f057cb2ce07960d04098c) +++ Ringtoets/Piping/src/Ringtoets.Piping.IO/Exporters/PipingConfigurationExporter.cs (.../PipingConfigurationExporter.cs) (revision af27351920210528ed4f8e121bcca04f6400434d) @@ -21,8 +21,11 @@ using System; using Core.Common.Base.IO; +using Core.Common.IO.Exceptions; using Core.Common.Utils; +using log4net; using Ringtoets.Common.Data.Calculation; +using Ringtoets.Piping.IO.Properties; namespace Ringtoets.Piping.IO.Exporters { @@ -31,6 +34,8 @@ /// public class PipingConfigurationExporter : IFileExporter { + private static readonly ILog log = LogManager.GetLogger(typeof(PipingConfigurationExporter)); + private readonly CalculationGroup calculationGroup; private readonly string filePath; @@ -56,7 +61,17 @@ public bool Export() { - throw new System.NotImplementedException(); + try + { + PipingConfigurationWriter.Write(calculationGroup, filePath); + } + catch (CriticalFileWriteException e) + { + log.ErrorFormat(Resources.PipingConfigurationExporter_Export_Error_exception_0_no_configuration_exported, e.Message); + return false; + } + + return true; } } } \ No newline at end of file Index: Ringtoets/Piping/src/Ringtoets.Piping.IO/Properties/Resources.Designer.cs =================================================================== diff -u -r9b940da26da12706c2b6665959e6de7f1a3dbc52 -raf27351920210528ed4f8e121bcca04f6400434d --- Ringtoets/Piping/src/Ringtoets.Piping.IO/Properties/Resources.Designer.cs (.../Resources.Designer.cs) (revision 9b940da26da12706c2b6665959e6de7f1a3dbc52) +++ Ringtoets/Piping/src/Ringtoets.Piping.IO/Properties/Resources.Designer.cs (.../Resources.Designer.cs) (revision af27351920210528ed4f8e121bcca04f6400434d) @@ -193,6 +193,15 @@ } /// + /// Looks up a localized string similar to {0} Er is geen configuratie geëxporteerd.. + /// + public static string PipingConfigurationExporter_Export_Error_exception_0_no_configuration_exported { + get { + return ResourceManager.GetString("PipingConfigurationExporter_Export_Error_exception_0_no_configuration_exported", resourceCulture); + } + } + + /// /// Looks up a localized string similar to Het XML-document dat de configuratie voor de berekeningen beschrijft is niet geldig.. /// public static string PipingConfigurationReader_Configuration_contains_no_valid_xml { Index: Ringtoets/Piping/src/Ringtoets.Piping.IO/Properties/Resources.resx =================================================================== diff -u -r9b940da26da12706c2b6665959e6de7f1a3dbc52 -raf27351920210528ed4f8e121bcca04f6400434d --- Ringtoets/Piping/src/Ringtoets.Piping.IO/Properties/Resources.resx (.../Resources.resx) (revision 9b940da26da12706c2b6665959e6de7f1a3dbc52) +++ Ringtoets/Piping/src/Ringtoets.Piping.IO/Properties/Resources.resx (.../Resources.resx) (revision af27351920210528ed4f8e121bcca04f6400434d) @@ -263,4 +263,7 @@ Het XML-document dat de configuratie voor de berekeningen beschrijft is niet geldig. + + {0} Er is geen configuratie geëxporteerd. + \ No newline at end of file Index: Ringtoets/Piping/test/Ringtoets.Piping.IO.Test/Exporters/PipingConfigurationExporterTest.cs =================================================================== diff -u -r36b1ab8fe4c395ade52f057cb2ce07960d04098c -raf27351920210528ed4f8e121bcca04f6400434d --- Ringtoets/Piping/test/Ringtoets.Piping.IO.Test/Exporters/PipingConfigurationExporterTest.cs (.../PipingConfigurationExporterTest.cs) (revision 36b1ab8fe4c395ade52f057cb2ce07960d04098c) +++ Ringtoets/Piping/test/Ringtoets.Piping.IO.Test/Exporters/PipingConfigurationExporterTest.cs (.../PipingConfigurationExporterTest.cs) (revision af27351920210528ed4f8e121bcca04f6400434d) @@ -20,10 +20,18 @@ // All rights reserved. using System; +using System.IO; +using System.Security.AccessControl; +using Core.Common.Base.Data; using Core.Common.Base.IO; +using Core.Common.TestUtil; using NUnit.Framework; using Ringtoets.Common.Data.Calculation; +using Ringtoets.Common.Data.Hydraulics; +using Ringtoets.Piping.Data; +using Ringtoets.Piping.Integration.TestUtils; using Ringtoets.Piping.IO.Exporters; +using Ringtoets.Piping.Primitives; namespace Ringtoets.Piping.IO.Test.Exporters { @@ -63,5 +71,119 @@ // Assert Assert.Throws(test); } + + [Test] + public void Export_ValidData_ReturnTrueAndWritesFile() + { + // Setup + string directoryPath = TestHelper.GetTestDataPath(TestDataPath.Ringtoets.Piping.IO, + "PipingConfigurationWriter"); + Directory.CreateDirectory(directoryPath); + string filePath = Path.Combine(directoryPath, "test.xml"); + + var calculation = PipingTestDataGenerator.GetPipingCalculation(); + calculation.InputParameters.ExitPointL = (RoundedDouble)0.2; + + var calculation2 = PipingTestDataGenerator.GetPipingCalculation(); + calculation2.Name = "PK001_0002 W1-6_4_1D1"; + calculation2.InputParameters.HydraulicBoundaryLocation = new HydraulicBoundaryLocation(1, "PUNT_SCH_17", 0, 0); + calculation2.InputParameters.SurfaceLine.Name = "PK001_0002"; + calculation2.InputParameters.ExitPointL = (RoundedDouble)0.2; + calculation2.InputParameters.StochasticSoilModel = new StochasticSoilModel(1, "PK001_0002_Piping", string.Empty); + calculation2.InputParameters.StochasticSoilProfile = new StochasticSoilProfile(0, SoilProfileType.SoilProfile1D, 0) + { + SoilProfile = new PipingSoilProfile("W1-6_4_1D1", 0, new[] + { + new PipingSoilLayer(0) + }, SoilProfileType.SoilProfile1D, 0) + }; + + var calculationGroup2 = new CalculationGroup("PK001_0002", false) + { + Children = + { + calculation2 + } + }; + + var calculationGroup = new CalculationGroup("PK001_0001", false) + { + Children = + { + calculation, + calculationGroup2 + } + }; + + var rootGroup = new CalculationGroup("root", false) + { + Children = + { + calculationGroup + } + }; + + var expoter = new PipingConfigurationExporter(rootGroup, filePath); + + try + { + // Call + bool isExported = expoter.Export(); + + // Assert + Assert.IsTrue(isExported); + Assert.IsTrue(File.Exists(filePath)); + + var actualXml = File.ReadAllText(filePath); + var expectedXml = File.ReadAllText(Path.Combine(directoryPath, "folderWithSubfolderAndCalculation.xml")); + + + Assert.AreEqual(expectedXml, actualXml); + } + finally + { + File.Delete(filePath); + } + } + + [Test] + public void Export_InvalidDirectoryRights_LogErrorAndReturnFalse() + { + // Setup + var calculationGroup = new CalculationGroup + { + Children = + { + PipingTestDataGenerator.GetPipingCalculation() + } + }; + + string directoryPath = TestHelper.GetTestDataPath(TestDataPath.Ringtoets.Piping.IO, + "Export_InvalidDirectoryRights_LogErrorAndReturnFalse"); + Directory.CreateDirectory(directoryPath); + string filePath = Path.Combine(directoryPath, "test.xml"); + + var exporter = new PipingConfigurationExporter(calculationGroup, filePath); + + try + { + using (new DirectoryPermissionsRevoker(directoryPath, FileSystemRights.Write)) + { + // Call + bool isExported = true; + Action call = () => isExported = exporter.Export(); + + // Assert + string expectedMessage = $"Er is een onverwachte fout opgetreden tijdens het schrijven van het bestand '{filePath}'. " + + "Er is geen configuratie geëxporteerd."; + TestHelper.AssertLogMessageIsGenerated(call, expectedMessage); + Assert.IsFalse(isExported); + } + } + finally + { + Directory.Delete(directoryPath, true); + } + } } } \ No newline at end of file