C# - sprawdzanie czy kod poprawnie zapisuje tekst do pliku

Question

Witam! Ostatnio piszę sobie pewien program, w którym fragment kodu jest odpowiedzialny za zapisywanie tekstu do pliku. Chciałbym przetestować jednostkowo ten kod, który używa StreamWriter. W internecie znalazłem takie rozwiązanie (ze względu na moje własne potrzeby trochę to zmieniłem):

using (var stream = new MemoryStream())
using (var writer = new StreamWriter(stream))
{
    writer.Write("text");
 
    string actual = Encoding.UTF8.GetString(stream.ToArray());
    Assert.AreEqual("text", actual);
}

Kiedy odpalam test to okazuje się że zmienna "actual" jest pustym string'iem a nie "text".

Moje pytanie jest takie: Co trzeba zmienić w powyższym kodzie, aby sprawdzał on czy poprawnie jest zapisywany tekst do pliku (zapisujemy tutaj "text" do pliku więc jak sprawdzić, że to jest zapisane)? Lub też może być całkiem inny sposób niż ten powyżej.

Z góry dziękuje za pomoc!

PS.

Przepraszam, że znowu zadaje te pytanie, ale ostatnio nikt nie odpowiedział.

Answer 1

Na oko, zabrakło flusha writera. Więc albo wywołaj Flush():

writer.Write("text");
writer.Flush();

Albo użyj MemoryStream dopiero po zamknięciu Writera:

                using (var writer = new StreamWriter(stream))
                {
                    writer.Write("text");
                }
                string actual = Encoding.UTF8.GetString(stream.ToArray());

inna sprawa, że... dlaczego to testujesz w unit teście? :P To wygląda, jakbyś testował już przetestowaną bibliotekę standardową. A przecież pisanie tekstu do pliku to w najprostszym wariancie (bez Streamów) w zasadzie jedna linia kodu. IMO lepiej odizolować kod generujący tekst (i go przetestować w unit teście) od linijki piszącej do pliku (którą raczej powinno się testować w testach integracyjnych).

Comments

Wielkie dzięki! Ten kod co jest w pytaniu, to nie ja napisałem, wziąłem go ze stacka.

Poza tym, serio da się bez użcyia streamów zapisać coś do pliku? Pierwszy raz coś takiego widzę.

---

Jasne, po prostu

File.WriteAllText("file.txt", "costam");

(jest też kilka podobnych metod do pisania/czytania tekstu/linii.)

https://docs.microsoft.com/en-us/dotnet/api/system.io.file?view=netcore-3.1#methods

---

Dzięki

Answer 2

Wg mnie możesz zrobić stream writter do pliku, a potem stream readerem odczytać zawartość pliku.

string testtext = "przykladowa tresc";
string filepaht = "C:\twojasciezka.txt";
using (var writer = new StreamWriter(filepaht ))
{
    writer.Write(testtext);
}
string odczytany  = "";
using ( var reader = new StreamReader( filepaht ) )
{
odczytany =  reader.Read();
}

 Assert.AreEqual(testtext, odczytany);

Jeśli nie chcesz tak możesz próbować inaczej. Najważniejsze, żeby zamknąć writtera bo inaczej nie zdąży on zachować informacji w pliku. Wypadałoby również ten plik usunąć po porównaniu.

 

PS. Jeśli są błędy w składni to przepraszam - pisałem z głowy, bez możliwości przetestowania

 

Comments

Ale czy wypada, żeby podczas testu jednostkowego tworzyć pliki i pracować na nich zamiast operować na atrapach?