Testy jednostkowe do tej klasy

Question

Witam! Piszę projekt w którym mam pewną klasę do której chciałbym napisać testy. Problem polega na tym, że nie mogę wykonać tych testów ponieważ wszystkie metody, które chcę przetestować są prywatne (tylko jedna metoda jest publiczna). Nie wiem co zrobić. W zasadzie z tego co mi wiadomo metod prywatnych nie powinno się testować, ale chciałbym jednak napisać jakieś testy, czuje taką potrzebę. Zastanawiam się czy winną tego wszystkiego nie jest to, że źle zaprojektowałem tę klasę albo czy nie jest ona zbyt duża, bądź czy niepotrzebnie staram się to wszystko testować. Generalnie metody które chciałem przetestować to (poniżej jest link do kodu klasy):

- CheckSelectedIndexes oraz SetIndexesOfField (w liniach 149 i 178)

- SetMinesOnBoard (w linii 194)

- LoadNumberOfMinesIntoDisplayedBoard (259)

- CalculateHowManyMinesAreAroundField (290)

- UnveilingFieldOrSeveralFields (380)

- GetGameTimeInTextVersion (437)

Kod klasy jest tutaj: https://github.com/DavidPL-coder/MultiGry/blob/master/MultiGry/MinesweeperGame.cs

Klasa ta jest częścią programu, który wyświetla menu gier. Użytkownik może sobie wybrać jakoś grę. Ta klasa to gra saper (wersja konsolowa).

Z góry dziękuje za pomoc!

PS.

Przepraszam, że znowu zadaje te pytanie, ale wcześniej nikt nie odpowiedział.

Answer 1

Pojawia się tutaj klasyczny problem na który wskazują unit testy. Klasa wykonuje zdecydowanie za dużo logiki. Fakt że praktycznie wszystko jest prywatne nie pomaga. Na chwilę obecną jest to raczej nietestowalne jednostkowo.

Comments

Wychodzi na to, że muszę rozbić klasę na mniejsze części. Największym problemem jest tutaj to, że niektóre funkcjonalności korzystają z pól tej klasy, które są też potrzebne w innych miejscach. Przeważnie potrzebują po jednej lub po dwa i nie jest to problemem można później przekazać wartości tych pól przez argumenty, ale metoda LoadNumberOfMinesIntoDisplayedBoard używa ich aż 5, i nie wiem jak przekazać tak dużą ilość danych, gdybym ją dał do innej klasy.

---

Ja zrobiłem tylko luźne review, nie wiem jaki problem modelujesz. Na pewno trzeba to bardziej zrobić. Jeśli twierdzisz że musisz przekazać za dużo danych to zastanów się czy muszę one być tam, gdzie są obecnie

---

Wydaje mi się, że powinny być w tam, gdzie są...

---

Masz rację, wydaje Ci się ​​​​​​

Opakowałeś całą grę w jedną klasę i chcesz to testować jednostkowo. Nie da rady.

---

Czyli mam nie próbować dzielić tej klasy na mniejsze klasy, tylko napisać to wszystko odnowa?

---

Teoretycznie możesz pisać od nowa, ale więcej się nauczysz dzieląc ten kod na składowe.

---

Dzięki za pomoc!