Escrevendo testes unitários para código legado: técnicas de isolamento

Escrevendo
testes unitários
para código legado:
técnicas de isolamento
André Ricardo Barreto de Oliveira (“Arbo") 
Core Software Engineer @ Liferay

Banco
de Dados +
Objeto
Relacional
Teste
automatizado
Orientação
a Objetos
Ambientes de
Desenvolvimento
Servidores
de Aplicação
+ REST
+ SOAP
Distribuição
+ Nuvem
Bibliotecas +
Frameworks
Refactoring +
Análise EstáticaProﬁling
Comunidade++
2015:
20 anos
Integração
Contínua Programação
Funcional +
Scala

Em várias empresas perto de você:
"O Gigantesco Projeto
Escrito em Java"
10+ anos em produção
milhares de classes
milhões de linhas
de código desktop / web / mobile
centenas de jars, wars, ears
dúzias de frameworks
… e crescendo!

Mas e os testes?
“Veja bem…"

Projetos Java
Gigantescos
QA
Testes
Manuais
Selenium
Desenvolvedores
Banco de Dados
Spring
Runner
customizado

Manual Prático
de Paraquedismo
"In the industry, legacy code
is slang for difﬁcult-to-change
code that we don't understand.
!
To me, legacy code
is simply code without tests."
!
- Michael C. Feathers

Testes de Caracterização
Classe não tem testes?
Escreva um teste que apenas
documenta o comportamento atual.

Testes de Caracterização
Feliz com a cobertura?
Implemente a nova funcionalidade.

public class MassMailingServiceTest
{
!
@Test
public void whatcangowrong()
{
new MassMailingService();
}
!
}

DatabaseException:
!
Você precisa estar conectado ao banco
de dados para realizar esta operação!

public MassMailingService()
{
this.limit =
SettingsFromDatabaseService
.getLimit();
}
#FAIL
Que fazer?

Alternativa 1
1. Estudar a documentação do framework
2. Instalar / importar / emprestar uma base de dados
3. Popular a base com os dados de teste
4. Logar na base
5. Rodar o teste

public MassMailingService(
Settings settings) // interface
{
this.limit = settings.getLimit();
}
Quando você pode alterar
a classe de negócio...

@Test
{
Settings s =
Mockito.mock(Settings.class);

Mockito.when(s.getLimit())
.thenReturn(42);

new MassMailingService(s);
}

@Test
{
PowerMockito.mockStatic(
SettingsFromDatabaseService.class);
!
PowerMockito.stub(method(
SettingsFromDatabaseService.class,
"getLimit"))
.toReturn(42);
!
}
Quando você não pode
alterar a classe de negócio...

public class MassMailingServiceTest
{
!
@Test
public void send()
{
new MassMailingService().send(
new Message("Hello"),
"andre.oliveira@liferay.com",
"andre@arbo.com.br");
}
!
}

new MassMailingService().send(
new Message("Hello"),
30 segundos depois…
Você possui 1 (uma) nova
mensagem
em
sua caixa postal
Você possui 1 (um
a) nova
m
ensagem
em
sua caixa postal

public void send(
Message message, String... targets)
{
for (Address address : targets)
{
RealSMTPSender
.send(message, address);
}
}
#FAIL

@Test
public void send()
{
Sender s = Mockito.mock(Sender.class);
Message message = new Message("Hello");

new MassMailingService(s).send(message,

Mockito.verify(s).send(message,
"andre.oliveira@liferay.com");

Mockito.verify(s).send(message,
}

@Test
public void send()
{
PowerMockito.mockStatic(
RealSMTPSender.class);
Message message = new Message("Hello");
new MassMailingService().send(message,
PowerMockito.verifyStatic();
RealSMTPSender.send(message,
"andre.oliveira@liferay.com");
PowerMockito.verifyStatic();
RealSMTPSender.send(message,
}

Testes unitários
com isolamento
http://martinfowler.com/bliki/UnitTest.html
Martin Fowler

Isolamento e legado
Testes de Caracterização?
Código legado, sem testes?
Isolamento vai te ajudar

Isolamento e legado
Código novo,
testes novos?
Use o bom senso, ou…
"TDD is dead. Long live testing"
http://david.heinemeierhansson.com/2014/tdd-is-dead-long-live-testing.html
!
"Is TDD dead?"
http://martinfowler.com/articles/is-tdd-dead/

100% de cobertura?
Sim, é possível!

if (service.result() > 5) { /* caso especial */ }

@Test public void happyDay() {
when(service.result()).thenReturn(1);
// do it + assert happy day
}
!
@Test public void casoEspecial() {
when(service.result()).thenReturn(42);
// do it + assert caso especial
}
Condicionais e casos especiais
Cada if branch deriva um caso de teste

try { service.danger(); }
catch (OpaException e) { /* caso especial */ }
!
@Test public void sorryDay() {
when(service.danger())
.thenThrow(OpaException.class);
// do it + assert caso especial
}
Tratamento de exceções
Cada catch branch deriva um caso de teste

if (pessoa.idade() < 0) {
throw new IdadeNegativaException();
}
!
@Expected(IdadeNegativaException.class)
@Test public void wtf() {
when(pessoa.idade()).thenReturn(-99);
// do it (vai lançar a exception)
}
Validações
Simulando entradas impossíveis com mocks

Resultado: cobertura de todos
os branches de execução
!
Novas funcionalidades /Refactorings

Bugﬁxes:
cobertura
mínima
• Rastrear a linha de código do bug
• Escrever teste para o fragmento de lógica
• Mock mínimo que reproduz o problema
• Protip: Extrair método testável

Testes unitários com
isolamento: benefícios
2
3
4
5
6
Rodam em < 1 segundo
(como integração
chegavam a 30+)
Dispensam preparar
base de dados ou
serviços
Fácil conseguir 100%
de cobertura
Blindagem contra
colaboradores mal
comportados
Simular exceptions,
casos especiais e dados
ruins
Modularização de
Projetos Java
Gigantescos
1

Happy testing!
André Oliveira
andre.oliveira@liferay.com
twitter.com/arbocombr
github.com/arboliveira/unit-tests-with-isolation

Escrevendo testes unitários para código legado: técnicas de isolamento

Recommandé

Recommandé

Contenu connexe

Similaire à Escrevendo testes unitários para código legado: técnicas de isolamento

Similaire à Escrevendo testes unitários para código legado: técnicas de isolamento (20)

Escrevendo testes unitários para código legado: técnicas de isolamento