Index: Application/Ringtoets/test/Application.Ringtoets.Storage.Test/BackedUpFileWriterTest.cs =================================================================== diff -u -r6a5d7b40b7ba4dcb73e393075338352d194e97c2 -r9e7b8344cc985249f312870b9030705371c82bfe --- Application/Ringtoets/test/Application.Ringtoets.Storage.Test/BackedUpFileWriterTest.cs (.../BackedUpFileWriterTest.cs) (revision 6a5d7b40b7ba4dcb73e393075338352d194e97c2) +++ Application/Ringtoets/test/Application.Ringtoets.Storage.Test/BackedUpFileWriterTest.cs (.../BackedUpFileWriterTest.cs) (revision 9e7b8344cc985249f312870b9030705371c82bfe) @@ -31,7 +31,7 @@ [TestFixture] public class BackedUpFileWriterTest { - private readonly string testWorkDir = Path.Combine(".", "SafeOverwriteFileHelperTest"); + private readonly string testWorkDir = TestHelper.GetScratchPadPath("SafeOverwriteFileHelperTest"); [OneTimeSetUp] public void SetUpFixture() @@ -153,9 +153,7 @@ { // Assert var message = Assert.Throws(test).Message; - var expectedMessage = string.Format( - "Er bestaat al een tijdelijk bestand ({0}) dat niet verwijderd kan worden. Dit bestand dient handmatig verwijderd te worden.", - temporaryFilePath); + var expectedMessage = $"Er bestaat al een tijdelijk bestand ({temporaryFilePath}) dat niet verwijderd kan worden. Dit bestand dient handmatig verwijderd te worden."; Assert.AreEqual(message, expectedMessage); temporaryFileStream.Dispose(); } @@ -188,8 +186,7 @@ using (new DirectoryPermissionsRevoker(notWritableDirectory, FileSystemRights.Write)) { // Assert - var expectedMessage = string.Format("Kan geen tijdelijk bestand maken van het originele bestand ({0}).", - filePath); + var expectedMessage = $"Kan geen tijdelijk bestand maken van het originele bestand ({filePath})."; var message = Assert.Throws(test).Message; Assert.AreEqual(expectedMessage, message); } @@ -248,18 +245,13 @@ TestDelegate test = () => helper.Perform(() => { fileStream = File.Open(temporaryFilePath, FileMode.Open); }); // Assert - var expectedMessage = string.Format( - "Kan het tijdelijke bestand ({0}) niet opruimen. Het tijdelijke bestand dient handmatig verwijderd te worden.", - temporaryFilePath); + var expectedMessage = $"Kan het tijdelijke bestand ({temporaryFilePath}) niet opruimen. Het tijdelijke bestand dient handmatig verwijderd te worden."; var message = Assert.Throws(test).Message; Assert.AreEqual(expectedMessage, message); } finally { - if (fileStream != null) - { - fileStream.Dispose(); - } + fileStream?.Dispose(); Directory.Delete(noAccessDirectory, true); } } @@ -289,18 +281,13 @@ }); // Assert - var expectedMessage = string.Format( - "Kan het originele bestand ({0}) niet herstellen. Het tijdelijke bestand dient handmatig hersteld te worden.", - filePath); + var expectedMessage = $"Kan het originele bestand ({filePath}) niet herstellen. Het tijdelijke bestand dient handmatig hersteld te worden."; var message = Assert.Throws(test).Message; Assert.AreEqual(expectedMessage, message); } finally { - if (fileStream != null) - { - fileStream.Dispose(); - } + fileStream?.Dispose(); Directory.Delete(noAccessDirectory, true); } } @@ -328,18 +315,13 @@ }); // Assert - var expectedMessage = string.Format( - "Kan het originele bestand ({0}) niet herstellen. Het tijdelijke bestand dient handmatig hersteld te worden.", - filePath); + var expectedMessage = $"Kan het originele bestand ({filePath}) niet herstellen. Het tijdelijke bestand dient handmatig hersteld te worden."; var message = Assert.Throws(test).Message; Assert.AreEqual(expectedMessage, message); } finally { - if (fileRightsHelper != null) - { - fileRightsHelper.Dispose(); - } + fileRightsHelper?.Dispose(); Directory.Delete(noAccessDirectory, true); } }