New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Post sobre Traits #39

Merged
merged 3 commits into from Mar 2, 2016

Conversation

Projects
None yet
4 participants
@mlalbuquerque
Copy link
Member

mlalbuquerque commented Feb 23, 2016

Mais um mini post, dessa vez sobre traits e como usei.

@jonataa

This comment has been minimized.

Copy link
Collaborator

jonataa commented Feb 23, 2016

Massa! Uma dúvida da galera tb é como testar uma Trait, já que não pode ser instanciada ou chamada de maneira estática. O único jeito que encontrei foi "mockando" o objeto com PHPUnit, conforme a documentação deles. Ainda não vi essa parte de testes com Traits em artigos brasileiros, fica aí como sugestão, quem sabe nos próximos.

@mlalbuquerque

This comment has been minimized.

Copy link
Member

mlalbuquerque commented Feb 24, 2016

Boa... Vou colocar que mostrarei como fazer isso no próximo post. Massa
mesmo

Em Ter, 23 de fev de 2016 20:15, Jonata Weber notifications@github.com
escreveu:

Massa! Uma dúvida da galera tb é como testar uma Trait, já que não pode
ser instanciada ou chamada de maneira estática. O único jeito que encontrei
foi "mockando" o objeto com PHPUnit, conforme a documentação
https://phpunit.de/manual/current/en/test-doubles.html#test-doubles.mocking-traits-and-abstract-classes
deles. Ainda não vi essa parte de testes com Traits em artigos brasileiros,
fica aí como sugestão, quem sabe nos próximos.


Reply to this email directly or view it on GitHub
#39 (comment).

Marcio Albuquerque

Analista de Infra SERPRO
Celular - 071 98880 4994

@mlalbuquerque

This comment has been minimized.

Copy link
Member

mlalbuquerque commented Feb 24, 2016

@jonataa, coloquei o adendo sobre testest com traits. VALEU!! @paulodealmeida, @hugohenrique, @gjuniioor, @gwmoura, @vinaocruz, se estiver tudo ok, vamos fazer o merge!!

@hugohenrique

This comment has been minimized.

Copy link
Member

hugohenrique commented Feb 24, 2016

Interessante o adendo @mlalbuquerque, vocês acham interessante comentar sobre como testar uma classe que usa um mock?

Alguma implementação tipo:

trait ConnectionRenew
{
    protected function reconnect()
    {
        $this->getConnection()->ping();
    }

    abstract protected function getConnection();
}

class SomethingService
{
    use ConnectionRenew;

    private $connection;

    public function __construct($connection)
    {
        $this->connection = $connection;
    }

    protected function getConnection()
    {
        return $this->connection;
    }
}
@mlalbuquerque

This comment has been minimized.

Copy link
Member

mlalbuquerque commented Mar 1, 2016

Cadê, pessoal? Letra ou não o post? E cadê os outros posts. Temos iniciativa, mas temos que começar a ter acabativa!!

@jonataa

This comment has been minimized.

Copy link
Collaborator

jonataa commented Mar 1, 2016

@mlalbuquerque eu não tinha visto seu adendo, esperava isso no próximo post! Ficou bom, por mim está ok!

+1

@jonataa

This comment has been minimized.

Copy link
Collaborator

jonataa commented Mar 1, 2016

Seria bom criar a seguinte regra: se tiver 2 ou 3 votos, pode realizar o merge. Mas, só dê o voto se realmente estiver ok.

@gildasio

This comment has been minimized.

Copy link
Collaborator

gildasio commented Mar 1, 2016

@mlalbuquerque, acredito que entendi a referència! 🌝

Não entendo muito de traits, mas me parece ok. De qualquer forma, bom esperar mais um voto e então, merge.

@hugohenrique

This comment has been minimized.

Copy link
Member

hugohenrique commented Mar 2, 2016

@mlalbuquerque o post está bacana!
O que você acha sobre o comentário que fiz acima?

No mais 👍 para merge

@mlalbuquerque

This comment has been minimized.

Copy link
Member

mlalbuquerque commented Mar 2, 2016

@hugohenrique queria comentar o que? Eu fiz um adendo sobre testes com Traits. Era isso?

jonataa added a commit that referenced this pull request Mar 2, 2016

@jonataa jonataa merged commit b4de885 into phpba:master Mar 2, 2016

@jonataa

This comment has been minimized.

Copy link
Collaborator

jonataa commented Mar 2, 2016

Mais um, para nossa alegria! =)

@gildasio

This comment has been minimized.

Copy link
Collaborator

gildasio commented Mar 2, 2016

lol, vi agora: Aquela "nota mental" é daquele jeito mesmo? Deve faltar um espaço apenas. Só confirma ai :)

@mlalbuquerque

This comment has been minimized.

Copy link
Member

mlalbuquerque commented Mar 2, 2016

Era pra estar como cabeçalho h6. Pq não apareceu assim? Alguém sabe
corrigir?

Em Qua, 2 de mar de 2016 16:53, Gildásio Júnior notifications@github.com
escreveu:

lol, vi agora: Aquela "nota mental" é daquele jeito mesmo? Deve faltar um
espaço apenas. Só confirma ai :)


Reply to this email directly or view it on GitHub
#39 (comment).

Marcio Albuquerque

Analista de Infra SERPRO
Celular - 071 98880 4994

@jonataa

This comment has been minimized.

Copy link
Collaborator

jonataa commented Mar 2, 2016

Acho que resolve dando um espaço antes da palavra, como @gjuniioor falou:

Errado

######Teste

Certo

###### Teste
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment