Your tests take too long to run

On a big application with lot of acceptance tests, integration tests, unit tests, performance tests, whatever you want tests you can wait sometime more than 1 hour to have your results.

YOU SHOULDN'T WAIT

There is not excuse. Whatever your change is, it doesn't justify that you have to waste one hour of your precious time. 

Sometime people says "During this time, I may use my brain to think about next tasks" you know what ? If I have to queue for one hour at the checkout, even if the cashier allow me to think about what I can do when I will be back at home, I just don't care and I want to burn the checkout, the cashier and the shop too.

SO BURN YOUR TESTS

Agree ? you don't want that anymore ? so what next

1. Refactor your test

Profiling tool are not only for your running application. For instance, I gained 50% of time changing a simple configuration in jBehave's steps retrieving in my current project. It wouldn't have happen without profiling.

2. Use crowd testing !!!

You are not alone on your journey there are guys that can help you : your colleagues and your continuous integration server.

Sound weird ? let me explain :

• 90 % of the time you know witch test may have been impacted by your change. (You might want to look at the next chapter at this point but don't do it unless you want to come back here later and it will be all scrambled in your mind)
• Run those tests on your machine. (You should take less than 5 minutes to do so) 
• Run your new crazy maven/make/ant/gradle/yourOwnStuffThatIsSoCool goal that will run only a chunk of the tests (Your are 7 in your team run only 1/5 of all tests for instance)
• Ask all your colleagues to do so.
• Grab the result, fix your test

How to do that ? That's up to you, but some ideas :

Randomly chosen chunk. If you and your colleague run tests quite often maybe it is reasonable to choose randomly your tests to run and hope to have a failure quite soon

Here is some example with different size of chunk and number of builds required to have all tests run at least once.

	chunck size	0.1
Number of build	0	5	10	15	20	30
Chance that a test is run	0%	41%	65%	79%	88%	96%

By running only 1/10 of your tests, you have to run your build 50 times in order to make sure (>99 %) all tests are run. If your team of 7 guys are running tests 5 times a days your get >96% chance that the failing test will be run in one day.

On top of that your tests now take virtually 5 minutes to run

	chunck size	0.2
Number of build	0	5	10	15	20	30
Chance that a test is run	0%	67%	89%	96%	99%	100%

By running only 1/5 of your tests, you have to run your build 20 times in order to make sure (>99 %) all tests are run.

On top of that your tests now take virtually 10 minutes to run  

	chunck size	0.5
Number of build	0	5	10	15	20	30
Chance that a test is run	0%	97%	100%	100%	100%	100%

By running only 50% of your tests, you have to run your build 20 times in order to make sure (>99 %) all tests are run.

On top of that your tests now take virtually 30 minutes to run


The good point of this strategy is that whatever the frameworks you use it should be easy to implement that.


Round Robin chunk

Better but less easy to put in place is to choose your chunk with an incremental system. It means that all test in the chunk you run will be different than for the next build. It is not always easy to implement since it should be stateful. (Some ideas later but you have to continue reading)


Almost Round Robin chunk
 
Each user of your system builds a predefined chunk of tests but in this solution the idea is to insure that if only x% of the guys are required to run all tests. (Yes some guys dare to be sick in my team)

It means that we are doing a more tests than our chunk. For instance if I have a team of 5 guys and if i want to have all tests runs even if 1 guy is absent I have to run 1/5 of the test + 1/4 tests to cover others colleagues chuncks   = 45 % of all tests 

Here is a reminder table :

		Number of colleagues
		4	5	6	7	8	9	10
number of colleagues potentially dead	1	58%	45%	37%	31%	27%	24%	21%
	2	92%	70%	57%	48%	41%	36%	32%
	3		95%	77%	64%	55%	49%	43%
	4			97%	81%	70%	61%	54%
	5				98%	84%	74%	66%
	6					98%	86%	77%
	7						99%	88%
	8							99%

So it's becoming quite interesting and worth investigation with large team.

Also one thing to understand is that you have to put an order to each members of your team and store it somewhere. (As a bash variable for instance)




Continuous distrubuted testing 

We can also think about having one way to have a daemon on each machine that run chunk of tests and communicate with each other to allow full test coverage in an acceptable time.

You can even think about having a predefined time to test and have a time limited chunk. (Chunk should take less than 5 minutes)

Most of the time you can even use your own VCS and commit one single little file that informs others what need to be tested in the next build
Also using your VCS you can easily verify that all the test has been run for a unique commit.

Although even if it s clearly cool, this solution is not easy to implement if you are thinking that you want test processes to be run in parallel

3. Test in priority what need to be tested

Challenge your project to know if it is really useful to run all tests always. does it worth it ? What are the benefits vs cost ? Don't be dogmatic

If you want to segregate your tests there's multiple strategies :
- Isolate business domains in your test (Annotation or other language artifact may help you)
- Create separate goal in your build tool to run only subsets of tests
- Link your test coverage tool with your build and VCS tool Unfortunatly it's your job here to do that, the idea is to say that if you modify one piece of code it should, most of the time, only impact new created test or test that previously cover the chunk of code you have modified.

My java 8 presentation

I did this presentation at Swissquote

That was great. First time I've considered using gimp to do a presentation. Was a great idea.

download the pdf

You say that a value is not used, prove it

Introduction 

Some time, we are doing tests by considering that some values are not relevant for the test so we put default value. But some time we actually rely on those default values to have our tests working. It can lead to hidden knowledge and false passed tests

For instance, if I create a class OldWomen. For my test instances It's reasonable to put 70 as a default value. It's an age that anyone (at least any developer) will consider as old.

Let say I have a medicine dispenser to test. I can have a story like this

Given a old woman with a cancer
when medicine dispenser is turned on
then the old woman remains in life


But for any reasons we decide that Women older than 80 always forget to plug in the medicine dispenser.

...our test will continue to work 



but in fact if the old woman is older than 80, she will die. (Indeed it's a simplified world)

Your test is not relevant and can make old women died.

Improving your default values

1. We are all using continuous integration, a same test in a normal team will probably be run more than 10 times a day (Most of the time just to say "Hello I'm here and I'm working fucking well").
2. We cannot test all cases 
3. We haven't the brain enough big to foreseen all impacts of our changes (If we could, we basically won't do any test).

So let's make the machine do the job. Let's implement a default value generator !!!

By randomly generating default value you will probably failed one day (and probably the first day) If your code isn't safe for all values.

Obviously this approach has some limitation and some corner cases will not be threated.

Some ideas to implement those generators

• Create a generic class  RandomValueGenerator<T>. You will be able to handle many cases this way
• When you create String value take care of alpha numeric/ASCII characters
• Generate nullable value if it's  relevant (for example 50% of the values returned will be null ). You can create a decorator to do that 

public static <U> RandomValueGenerator<U> nullable(final RandomValueGenerator<U> randomValueGenerator) {

• Generate zero value if it's  relevant (for example 20% of the value returned will be 0)  you can also create a decorator to handle that. 
• Support enum and allow to have only a limited number of value RandomValueGenerator.fromPossibleValues(E... values) 
• Use it every where in your unit test or acceptance test (How much entity builder do you have in your code ?)
• It's difficult to find the good ranges for values (Is it usefull to generate amount that are equals to 100000000000000 $ ?). But don't assert too mu
• You can also combine some random generator :

With a method  RandomValueGenerator.combine(RandomValueGenerator... randomGenerator) 

 you can have
RandomValueGenerator.combine(
 RandomValueGenerator.fromPossibleValues(1.,0.,NaN),RandomValueGenerator.doubleValue())

Here it will provide you :

• half of the time a value that is known to be error prone
• half of the time a value that is actually random

Reproduce your tests

A good test should be reproductible. Fortunatly most of the random generator are initialize with a seed and given that seed all value are predictible. Then it's easy to have two mode :

1. By default, the seed is randomly chosen and we reinitialized the random generator at the beginning of all test to this seed
2. In order to reproduce a failing test, if a property is set with the seed, we use this seed.

Here is an exemple in Java

private final static Random random;

static {
if (System.getProperty("test.random.seed") == null) {
seed = new Random().nextLong();
} else {
seed = Long.getLong(System.getProperty("test.random.seed"));
}
random = new Random(seed);
}

public static void newTest() {
LOGGER.info("running test with random generator initialized with seed " + seed);
random.setSeed(seed);
}

You need to find a good way to call this newTest method on every tests. It's your job here !!! 

Non regression testing

At this point we tested that unexpected change in GIVEN values has no impact on THEN values but we also want to be sure that only expected values has changed on a test. Let's take again our old  example:

Given a old woman aged of 90 with a cancer
when medicine dispenser is turned on
then the old woman will die

But it's not only that the old woman will die that changed, there is also a good news, the electricity bill will not increase dramatically before people discover the old woman.

When you code do you always have in mind this kind of change ? It why non regression testing are so important. Although, I don't want to  change my test just to handle that case (It probably doesn't worth it) but I want to acknowledge that this change in behavior is ok.


There is no easy solution to solve this problem, but here some ideas :
- You need to compare state of the system at the end of the test with the expected state. It could be difficult or simple depending on your system. If your are in a db oriented project you may store the resulting database after the test.
- For the first run you cannot test anything (It's non regression testing... ) but you should store the results of this first test in the source base of the project.
- For non regression testing always use the same seeds. You can use lot of different seed but for each seed you should expect to have the same result.
- Reuse the seeds that have in the past broken your test.

Le test builder

"Putain c'est quoi ce que tu testes avec ce code de merde que tu dis être un test unitaire"

C'est par cette douce pensée que j'envisage souvent les codes reviews des tests unitaires qui me sont soumis.

Moi j'aime bien les tests unitaires, je les respecte et j'essaye de les rendre heureux ou du moins utiles. Sauf que des fois, c'est le drame, une méthode à mocker toute tordue, dix scénarios de test pour la même méthode, le resultat à vérifier qui devient trop compliqué et le test devient trop complexe, illisible et donc un peu inutile.


D'ou un petit pattern que je me suis amusé à créer : le test builder 

Le test builder c'est quoi ?

C'est une classe utilitaire permettant d'ecrire les scénarios de test unitaire pour une méthode à la mode DSL.

Heu... le test builder c'est quoi ?

bon d'accord, prenons un exemple concret. J'ai une méthode qui s'appele

public Collection<ObjetAVendreAvecLeurPrix> queVendreAMémée(Mémée laMémée){
}

dans la classe VRPService

Vous l'aurez compris, cette méthode permet de connaître une liste d'object que l'on peux vendre à mémée et leur prix en fonction de la mémée.

Cette méthode est sans doute assez compliquée à tester et il faudra sans doute de nombreuses mémées différentes pour la tester correctement.

 on aura par exemple les tests suivants :
 testQueVendreAMémée_qandMéméeEstRicheEtGénéreuse
 testQueVendreAMémée_qandMéméeEstRicheMaisRadine
 testQueVendreAMémée_qandMéméeEstPauvreEtRadine
 testQueVendreAMémée_qandMéméeEstPauvreEtGénéreuse

On supose aussi, pour les besoins de l'exemple, que la méthode queVendreAMémée utilise un service richesseDeMéméeService avec une méthode isMéméeRiche(Mémée laMémée)



Un des test unitaire resemblerait à ca

@Test
public void testQueVendreAMémée_qandMéméeEstRicheEtGénéreuse(){
 
 
  RichesseDeMéméeService richesseDeMéméeService = mock(RichesseDeMéméeService.class);
  Mémée méméeRicheEtGénéreuse = new Mémé();
  mémée.setGénérosité(Mémée.Généreuse);
  mémée.ajouteAuxObjetsQuePossedentMémée(ObjectAVendre.Aspirateur);
  when(richesseDeMéméeService.isMéméeRiche(méméeRicheEtGénéreuse)).thenReturn(true);

  Collection<ObjetAVendreAvecLeurPrix> resultats = new VRPService().QueVendreAMémée(méméeRicheEtGénéreuse);

  assertEquals(1,resultats.size);
  assertEquals(ObjectAVendre.Télévision,resultats.get(0).getObjectAVendre()); // ok get(0) n'existe pas dans Collection mais bon, les mémées riches et généreuses non plus et personne ne me fait chier avec ça
  assertEquals(100.000,resultats.get(0).getPrix());

}

vous avez vu c'est assez chiant à lire et pourtant c'est assez clair je pense. Et puis pour tester la mémée pauvre ça va faire de la copie de code inutile.


Bref transformons ça avec notre test builder

tout d'abord on va ecrire une classe interne notre "test builder"


private static class QueVendreAMéméeTestBuilder{
  Collection<ObjetAVendreAvecLeurPrix> resultats;
  Mémée laMéméeATesté = new Mémée();
  RichesseDeMéméeService richesseDeMéméeService = mock(RichesseDeMéméeService.class);

  public QueVendreAMéméeTestBuilder méméeGénéreuse(Boolean généreuse){
    laMéméeATesté.setGénérosité((généreuse)?Mémée.Généreuse:Mémée.Radine);
    return this;
  }

  public QueVendreAMéméeTestBuilder méméePossède(ObjectAVendre object){
    laMéméeATesté.ajouteAuxObjetsQuePossedentMémée(object);
    return this;
  }


  public QueVendreAMéméeTestBuilder méméRiche(Boolean riche){
    when(richesseDeMéméeService.isMéméeRiche(laMéméeATesté)).thenReturn(riche);
    return this;
  }

  public QueVendreAMéméeTestBuilder regardeCeQueTuPeuxVendreAMémée(){
    resultats = new VRPService().QueVendreAMémée(laMéméeATesté);
    return this;
  }

  public QueVendreAMéméeTestBuilder nombreDobjetAVendre(int nb){
    assertEquals(nb,resultats.size);
    return this;
  }

  public QueVendreAMéméeTestBuilder onPeuxVendre(ObjectAVendre object, double aQuelPrix){
    for (ObjectAVendre o : resultats){
      if (o.getObjectAVendre().equals(object)){
    assertEquals("Mémée peux acheter " + object + " a " o.getPrix() " et pas à " + aQuelPrix, aQuelPrix, o.getPrix());
    return this;
      }
    }

    assertFail("Mémée n'a jamais voulu acheter : " + object);
  }
}



ensuite on va créer le test qui va avec

@Test
public void testQueVendreAMémée_qandMéméeEstRicheEtGénéreuse(){
  new QueVendreAMéméeTestBuilder()
      .méméeGénéreuse(true)
      .méméeRiche(true)
      .méméePossède(ObjectAVendre.Aspirateur)
      .regardeCeQueTuPeuxVendreAMémée()
      .nombreDobjetAVendre(1)
      .onPeuxVendre(ObjectAVendre.Télévision,100.000);
}


et comme je suis généreux comme mémée je vous offre la version méméePauvreEtGénéreuse
@Test
public void testQueVendreAMémée_qandMéméeEstPauvreEtGénéreuse(){
  new QueVendreAMéméeTestBuilder()
      .méméeGénéreuse(true)
      .méméeRiche(false)
      .méméePossède(ObjectAVendre.Aspirateur)
      .regardeCeQueTuPeuxVendreAMémée()
      .nombreDobjetAVendre(1)
      .onPeuxVendre(ObjectAVendre.ProgrammeTV,500);
}


Reste une question en suspens, que fait la mémé pauvre avec son programme TV sans TV ? Laissez vos commentaires.

developpement d'une librairie pour prioriser des traitements

Imaginons que vous travaillez pour une chaine de restauration rapide. A l'heure du rush, un bus rempli de chef d'état se gare sur votre parking et se disent "on se tapperais bien un ptit mc do". Des lors, vous vous dites que ça serait pas mal de les servir rapidement... mais quand même faut pas déconner les clients normaux c'est quand même eux qui ramène les pépetes le reste de l'année et il va bien falloir les servir sans qu'ils soient trop laisés.

Il y a quelques mois, j ai été  confronté à ce genre de problématique. Sauf que ce n'était pas des sandwichs que je fournissais mais de jolis retour de méthode.

De là est né une petite librairie que vous n' avez jamais entendu parlé : priority queue proxy.

Le priority proxy c'est l' idée de rendre transparent la gestion de la priorisation des appels de services que ce soit au niveau consumer ou producer. Et comme son nom l' indique on fait tout ça avec un beau proxy java et des thread queues.

Tout ca marche merveilleusement bien avec un bon vieux spring mais pas que.  

Petit exemple avec spring:

On crée l'éxecuteur, c'est lui qui s'occupe de dire qui s'execute ou 

<bean id="priorityQueueProxyExecutor" class"....priorityExecutor.Executor">
<constructor-args value="10">
</bean>

L'argument du constructeur est la taille du tread pool partagé par tous les clients. C'est sur ce thread pool que sont gérées les priorités.


On créer les services :

<bean id="lowPriorityService" factory-bean="je vous l'explique plus tard">
              <constructor-arg value="l'interface du service"/>
              <constructor-arg><bean class="monImplementationDeService"></constructor-arg>
              <constructor-arg>5</constructor-arg>
              <constructor-arg>0</constructor-arg>

              <constructor-arg>priorityQueueProxyExecutor</constructor-arg>
</bean>


<bean id="highPriorityService" ...

              <constructor-arg value="l'interface du service"/>
              <constructor-arg>...</constructor-arg>
              <constructor-arg>10</constructor-arg>
              <constructor-arg>5</constructor-arg>
              <constructor-arg>priorityQueueProxyExecutor</constructor-arg>
</bean>


Explication, On a maintenant deux services différents qui représentent en réalité la même chose mais avec des priorité différentes : 

Le premier service a une priorité de 5 et n'a pas de thread réservé pour lui tout seul.

Le deuxième service a une priorité de 10 et possede 5 slot privé qu'il est seul a utilisé. (Des caisses réservé pour les présidents dans le cas du McDo)




Tout ça est tres joli et ou je le telecharge me direz vous ? Bah soit vous venez travaillez avec moi dans ma boite, soit vous ne pouvez-pas.

Mais n' envisagez pas pour autant le suicide car je vous propose de faire la version 2 en LGPL avec des fonctionnalités à vous faire pousser des rouflaquettes:

• Gestion de pool de thread privé plus fine.
• Meilleure gestion de la concurence (utilisation eventuel des packages java.util.concurent).
• Tread affinity pour une configuration intelligente des threads. 
• Gestion des priorités même sur une ferme de provider de service (un cloud pour faire jeune)
• Gestion des priorités changeantes (Finalement les types dans le bus etaient certes des chefs d' état mais surtout d'anciens dictateurs balayés par les révolutions arabes) 
• Configuration des priorités au runtime...   
• D'autres idées m' étaient venues mais je mettrais à jour plus tard car ça ne vient plus  

Alors tenté par une aventure open source ?

Supporting i18n inside Javascript backend JAVA

Résumé :

Quand je code du javascript avec du .jsp (ou .gsp ou .ceQueJeVeux) j'aime bien écrire mon code dans une classe javascript. Mais quand je commence à toucher à l'internationalisation, je suis tenté (30 secondes) de mettre mon javascript tout moche dans mes belles pages afin de pouvoir utiliser les resources i18n.

Cet Article va vous montrer trois façons différentes de pouvoir gérer cette problématique plus ou moins proprement. La troisieme technique vous permettra de plus de briller en société (ou pas).

J'exclus toute idée de faire de l'ajax pour faire ces traduction parce que franchement ça peut etre vite crade et faire plein de requêtes pas franchement utiles.

1 - Du javascript dans mes pages...

La premiere idée est la plus simple mais sans doute la moins bonne. L'idée est de faire une page qui contient que les traductions des éléments javascript et l'importer dans nos pages (quelque soit la technique)

Une telle page peut resembler à ca :

<%@ taglib prefix="fmt" uri="http://java.sun.com/jsp/jstl/fmt"%>

<script type="text/javascript">
var i18n = new Object();
i18n.disclaimerTitle = "<fmt:message key="disclaimer.title"/>";
i18n.cancelDisclaimer = "<fmt:message key="disclaimer.cancelButton"/>";
i18n.confirmDisclaimer = "<fmt:message key="disclaimer.confirmButton"/>";
i18n.serverTimeOut = "<fmt:message key="serverTimeOut"/>";
i18n.pleaseWait = "<fmt:message key="pleaseWait"/>";
</script>


Ensuite dans son javascript on à plus qu'a utiliser notre objet par exemple : <script type="text/javascript">
alert(i18n.disclaimerTitle); </script>


C'est bien parce que :

• C'est simple à mettre en place.

C'est nul parce que :

• On récupère du coup toutes les traductions même si on en a pas besoin.
• Nos javascript ne fonctionnent pas sans la traduction (ca peut etre résolu je vous laisse réfléchir comment)

2- Du javascript généré à la volée.

L'idée c'est d'appeler le javascript en passant par une servlet (ou un autre truc du genre).

Notre servlet à plusieurs responsabilités.

• Récupérer la langue (ça peut être un paramètre de la requête)
• Récupérer le javascript non traduit
• Traduire les parties à traduire
• Renvoyer le fichier
• Eventuellement, géré un cache

pour traduire les parties à traduire on peut utiliser se bout de code     protected String translateJS(String jsString, String lang) {
        Pattern pattern = Pattern.compile("i18n\\{[a-zA-Z0-9.]*\\}");
        Matcher matcher = pattern.matcher(jsString);

        Boolean find = false;
        StringBuffer translatedJsString = new StringBuffer();
        while (matcher.find()) {
            find = true;
            String key = matcher.group().substring(5, matcher.group().length() - 1);
            matcher.appendReplacement(translatedJsString, translate(key, lang));
        }
        if (!find) {
            translatedJsString.append(jsString);
        }

        return translatedJsString.toString();
    }


Le test unitaire qui va avec pour mieux comprendre : public class TestI18nJavascriptServlet {
    private I18nJavascriptServlet i18nJavascriptServlet;

    @Before
    public void initMock() {
        i18nJavascriptServlet = Mockito.spy(new I18nJavascriptServlet());
        Mockito.doReturn("test1-en").when(i18nJavascriptServlet).translate(Mockito.eq("test1"), Mockito.eq("en"));
        Mockito.doReturn("test2-en").when(i18nJavascriptServlet).translate(Mockito.eq("test2"), Mockito.eq("en"));

    }

    @Test
    public void testNoTranslation() {
        String js = "no translation'";
        Assert.assertEquals(js, i18nJavascriptServlet.translateJS(js, "en"));
    }

    @Test
    public void testSimpleTranslation() {
        String js = "translation for i18n{test1}'";
        Assert.assertEquals("translation for test1-en", i18nJavascriptServlet.translateJS(js, "en"));
    }

    @Test
    public void testMultiTranslation() {
        String js = "translation for i18n{test1} and i18n{test2}'";
        Assert.assertEquals("translation for test1-en and test2-en", i18nJavascriptServlet.translateJS(js, "en"));
    }

}


C'est bien parce que :

• On récupere les traductions au besoin.
• Les pages javascript sont fonctionnelles en static mais pas top (on à pas de traduction par défaut. Quoi qu'on peut reprendre l'idée du chapitre suivant).

C'est nul parce que :

• C'est un peut chiant à mettre en place.
• Ca me permet pas de briller en société

3- Du javascript généré à la volée version spring EL.

C'est presque la même solution que la précédente mais elle devient un peu plus élégante. Au lieu de faire du parsing comme précédement, on va utiliser les expression langage de Spring (SpEL)

Le code type POC que j'explique apres:

    public class TemplateParserContext implements ParserContext {
        public String getExpressionPrefix() {
            return "i18n(";
        }
        public String getExpressionSuffix() {
            return ")";
        }
        public boolean isTemplate() {
            return true;
        }
    }

    @Test
    public void elParser() throws SecurityException, NoSuchMethodException {
        String jsTextToTranslate = "alert(i18n.translate('hello','Salut les filles')))";
        jsTextToTranslate = jsTextToTranslate.replaceAll("i18n.translate", "i18n(#translate");
        ExpressionParser parser = new SpelExpressionParser();
        StandardEvaluationContext context = new StandardEvaluationContext();
        context.registerFunction("translate", TestI18NSpelJavascript.class.getDeclaredMethod("translate", String.class, String.class));
        String translated = parser.parseExpression(jsTextToTranslate, new TemplateParserContext()).getValue(context, String.class);
        Assert.assertEquals("alert(\"translated:hello->Salut les filles\")", translated);
    }

    public static String translate(String key, String defaultValue) {
        return '\"' + "translated:" + key + "->" + defaultValue + '\"';
    }
WTF ? me direz vous...

• l'inner classe me sert à définir pour SPel l'endroit ou chercher les expression langages : dans mon cas je dis que c'est dans une chaine i18n(monEl)
• j'enregistre une function translate pour le parser SpEL qui comme son nom l'indique permet de faire le travail de traduction.
• La transformation du javascript avec le replaceAll me permet d'avoir du coté javascript original : i18n.translate(maclef,matraductionparDefault) et du coté du javascript à transformer via el : i18n(#translate(maclef,matraductionparDefault)). Car coté SpEL, une fonction commence toujours par # (c'est le code antlr généré donc y a pas moyen de modifier ça à la volée)
• je fais gaffe que ma méthode translate me renvoie une string au sens js. (les petites quotes qui vont bien)
• J'importe dans ma page original "i18n.js" qui défini un objet i18n qui possède la fonction translate qui renvoie la défault value

C'est bien parce que :

• On récupere les traductions au besoin.
• Les pages javascript sont fonctionnelles en static.
• On peut envisager de faire d'autres choses via SpEL même si je n'ai pas d'idée comme ça.  (si on s'en fous que le js ne marche plus en static sinon ... )
• Ca me permet de briller en société

C'est nul parce que :

• Il faut spring.
• C'est quand même un petit peu beaucoup

/// TODO finish translation Summary When i work with javascript + .JSP (or .gsp or .whatEverYouWant), i like to write my javascript code inside js class... but when internationalization involved, we put js code inside our pages in order to use java i18n ressources.

Tdd is not enough

Je viens de faire un petit truc en TDD et je me suis rendu compte que mon code était tout crade par rapport à ce qu'on pouvait faire.


L'idée c'était de faire une methode qui transforme les sauts de ligne ASCII en paragraphe ou en saut de ligne HTML
ex

Lalala
lilili

toto
tata

se transforme en

<p>Lalala<br/>Lilili</p><p>toto<br/>tata</p>

c'est fou non ?


Bref pour faire ça je me suis dis que j'allais sortir mon copain le TDD. Alors je fais mes tests au fur et à mesure du dev et j'obtiens ce jeu de tests

    @Test
    public void testFormatDisclaimerForHTMLPrinting() {
        Assert.assertEquals("<p>test</p>", DisclaimerAction.formatDisclaimerForHTMLPrinting("  \ntest"));
        Assert.assertEquals("<p>test</p>", DisclaimerAction.formatDisclaimerForHTMLPrinting("  \ntest\n"));
        Assert.assertEquals("<p>test<br/>test</p>", DisclaimerAction.formatDisclaimerForHTMLPrinting("  \ntest\ntest"));
        Assert.assertEquals("<p>test</p><p>test</p>", DisclaimerAction.formatDisclaimerForHTMLPrinting("  \ntest\n   \ntest"));
        Assert.assertEquals("<p>test</p><p>test</p>", DisclaimerAction.formatDisclaimerForHTMLPrinting("  \ntest\n \n \n  \ntest"));
        Assert.assertEquals("<p>test</p><p>test</p>", DisclaimerAction.formatDisclaimerForHTMLPrinting("  \ntest\n \n \n  \ntest"));
        Assert.assertEquals("<p>test</p><p>test<br/>toto</p>",
                DisclaimerAction.formatDisclaimerForHTMLPrinting("  \ntest\n \n \n  \ntest\ntoto"));

    }

et surtout ce code tout crade :
    public static String formatDisclaimerForHTMLPrintingOld(String toFormat) {
        String[] lines = toFormat.trim().split("\n");
        String toReturn = "<p>";
        for (int i = 0; i < lines.length; i++) {
            if (StringUtils.isEmpty(lines[i].trim())) {
                toReturn += "</p><p>";
                while (i < lines.length - 1 && StringUtils.isEmpty(lines[i + 1].trim())) {
                    i++;
                }

            } else {
                toReturn += lines[i];
                if (i + 1 < lines.length && !StringUtils.isEmpty(lines[i + 1].trim())) {
                    toReturn += "<br/>";
                }
            }
        }

        return toReturn + "</p>";
    }

J'ai pas trop essayer de reflechir en écrivant ça. Juste de prendre le test qui marche pas et d'écrire le code qui le fait marcher.

Alors ce matin je regarde d'un oeil coupable ce code, je commence à réfléchir et j'écris ça :

public static String formatDisclaimerForHTMLPrinting(String toFormat) {
        String toReturn = toFormat.trim();
        toReturn = toReturn.replaceAll("\\s*\\n\\s*\\n\\s*", "</p><p>");
        toReturn = toReturn.replaceAll("\\n", "<br/>");
        return "<p>" + toReturn + "</p>";
}

plus concis, plus simple, plus joli... et il passe bien mes tests.


Dans le TDD y a bien cette phase de refactoring qui est précisée cependant c'est quand même un peu difficile parce qu'une fois que j'ai écris mon code crade j'ai beau temps de le laisser comme ça s'il y a une autre urgence.

hum, je ne sais quelle conclusion tirer de tout ça. Si ce n'est que l'approche originale de TDD est peut être un peu trop dogmatique et qu'il faut toujours réfléchir avant de coder (... arf toujours pas de solution contre ce mal).