Index: Ringtoets/Common/test/Ringtoets.Common.Data.Test/Hydraulics/HydraulicBoundaryDatabaseTest.cs =================================================================== diff -u -r35b7d54a7d06c2656986b10fc3f757b36a2dfd77 -rfe55aedc4ccec24ea5827ecf23a965d2a2cb1ed8 --- Ringtoets/Common/test/Ringtoets.Common.Data.Test/Hydraulics/HydraulicBoundaryDatabaseTest.cs (.../HydraulicBoundaryDatabaseTest.cs) (revision 35b7d54a7d06c2656986b10fc3f757b36a2dfd77) +++ Ringtoets/Common/test/Ringtoets.Common.Data.Test/Hydraulics/HydraulicBoundaryDatabaseTest.cs (.../HydraulicBoundaryDatabaseTest.cs) (revision fe55aedc4ccec24ea5827ecf23a965d2a2cb1ed8) @@ -47,115 +47,102 @@ } [Test] - public void Constructor_WithParameters_ExpectedValues([Values(true, false)] bool usePreprocessor) + public void UsePreprocessor_SetValueWithCanUsePreprocessorTrue_ExpectedValueSet() { // Setup - const string preprocessorDirectory = "Preprocessor"; + bool usePreprocessor = new Random(11).NextBoolean(); + var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase + { + CanUsePreprocessor = true + }; // Call - var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase(usePreprocessor, preprocessorDirectory); + hydraulicBoundaryDatabase.UsePreprocessor = usePreprocessor; // Assert - Assert.IsInstanceOf(hydraulicBoundaryDatabase); - Assert.IsNull(hydraulicBoundaryDatabase.FilePath); - Assert.IsNull(hydraulicBoundaryDatabase.Version); - CollectionAssert.IsEmpty(hydraulicBoundaryDatabase.Locations); - Assert.IsTrue(hydraulicBoundaryDatabase.CanUsePreprocessor); Assert.AreEqual(usePreprocessor, hydraulicBoundaryDatabase.UsePreprocessor); - Assert.AreEqual(preprocessorDirectory, hydraulicBoundaryDatabase.PreprocessorDirectory); } - [TestCase(true)] - [TestCase(false)] - public void CanUsePreprocessor_Always_ExpectedValuesSet(bool canUsePreprocessor) + [Test] + public void UsePreprocessor_SetValueWithCanUsePreprocessorFalse_ThrowsInvalidOperationException() { // Setup + bool usePreprocessor = new Random(11).NextBoolean(); var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase(); // Call - hydraulicBoundaryDatabase.CanUsePreprocessor = canUsePreprocessor; + TestDelegate test = () => hydraulicBoundaryDatabase.UsePreprocessor = usePreprocessor; // Assert - Assert.AreEqual(canUsePreprocessor, hydraulicBoundaryDatabase.CanUsePreprocessor); - Assert.IsFalse(hydraulicBoundaryDatabase.UsePreprocessor); - Assert.IsNull(hydraulicBoundaryDatabase.PreprocessorDirectory); + string message = Assert.Throws(test).Message; + Assert.AreEqual($"{nameof(HydraulicBoundaryDatabase.CanUsePreprocessor)} is false.", message); } [Test] - public void GivenDatabaseWithPreprocessorSettings_WhenSettingCanUsePreprocessorFalse_ThenPreprocessorSettingsReset() + public void PreprocessorDirectory_SetValidValueWithCanUsePreprocessorTrue_ExpectedValueSet() { - // Given + // Setup + const string preprocessorDirectory = "OtherPreprocessor"; var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase { - CanUsePreprocessor = true, - UsePreprocessor = true, - PreprocessorDirectory = "PreprocessorDirectory" + CanUsePreprocessor = true }; - // When - hydraulicBoundaryDatabase.CanUsePreprocessor = false; - - // Then - Assert.IsFalse(hydraulicBoundaryDatabase.CanUsePreprocessor); - Assert.IsFalse(hydraulicBoundaryDatabase.UsePreprocessor); - Assert.IsNull(hydraulicBoundaryDatabase.PreprocessorDirectory); - } - - [Test] - public void UsePreprocessor_SetValueWithCanUsePreprocessorTrue_ExpectedValueSet() - { - // Setup - bool usePreprocessor = new Random(11).NextBoolean(); - var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase(!usePreprocessor, "Preprocessor"); - // Call - hydraulicBoundaryDatabase.UsePreprocessor = usePreprocessor; + hydraulicBoundaryDatabase.PreprocessorDirectory = preprocessorDirectory; // Assert - Assert.AreEqual(usePreprocessor, hydraulicBoundaryDatabase.UsePreprocessor); + Assert.AreEqual(preprocessorDirectory, hydraulicBoundaryDatabase.PreprocessorDirectory); } [Test] - public void UsePreprocessor_SetValueWithCanUsePreprocessorFalse_ThrowsInvalidOperationException() + public void PreprocessorDirectory_SetValidValueWithCanUsePreprocessorFalse_ThrowsInvalidOperationException() { // Setup - bool usePreprocessor = new Random(11).NextBoolean(); var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase(); // Call - TestDelegate test = () => hydraulicBoundaryDatabase.UsePreprocessor = usePreprocessor; + TestDelegate test = () => hydraulicBoundaryDatabase.PreprocessorDirectory = "Preprocessor"; // Assert string message = Assert.Throws(test).Message; Assert.AreEqual($"{nameof(HydraulicBoundaryDatabase.CanUsePreprocessor)} is false.", message); } [Test] - public void PreprocessorDirectory_SetValidValueWithCanUsePreprocessorTrue_ExpectedValueSet() + public void GivenDatabaseWithPreprocessorSettings_WhenSettingCanUsePreprocessorFalse_ThenPreprocessorSettingsReset() { - // Setup - const string preprocessorDirectory = "OtherPreprocessor"; - var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase(true, "Preprocessor"); + // Given + var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase + { + CanUsePreprocessor = true, + UsePreprocessor = true, + PreprocessorDirectory = "PreprocessorDirectory" + }; - // Call - hydraulicBoundaryDatabase.PreprocessorDirectory = preprocessorDirectory; + // When + hydraulicBoundaryDatabase.CanUsePreprocessor = false; - // Assert - Assert.AreEqual(preprocessorDirectory, hydraulicBoundaryDatabase.PreprocessorDirectory); + // Then + Assert.IsFalse(hydraulicBoundaryDatabase.CanUsePreprocessor); + Assert.IsFalse(hydraulicBoundaryDatabase.UsePreprocessor); + Assert.IsNull(hydraulicBoundaryDatabase.PreprocessorDirectory); } - [Test] - public void PreprocessorDirectory_SetValidValueWithCanUsePreprocessorFalse_ThrowsInvalidOperationException() + [TestCase(true)] + [TestCase(false)] + public void CanUsePreprocessor_Always_ExpectedValuesSet(bool canUsePreprocessor) { // Setup var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase(); // Call - TestDelegate test = () => hydraulicBoundaryDatabase.PreprocessorDirectory = "Preprocessor"; + hydraulicBoundaryDatabase.CanUsePreprocessor = canUsePreprocessor; // Assert - string message = Assert.Throws(test).Message; - Assert.AreEqual($"{nameof(HydraulicBoundaryDatabase.CanUsePreprocessor)} is false.", message); + Assert.AreEqual(canUsePreprocessor, hydraulicBoundaryDatabase.CanUsePreprocessor); + Assert.IsFalse(hydraulicBoundaryDatabase.UsePreprocessor); + Assert.IsNull(hydraulicBoundaryDatabase.PreprocessorDirectory); } [TestCase(null)] @@ -164,7 +151,10 @@ public void PreprocessorDirectory_SetInvalidValueWithCanUsePreprocessorTrue_ThrowsArgumentException(string preprocessorDirectory) { // Setup - var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase(true, "Preprocessor"); + var hydraulicBoundaryDatabase = new HydraulicBoundaryDatabase + { + CanUsePreprocessor = true + }; // Call TestDelegate test = () => hydraulicBoundaryDatabase.PreprocessorDirectory = preprocessorDirectory;