[PHPUnit] Nadpisanie metody statycznej Opcje

[kapslokk]

Cześć,
próbuje napisać test, dla klasy, która korzysta z innej. Przykład:

[PHP] pobierz, plaintext 
class A{
  public static function get(){ 
   return 1;
  }
}
 
class B{
 private $a;
 public function __construct($a){
  $this->a = $a;
 }
public function doSomething(){
 return $this->a->get();
}
}
[PHP] pobierz, plaintext 

No i jak próbuje zrobić mock metody get i jest tam "static", to po prostu pomija ten test. Jeżeli usunę "static" - wszystko działa ok.

Nie pytajcie po co ten static tam w ogole jest - nie moja wina, grzebe w starym kodzie, nie mogę go usunąć.
Używam PHPUnit 5.4.2

[LowiczakPL]

Coś mało dałeś jak na mocka ale czy nie tak to powinno wyglądać w Twoim przypadku

[PHP] pobierz, plaintext 
class A
{
    public static function doSomething()
    {
        return static::get();
    }
 
    public static function get()
    {
        return '1';
    }
}    
 
class ATest extends PHPUnit_Framework_TestCase
{
    public function testDoSomething()
    {
        $class = $this->getMockClass(
          'A',        
          array('get') 
        );
 
        $class::staticExpects($this->any())
              ->method('get')
              ->will($this->returnValue('1'));
 
        $this->assertEquals(
          '1',
          $class::doSomething()
        );
    }
}
[PHP] pobierz, plaintext 

[kapslokk]

Dałem przykład klas, do których chce zrobić mocka, samego mocka nie dawałem, bo po prostu nie działał. Co do ::staticExpects to w mojej wersji PHPUnit nie ma tej metody.

Ten post edytował kapslokk 9.06.2016, 07:15:12

[LowiczakPL]

To wklej swój kod łatwiej będzie to ogarnąć.

[Crozin]

1. Wiem, że PHP pozwala na pewne głupoty, ale dlaczego wywołujesz metody statyczne jak metody obiektu?
2. Tutaj mamy koronny przykład dlaczego w zdecydowanej większości przypadków użycie statycznych metod jest złe i niepożądane. Metody statyczne są praktycznie niemożliwe do testowania bez "hacków" w postaci np. dynamicznej podmiany byte-code'u.
3. Opisz może dokładniej swój przypadek (albo podaj wręcz faktyczny kod) - być może wcale nie ma potrzeby do tworzenia mocka.

[kapslokk]

1. Nie pytaj, bo po prostu nie potrafię odpowiedzieć na to pytanie. Ktoś przede mną w firmie zrobił głupotę i teraz z tego korzystam,
2. Wiem, gdybym robił to od początku, zrobił bym to inaczej.
3. Ogólnie chodzi o to, że nowe funkcjonalności w serwisie piszemy obiektowo, natomiast znaczna większość jest napisana strukturalnie. PM nie pozwolił nam się od tego odciąć (czemu? nie mam pojęcia, nie dało się mu wytłumaczyć), dlatego ktoś kiedyś zrobił klasę, która służy tylko do tego, żeby wyciągać zmienne globalne przez nią - jest napisana z nadzieją, że kiedyś uda nam się odciąć od tych globali, niestety ten ktoś wsadził tam static i zamiast wstrzykiwać odpowiednie zależności do innych klas używał w nich Klasa::get('costam'). W tym co ja pisałem, używając tej klasy, wstrzykiwałem jej obiekt jako zależność, no ale nie mogę tak po prostu usunąć tego "static" - bo inne funkcjonalności przestaną działać. Stąd cały problem.
Czyli ogólnie to static jest nie do ruszenia - bo tak
Mam zatem np klasę do pobierania jakiegoś tam linku która w metodzie get() ma np. $this->globals->get('pageUrl').'/'.$this->pobierzTamJakisLink(). Chciałem ją przetestować mockując $this->globals->get() tak, żeby zawsze zwracał pusty ciąg znaków.

@LowiczakPL
A:

[PHP] pobierz, plaintext 
<?php
 
namespace Example;
 
 
class A
{
    public function doSomething()
    {
        return ('A->doSomething');
    }
}
[PHP] pobierz, plaintext 

B:

[PHP] pobierz, plaintext 
<?php
 
namespace Example;
 
 
class B
{
    public static function doSomething()
    {
        return ('B->doSomething');
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
 public function testExample(){
 
        $a = $this->getMockBuilder('Example\A')->setMethods(['doSomething'])->getMock();
        $a->expects($this->any())->method('doSomething')->will($this->returnValue('AMock->doSomething'));
 
        $b = $this->getMockBuilder('Example\B')->setMethods(['doSomething'])->getMock();
        $b->expects($this->any())->method('doSomething')->will($this->returnValue('BMock->doSomething'));
 
        var_dump($a->doSomething());
        var_dump($b->doSomething());
        exit;
    }
[PHP] pobierz, plaintext 

Zwraca:
PHPUnit 5.4.4 by Sebastian Bergmann and contributors.

string(18) "AMock->doSomething"


Time: 65 ms, Memory: 4.00MB


WARNINGS!
Tests: 1, Assertions: 0, Warnings: 1.

[Crozin]

Możesz spróbować wyrzucić odwołania do nieszczęsnej klasy statycznej do pośredniczącego obiektu.

[PHP] pobierz, plaintext 
class Config {
  public static get(...);
  public static set(...);
}
 
interface ConfigProviderInterface {
  public get();
  public set();
}
 
// implementacja wykorzystywana w normalnym kodzie
class StaticConfigProvider implements ConfigProviderInterface {
    public get() {
        return Config::get(...);
    }
 
    public set() {
        Config::set(...);
    }
}
 
// implementacja wykorzystywana w testach
class ArrayConfigProvider implements ConfigProviderInterface {
    private $config;
 
    public get() {
        return $this->config[...];
    }
 
    public set() {
        $this->config[...] = ...;
    }
}
 
class MyClass {
  private $config;
 
  public __construct(ConfigProviderInterface $config) {
    $this->config = $config;
  }
 
  public doSth() {
    return 'abc' . $this->config->get(..);
  }
}
 
// ---- TESY:
 
$config = new ArrayConfigProvider([....]);
$class = new MyClass($class);
 
$this->assertEquals('abc123', $class->doSth());
[PHP] pobierz, plaintext 

[kapslokk]

To ma sens będę próbował w wolnej chwili Dzięki

Ten post edytował kapslokk 10.06.2016, 07:48:36