zamierzam pisać pierwsze testy jednostkowe i otworzyłem pierwszą klasę do testowania :


(klasa jest bez sensu bo ją przeredagowałem by nie skupiać się na zagnieżdżonych elsach itp, a skupić się na strukturze klasy, że jedna metoda wywołuje 3 inne, i że te metody nic nie zwracają tylko przypisują wartość właściwościom klasy))

[PHP] pobierz, plaintext 
class Order
{
    private $em;
    private $orders;
    private $tableDetails;
    private $forms;
 
    function __construct(EntityManager $em)
    {
        $this->em = $em;
    }
 
    public function prepareOrder($tableDetails, $tmpForms)
    {
        $this->setFields($tableDetails, $tmpForms);
 
        $this->setFalse();
        $this->setFilter();
 
        $this->prepareRepository();
    }
 
    private function setFields(array $fields, array $forms)
    {
        $this->tableDetails = $fields;
        $this->forms = $forms;
    }
 
    private function setFalse()
    {
        if ($this->isAnyFormValid()){
            $this->tableDetails['filterField'] = false;
        }
    }
 
    private function setFilter()
    {
        if(!($this->tableDetails['filter']))
        {
            $this->prepareFilterValue();
        }
 
    }
 
    private function prepareFilterValue(){
        if(!($this->tableDetails['filterField'])){
            $this->tableDetails['filter'] = 'all';}
    }
 
 
 
    private function isAnyFormValid()
    {
        if(($this->forms['StatusForm']->isValid())
        {
            return 1;
        }
    }
 
    private function prepareRepository()
    {
        if($this->tableDetails['filter'] == 'all'){
            $this->orderRepositoryMaker();
    }
 
 
    private function orderRepositoryMaker()
    {
        $this->orders = $this->em->getRepository('AppBundle:Zamowienie')
            ->findAll(..)
        if (!$this->orders) {throw new \Exception('Nie można znaleźć zamówień');}
    }
 
 
 
    public function getTableDetails()
    {
        return $this->tableDetails;
    }
 
    public function getOrder()
    {
        return $this->orders;
    }
}
[PHP] pobierz, plaintext 

także mam 3 metody publiczne:
- prepareOrder
- getTableDetails
- getOrder


jak mam np testować metodę prepareOrder() która nic nie zwraca, wywołuje kilka metod prywatnych, które przypisują wartości właściwościom klasy? Albo jak zmodyfikowac klasę by była test-friendly?

Troche nie na temat, ale dobrze by ta metoda:

[PHP] pobierz, plaintext 
    private function isAnyFormValid()
    {
        if(($this->forms['StatusForm']->isValid())or
            ($this->forms['DataZamForm']->isValid())or
            ($this->forms['NrKlientaForm']->isValid()))
        {
            return 1;
        }
    }
[PHP] pobierz, plaintext 

Wygladala tak:

[PHP] pobierz, plaintext 
    private function isAnyFormValid()
    {
        if(($this->forms['StatusForm']->isValid())or
            ($this->forms['DataZamForm']->isValid())or
            ($this->forms['NrKlientaForm']->isValid()))
        {
            return true;
        }
 
return false;
 
    }
[PHP] pobierz, plaintext 

lub krocej:

[PHP] pobierz, plaintext 
    private function isAnyFormValid()
    {
        return $this->forms['StatusForm']->isValid() ||
            $this->forms['DataZamForm']->isValid() ||
            $this->forms['NrKlientaForm']->isValid();
 
    }
[PHP] pobierz, plaintext 

Czyli jesli metoda zwraca wynik logiczny to ma zwracac wynik logiczny a nie 1 czy nic.

dla przykładu z :
http://forum.php.pl/index.php?showtopic=253330

no to przetestować czy ustawia to prosto sprawdzasz czy zostało ustawione zwracając sobie geter i czy instancja jest ok, a co do reszty to dlaczego ta metoda robi magie, polecałbym zapoznać się z zasadami SOLID, a problem rozwiążę się sam. W skrócie ta metoda nie powinna robić wszystkiego, seter nie ma logiki.

dzięki. (cholernie ciężko mi idzie decydowanie czym jest JEDNA ODPOWIEDZIALNOŚĆ klasy, no bo to zależy z jakiego poziomu spojrzeć na klasę. Klasa może być odpowiedzialna za zwrócenie repozytorium, ale po drodze robi też parę dupereli, no i ciężko mi roztrzygać czy każdy duperel ma być osobną klasą czy prywatną metodą. Czy jeśli duperel ma 40 linijek kodu to już ma być osobną klasą mimo że tak naprawdę nie robi nic ważniejszego niż inny duperel z 3 linijkami kodu)


link nie działa.

Tam pod tym linkiem był ten Twój drugi przykład z kodem, ale moderator zamknął temat, ok ale czemu upychasz to w metody prywatne? zrób tak żeby każdy kto jej użyje wiedział co się tam dzieje, wtedy tez będzie łatwiej to przetestować, jak wyciągniesz to co masz zrobić na zewnątrz i z tamtego poziomu to wywołasz.

czyli masa publicznych pierdół wywoływana w kontrolerze. Specjalnie chowałem to w prywatnych bo każą chude kontrolery robić.

mogą być prywatne metody, ale musisz tak zaprojektować żeby jak ktoś weźmie Twój kod wiedział co się tam stanie jak go wykona, dlatego są ważne testy, bo one dokumentują na bieżąco Twój kod, skoro kod nie jest testowany, to znaczy, ze trzeba go przebudować tak, żeby dało się go przetestować

Osobiście wydzieliłbym Encje, czy repozytorium czy co to tam masz od reszty logiki

Ta klasa jest paskudna, ale jej publiczne metody są jasne.

Na moje oko w kontrolerze wygląda to dość czytelnie:

[PHP] pobierz, plaintext 
        $manager_order = $this->get('app.manager_order');
        $manager_order->prepareOrder($tableDetails, $tmpForms);
        $orders = $manager_order->getOrder();
        $tableDetails = $manager_order->getTableDetails();
[PHP] pobierz, plaintext 

Mam opory przed wyprowadzaniem kodu prywatnych metod do nowych klas, który
- jest bardzo silnie związany z właściwościami klasy.
- jest specyficzny, jednorazowego użytku. Tak jak metoda prepareRepository() przeniesiona do osobnej klasy i tak nigdy drugi raz użyta nie będzie.