Vou partir do princípio que não iremos entrar no mérito de como refatorar um código com métodos estáticos, pois eu entendo que isto seria uma outra discussão.
Vamos as suas perguntas.
Se podemos fazer, quando devemos fazer?
Normalmente você precisa de um mock de método estático quando ele guarda muita complexidade que simplesmente tentar contorná-la acaba não sendo viável.
Quais são os casos em que mock de método estático é benéfico?
Exemplo de uma classe que usa método estático complexo:
class ClassComEstatico () {
public void metodo(String variavel) {
//algum codigo simples
BigDecimal valor = CalcularUtils.calculoComplexo(variavel); // problema...
}
}
O CalcularUtils.calculoComplexo(variavel)
pode ser um verdadeiro buraco negro, chamando outros Singletons, dependendo de outros métodos estáticos, chamando services por ServiceLocator
... a imaginação é o limite. E em um teste você teria que lidar com todas estas dependências escondidas deste método aparentemente inofensivo. Neste caso, acaba sendo benéfico criar um mock para ele.
Se o método do CalcularUtils
fosse um simples arredondar(variavel)
você não precisaria se preocupar em mockar ele, dada a simplicidade do método. Por isto normalmente não nos preocupamos em mockar os Utils
/Helpers
.
Quando consigo detectar que o uso desse mock está sendo abusivo?
Entendo que isto começa a ocorrer quando seu setup
de teste acaba tendo muitos mocks de métodos estáticos. Isto pode gerar uma confusão muito grande, porque você não saberá identificar facilmente qual estático mockado corresponde a qual parte do seu código sendo testado pelo teste unitário. Exemplo:
@Test
public void teste() {
//mock de estático 1PowerMockito.mockStatic(ClassComEstaticos1.class);
//mock de estático 2PowerMockito.mockStatic(ClassComEstaticos2.class);
//mock de estático 3PowerMockito.mockStatic(ClassComEstaticos3.class);
Resultado resultado = service.calcular(1, 2);
//asserts
}
Assim, vêem as perguntas: Quem precisa do mock de estático 1ClassComEstaticos1
? E do 2ClassComEstaticos2
? E do 3ClassComEstaticos3
? É o próprio método calcular()
que precisa ou é algum método que o calcular
chama internamente? Ou ainda é outra classe que o calcular
usa? E qual destes mocks ele usa?
Isto fará com que seja extremamente difícil de manter os testes. Abusivo ou não, se você precisar fazer o teste não terá como desviar deles a menos que consiga reescrever o código e removendo os métodos estáticos.