[ghost1511]

Jak testować settery? Czasami oprócz zwykłego przypisania dajemy tam walidację, jakieś parsowanie. Chcielibyśmy sprawdzić czy wartość się rzeczywiście zmieniła.

#php #phpunit #testyjednostkowe

[whatdoi]

Gdzie widzisz problem? Do zmiennej przypisujesz wartosci, nastepnie odpalasz sobie taki setter i sprawdzasz jakis assert, exception czy cokolwiek chcesz.

[ghost1511]

@whatdoi: No dobra. Ale właściwość jest prywatna. Aby się do niej dostać trzeba dostać się użyć refleksji.

[ghost1511]

@whatdoi: I tutaj rodzi się pytanie czy jest lepsze rozwiązanie.

[whatdoi]

Jezeli masz setter to masz i getter, a setter masz public albo protected. Wiec wyjatki i wartosci mozesz przetestowac.

[MQs]

@ghost1511: Jak nie masz gettera to testuj metody, które używają właściwości (settera testujesz pośrednio). Generalnie taki setter z walidacją/parsowaniem to side effect i powinno się tego unikać.

[f.....d]

@ghost1511: wtedy testujesz go w parze z getterem.

[f.....d]

@MQs: walidacja nie będzie side effectem - imho powinien być rzucony w tej sytuacji wyjątek (InvalidArgumentException albo coś w ten deseń)

[uirapuru]

@ghost1511: setter nie jest walidatorem, moim zdaniem powinno się to rozdzielić. walidacja przeszla - ustawiamy setter. ps. setterów i getterów osobiście nie testowałbym

[MQs]

@fmfd: Jeśli chodzi o używanie wyjątków do flow controll to obecnie nie mam na ten temat klarownej opinii. Rozpocząłem ze stanowczym "nie", ale... pojawiły się wyjątki;)

[f.....d]

@MQs: a kto ma sie bronic przed ustawieniem zlej wartosci w obiekcie? Sam obiekt czy cos zewnetrznego dla niego? :)

[f.....d]

@MQs: http://stackoverflow.com/questions/20378045/can-i-write-validation-logic-in-setter-methods

[MQs]

a kto ma sie bronic przed ustawieniem zlej wartosci w obiekcie? Sam obiekt czy cos zewnetrznego dla niego? :)

@fmfd: Generalnie coś zewnętrznego - developer:) Jeśli nie ma na to wpływu, bo np. publikuje moduł (używany przez kogoś innego) to rzuca wyjątkami, bo nieodpowiednie dane rozwalą program w głębszych warstwach i do własnego błędu trzeba by dochodzić przez debugowanie modułu.

Walidacja to nie wyjątek imo - to część funkcji.

Linkami

[f.....d]

@MQs: jak publikujesz biblioteke to raczej nie masz wplywu na to czy ktos rozsadny bedzie z niej korzystac :P

Dlatego exception jest imo na miejscu. Zreszta to powszechna praktyka przy korzystaniu z guavy: Preconditions.check*()

[MQs]

jak publikujesz biblioteke to raczej nie masz wplywu na to czy ktos rozsadny bedzie z niej korzystac :P

@fmfd: Przecież to napisałem.