Index: Ringtoets/Common/test/Ringtoets.Common.Data.Test/UpdateDataStrategies/ReplaceDataStrategyBaseTest.cs =================================================================== diff -u -r66239a42b315209acc4d124200c58d8be1a18195 -r56ac4eb28f5fcc5b20117474e9e4030399d6806a --- Ringtoets/Common/test/Ringtoets.Common.Data.Test/UpdateDataStrategies/ReplaceDataStrategyBaseTest.cs (.../ReplaceDataStrategyBaseTest.cs) (revision 66239a42b315209acc4d124200c58d8be1a18195) +++ Ringtoets/Common/test/Ringtoets.Common.Data.Test/UpdateDataStrategies/ReplaceDataStrategyBaseTest.cs (.../ReplaceDataStrategyBaseTest.cs) (revision 56ac4eb28f5fcc5b20117474e9e4030399d6806a) @@ -24,6 +24,7 @@ using System.Linq; using Core.Common.Base; using NUnit.Framework; +using Ringtoets.Common.Data.Exceptions; using Ringtoets.Common.Data.TestUtil; using Ringtoets.Common.Data.UpdateDataStrategies; @@ -64,7 +65,7 @@ // Assert string paramName = Assert.Throws(call).ParamName; - Assert.AreEqual("targetCollection", paramName); + Assert.AreEqual("targetDataCollection", paramName); } [Test] @@ -112,7 +113,7 @@ } [Test] - public void ReplaceData_ImportedDataCollectionContainsDuplicateItems_ThrowsArgumentException() + public void ReplaceData_ImportedDataCollectionContainsDuplicateItems_ThrowsUpdateDataException() { // Setup var strategy = new ConcreteStrategyClass(new TestFailureMechanism()); @@ -130,13 +131,14 @@ // Assert CollectionAssert.IsEmpty(collection); - var exception = Assert.Throws(call); + var exception = Assert.Throws(call); string expectedMessage = $"TestItem moeten een unieke naam hebben. Gevonden dubbele elementen: {duplicateName}."; Assert.AreEqual(expectedMessage, exception.Message); + Assert.IsInstanceOf(exception.InnerException); } [Test] - public void ReplaceData_ImportedDataCollectionContainsNull_ThrowsArgumentException() + public void ReplaceData_ImportedDataCollectionContainsNull_ThrowsUpdateDataException() { // Setup var strategy = new ConcreteStrategyClass(new TestFailureMechanism()); @@ -154,13 +156,15 @@ // Assert CollectionAssert.IsEmpty(collection); - Assert.Throws(call); + + var exception = Assert.Throws(call); + Assert.IsInstanceOf(exception.InnerException); } [Test] [TestCase("")] [TestCase(" ")] - public void ReplaceData_InvalidSourceFilePath_ThrowsArgumentException(string invalidPath) + public void ReplaceData_InvalidSourceFilePath_ThrowsUpdateDataException(string invalidPath) { // Setup var strategy = new ConcreteStrategyClass(new TestFailureMechanism()); @@ -178,7 +182,9 @@ // Assert CollectionAssert.IsEmpty(collection); - Assert.Throws(call); + + var exception = Assert.Throws(call); + Assert.IsInstanceOf(exception.InnerException); } [Test]