[T.....n]

#programowanie #csharp

Unit testy (testy jednostkowe). Moja klasa A polega na bezpośrednich informacjach obiektu klasy B (z zewnętrznej biblioteki). Np:

if(b.jakieśPoleTylkoDoOdczytu)
...

W tym układzie wszystko jest jasne i proste. Potrzebuje jednak wykonać unit testy mojej klasy A, które zależne są bezpośrednio od B. Nie powinno się jednak mockować (przepraszam jeśli inaczej się to odmienia) klas - w tym wypadku B. Czy powinienem wszystkie relacje pomiędzy A i B przenieść jakoś na osobny interfejs i dopiero wtedy go mockować? Czy może zupełnie źle do tego podchodzę?

[Lethal_Jelly]

@Telhurin: Zrób wrapper do zewnętrznej klasy B, z potrzebnym Ci interfejsem. Wtedy można uznać taką klasę za część zewnętrznego systemu a takiego kodu się nie testuje jednostkowo, jedynie integracyjnie itd. Nie testuj jednostkowo/mockuj kodu który może się zepsuć nawet jak źle zinterpretowałeś dokumentację serwisu, to nic nie da :)

Oczywiście wtedy tą swoją klasę B łatwo mockujesz przewidując co ma zwrócić przy testowaniu klasy A

[k.....l]

@Telhurin:

Dlaczego uważasz, że nie powinno się mockować zewnętrznych zależności?

[T.....n]

@kisi3l: Nie nie, wprost przeciwnie ja chcę to zrobić. Napisałem tylko że nie powinno się mockować klas, a interfejsy.

[ogrod87]

@Telhurin: Dlaczego nie powinno się mockować klas?

[T.....n]

@ogrod87: Bardzo możliwe, że dotyczy to tylko C# czy może biblioteki, której używam.
http://stackoverflow.com/questions/12174304/is-it-recommended-to-mock-concrete-class
http://nsubstitute.github.io/help/creating-a-substitute/

[ogrod87]

@Telhurin: Ja chciałem Ciebie o zdanie zapytać, bo ja akurat wiem, czy powinno, czy nie powinno. A przede wszystkim kiedy powinno się mockować ( ͡° ͜ʖ ͡°)

[T.....n]

@ogrod87: Obawiam się, że jestem bardzo słabym autorytetem w tej dziedzinie :D, więc mogłem jedynie powiedzieć Ci skąd to wytrzasnąłem.

[T.....n]

@ogrod87: Mam/miałem wątpliwości z pierwszego postu ponieważ o ile wrap jest jeszcze do przełknięcia, interfejs wyłącznie dla tego wrapu (tylko żeby potem przez niego mockować) wydaje się na pierwszy rzut oka bardzo sztuczny.

[ogrod87]

@Telhurin: A jeśli nie chcesz mockować, to co powiesz na fake/stub?

[T.....n]

@ogrod87: Miałoby to chyba sens, ale jestem wymaganiami ograniczony do biblioteki Nsubstitute, która jak rozumiem rzeczy takie jak właśnie stub, fake czy spy zastępuje po prostu szerszym substitute.

[ogrod87]

@Telhurin: ja bym zaleznosci nie przenosil na osobny interfejs ale do metody. Potem z nsubstitute uzyc forParts. I zrobic fake returna tylko dla tej metody, ktorą napiszesz. Mam nadzieje, ze rozumiesz

[T.....n]

@ogrod87: Ja cię kręcę :O, to może być to... jesteś dobrym człowiekiem <3!