Forum PHP.pl

Hej,

Mam sytuację, w której jest metoda a jeden z jej argumentów idzie przez referencję. Mój problem polega na tym, że ta referencja zwraca IDki, które później są sprawdzane w innej metodzie (tej, która wywołała) czy IDki są w tablicy, jeśli tak to zwróć true. Poniżej przedstawiam schemat (nazwy klas, metod - zmyślone na potrzeby tematu, flow jest jednak ten sam):

[PHP] pobierz, plaintext 
<?php
 
class Db
{
	public function update($rows, &$success, &$error)
	{
		if ((..)) {
			$success[] = $this->getId();
		} else {
			$error[] = $this->getId();
		}
	}
}
 
class Prepare
{
	protected $db;
 
	public function setDb($db)
	{
		// itd.
	}
 
	public function make()
	{
		// $rows = (...) jakaś tam obróbka danych
		$success = array();
		$error = array();
 
		$this->db->update($rows, $success, $error);
 
		return ((count($success) > 0 AND count($error) == 0) ? true : false);
	}
}
 
class TestPrepare extends PHPUnit_Framework_TestCas
{
	public function testMakeMethod()
	{
		$mck = Mockery::mock();
		$mck->shouldReceive('update')
			->with(
				['id' => 51, 'name' => 'John', 'password' => 'Hour']
				// REFERENCE?
				// REFERENCE?
			)
			->once()
			->andReturn(true)
 
		$prepare = new Prepare();
		$prepare->setDb($mck);
		$response = $prepare->make();
 
		$this->assertTrue($response)
	}
}
[PHP] pobierz, plaintext 

I teraz dla #44, #45 linijki nie wiem co mam wstawić, aby metoda, którą mockuje przekazywała w sobie też referencję z testowymi danymi, próbowałem coś w stylu:

[PHP] pobierz, plaintext 
			->with(
				['id' => 51, 'name' => 'John', 'password' => 'Hour']
				\Mockery::on(function(&$error) {
					$error = array();
 
					return true;
				}),
				\Mockery::on(function(&$success) {
					$success[] = 51;
 
					return true;
				}),
			)
[PHP] pobierz, plaintext 

ale to nie działa, ponieważ jak zrobię sobie:

[PHP] pobierz, plaintext 
var_dump($success, $errors)
[PHP] pobierz, plaintext 

przed tym return w klasie: Prepare to cały czas mam pustą tablice - w ogóle nie dodało mojego "51" (jako ID) w konsekwencji mam, że test nie przeszedł. Ma ktoś jakiś pomysł?

Stworzyłeś idealny przykład na to, dlaczego nie należy używać referencji (nie mówię o obiektach, które mają referencje tworzone przez mechanizm języka).

http://docs.mockery.io/en/latest/reference...behaviours.html

[PHP] pobierz, plaintext 
<?php
 
$m->shouldReceive('insert')->with(
        \Mockery::on(function(&$data) {
            if (!is_array($data)) return false;
            $data['_id'] = 123;
            return true;
        }),
        \Mockery::any()
    );
[PHP] pobierz, plaintext 

Nie znam mockery, ale być może linijka
if (!is_array($data)) return false;
jest jednak istotna (php to bardzo magiczny język z wieloma magicznie działającymi bibliotekami (IMG:style_emoticons/default/smile.gif) ).

Ten post edytował lukaskolista 29.12.2016, 15:56:20

@wujek2009 teoria jak i praktyka mówi że jeżeli masz problem z przetestowaniem kodu to najprawdopodobniej napisałeś go źle. Na twoim miejscu nie tracił bym czasu na szukanie rozwiązania odnośnie testowania a skupił się na przemyśleniu kodu który napisałeś

[PHP] pobierz, plaintext 
class Db
{
	public function update($rows, &$success, &$error)
	{
		if ((..)) {
			$this->success[] = $this->getId();
		} else {
			$this->errored[] = $this->getId();
		}
	}
 
	public function getSuccessfullyProcessed()
	{
		return $this->success;
	}
 
	public function getErrored()
	{
		return $this->errored;
	}
}
[PHP] pobierz, plaintext 

Dzięki Panowie za odpowiedź. Jeśli chodzi o samą prezentację kodu to wygląda to tak, że do istniejącego rozwiązania muszę dopisać testy.