Tests:

• Readable (i.e. simple, car plus facile à maintenir)
• Isolated OR Integrated
• Thorough (testé aux limites)
• Explicit (un test contient tout ce qu'il faut pour tourner tout seul)

Tiens, justement, je mets ça en place un peu plus drastiquement en ce moment.

Ce Joël a toujours le chic de faire rire quoi qu'il écrive :

Les tests unitaires en Java, c'est le pied grâce à JUnit ! C'est vraiment le pied ! Dans un projet, j'ai même eu l'occasion de m'en servir pour des tests d'intégrations !
Cela devient plus compliqué dès qu'il faut tester des méthodes asynchrones utilisant des callback (ou listener, as your prefer). Je n'ai rien de complètement figé dans ma tête actuellement, mais voilà ce que je note :

• Le framework Awaitility (http://code.google.com/p/awaitility/) permettrait de faire ça proprement, mais ça ne m'a pas tenté.
• Certains disent qu'il faut mocker et faire en deux étapes... Mouais, à mon avis ce n'est pas adaptable dans tous les cas. Pour un test d'intégration système par exemple, il vaut mieux éviter de mocker.

• La solution générique consiste à attendre l'appel du thread à l'aide de synchronized, wait et notify. Dans la réalité, il semble qu'il vaut mieux utiliser CountDownLatch (http://java.sun.com/j2se/1.5.0/docs/api/java/util/concurrent/CountDownLatch.html) afin d'éviter que le callback n'arrive avant que l'on est eu le temps d'attendre... ce serait dommage ;-)
  Actuellement, pour moi, la meilleure solution consiste donc à utiliser un code du type :
  
public class AsynchronousTest
{
/**

  • Asynchronous helper
    */
    private CountDownLatch lock = new CountDownLatch(1);
    /**
  • Received data from the asynchronous call
    */
    private Object receiveddata;

  @Test
  public void testAsynchronousCall() throws Exception
  {
  Object parameters = null;
  myAsynchronousCall(parameters, new MyCallback()
  {
  @Override
  public void onSuccess(Object data)
  {
  receiveddata = data;
  lock.countDown();
  }
  });

  boolean releaseBeforeTimeout = lock.await(2000, TimeUnit.MILLISECONDS);
  // Check timeout
  if (!releaseBeforeTimeout) {
          // Do stuff
          fail("Timeout");
  }
  
  assertNotNull(receiveddata);
  // Other tests

  }
}