From 5a47082956f764972b7f97ae854246dd7177b4b4 Mon Sep 17 00:00:00 2001 From: Vitaliy Fedorchenko Date: Tue, 11 Jun 2024 18:43:51 +0300 Subject: [PATCH] Fixed test that fails on non-Windows env because of \r\n constant in expected outputs --- src/NReco.Csv.Tests/CsvWriterTests.cs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/NReco.Csv.Tests/CsvWriterTests.cs b/src/NReco.Csv.Tests/CsvWriterTests.cs index 887f2f6..3ae8c25 100644 --- a/src/NReco.Csv.Tests/CsvWriterTests.cs +++ b/src/NReco.Csv.Tests/CsvWriterTests.cs @@ -32,7 +32,7 @@ public void CsvWriterTest() { csvWriter.NextRecord(); var expected = "AAA,\"A\"\"AA\",\" AAA \",\"Something, again\",\"Something\nonce more\"\r\nJust one value\r\n"; - Assert.Equal(expected, strWr.ToString()); + Assert.Equal(applyEnvNewLine(expected), strWr.ToString()); strWr = new StringWriter(); csvWriter = new CsvWriter(strWr); @@ -44,10 +44,11 @@ public void CsvWriterTest() { csvWriter.NextRecord(); expected = ", ,\"\"\"\", aa \r\n"; - Assert.Equal(expected, strWr.ToString()); + Assert.Equal(applyEnvNewLine(expected), strWr.ToString()); } - + // to keep test env-independent let's use platform-specific NewLine constant + string applyEnvNewLine(string s) => s.Replace("\r\n", Environment.NewLine); } }