PHPUnit criar mock de método.

[Diogo Alexsander 1]

Boa tarde, pessoal.

 

Estou há alguns dias tentando passar de um problema no PHPUnit, mas tá estranho.

Antes de tudo... sou novo no processo de TDD.

 

Estou tentando "mockar" um método de uma classe, e quero me certificar de que quando eu chame este método, ele chame um terceiro método.

 

Bom, segue código com comentários do que eu quero fazer mas não estou conseguindo...

 

 

Classe: Usuario

<?php

namespace Minha\Classe\Concreta;

class Usuario
{
    /** 
     * Este é o método que eu tento mockar, para posteriormente, verificar se ele chama o método teste2.
     */
    public function teste()
    {
        $this->teste2();
    }

    private function teste2()
    {

    }
}

O teste unitário para ela...

<?php

class UsuarioTest extends \PHPUnit_Framework_TestCase
{
    public function testMockingMethods()
    {
        // Crio um mock da minha classe Usuário com o método teste.
        $mock = $this->getMockBuilder('Minha\Classe\Concreta\Usuario')
            ->disableOriginalConstructor()
            ->setMethods(['teste'])
            ->getMock();

        // Espero que o método teste2 seja chamado uma vez apenas.
        $mock->expects($this->once())
            ->method('teste2');

        // Chamo o método "teste" para rodar o teste de fato e certificar que "teste2" foi chamado.
        $mock->teste();
    }
}

Ao rodar esse teste, o resultado é o seguinte...

vagrant@vm:/www$ (master) ✖ ./library/vendor/bin/phpunit
PHPUnit 4.8.21 by Sebastian Bergmann and contributors.

Runtime:    PHP 5.5.9-1ubuntu4.17 with Xdebug 2.2.3
Configuration:  /www/teste/phpunit.xml

F

Time: 13.13 seconds, Memory: 14.25MB

There was 1 failure:

1) UsuarioTest::testMockingMethods
Expectation failed for method name is equal to <string:teste2> when invoked 1 time(s).
Method was expected to be called 1 times, actually called 0 times.

Pois bem, eu não sei o que está errado na minha classe de teste.

Já verifiquei várias vezes na própria documentação do PHPUnit mas não sei o que estou fazendo errado ainda.

 

Agradeço desde já.

 

 

 

[Gabriel Heming 766]

O mock é para substituir uma funcionalidade, como por exemplo, ao invés de escrever em arquivo (que seria a funcionalidade original do método) escrever em memória. Você não chega a estar substituindo-a, apenas quer garantir que o test chame test2.

 

Seu questionamento entra mais no caso de testes sobre métodos privados, que não são aconselhados.

http://stackoverflow.com/questions/105007/should-i-test-private-methods-or-only-public-ones

 

Caso seu método privado tenha uma alta complexidade, então, nesse caso, seria aconselhado você separar a lógica dele em outros métodos ou, até, em outra classe, conforme a técnica de refatoração Method Object

[Diogo Alexsander 1]

.

[Gabriel Heming 766]

Pelo o que eu entendo (não consigo testar no momento) você não precisa realizar o mock de teste2, pois não tem intenção de mudar o comportamento do método. Se você o fizer, as chamadas internas não serão mais realizadas.

 

Se você deseja realizar a alteração do comportamento do método teste2, adicione no setMethods, senão, não o utilize.

 

Os demais passos parecem estarem corretos.

 

Nos exemplos apresentados no link mencionado, há a questão de alterar o comportamento do método sim_ou_nao. Pois em um momento ele precisa retornar um valor e, em outro, o valor contrário.

 

Ps.: Eu exclui um dos seus posts duplicados e você removeu o texto do outro.

[Diogo Alexsander 1]

Quanto eu meu post anterior, na verdade eu esqueci de responder com citação da primeira vez, então dupliquei a resposta e coloquei a citação, pra daí apagar o primeiro. Você apagou o que eu tinha corrigido, hehe.

 

Entendi. Só devo fazer mock quando quero alterar o comportamento do método então, pra que seja testado nas várias possíveis situações.

Bom, eu vou marcar este tópico como resolvido. O que você respondeu agora vai me fazer escrever os testes de forma diferente e eu vou pesquisar mais sobre o uso de mocks.

 

Muito obrigado pelo esclarecimento.

[Gabriel Heming 766]

De certa forma, como não consigo testar o comportamento no momento, é o que meu conhecimento está me deduzindo.

 

Caso necessário, podes voltar a este tópico.

 

Sobre o post apagado:

Citações de posts em sequência, são removidas por recomendação, pois, são desnecessárias. Somente são recomendadas quando são para trechos ou posts mais antigos da thread. Conforme orientação nº 8 e como pode ser visto no tópico abaixo:

http://forum.imasters.com.br/topic/542539-atencao-orientacoes-e-regras-do-forum-de-php/

http://forum.imasters.com.br/topic/464950-resolvidodesativar-cliente-depois-de-1-mes/

 

Como o conteúdo de ambas era igual, só mudava a citação, e, posteriormente, eu removeria a citação, ficava mais fácil apenas excluir o post.

[Diogo Alexsander 1]

Certo, entendi.

 

Obrigado.