This is an automated email from the git hooks/post-receive script. New commit to branch develop in repository wao. See http://git.codelutin.com/wao.git commit a9b400f2504df440fe4d9e210d403a936d5b3bb3 Author: Brendan Le Ny <bleny@codelutin.com> Date: Tue Nov 4 10:55:27 2014 +0100 Déplacement d'un assertion de test dans un verify guava --- .../src/main/java/fr/ifremer/wao/WaoUtils.java | 17 ++++++++++++++++- .../src/test/java/fr/ifremer/wao/WaoUtilsTest.java | 12 ------------ 2 files changed, 16 insertions(+), 13 deletions(-) diff --git a/wao-persistence/src/main/java/fr/ifremer/wao/WaoUtils.java b/wao-persistence/src/main/java/fr/ifremer/wao/WaoUtils.java index 0da34ca..00f0c76 100644 --- a/wao-persistence/src/main/java/fr/ifremer/wao/WaoUtils.java +++ b/wao-persistence/src/main/java/fr/ifremer/wao/WaoUtils.java @@ -183,10 +183,25 @@ public class WaoUtils { count.increment(); aDay = DateUtils.addDays(aDay, 1); } while ( ! aDay.after(observationEndDate)); // not after équivaut à before ou equals + + int observationDaysTotal = 0; Map<Date, Integer> result = new TreeMap<>(); for (Map.Entry<Date, MutableInt> entry : tempResult.entrySet()) { - result.put(entry.getKey(), entry.getValue().intValue()); + Date month = entry.getKey(); + int observationDays = entry.getValue().intValue(); + observationDaysTotal += observationDays; + result.put(month, observationDays); } + + // on vérifie la cohérence de notre répartition sur les différents mois en vérifiant qu'on a a bien réparti tous les jours d'observations du contact sur un mois + int expectedObservationTimeInDays = getObservationTimeInDays(observationBeginDate, observationEndDate); + Verify.verify( + expectedObservationTimeInDays == observationDaysTotal, + "Pour la période " + observationBeginDate + " à " + observationEndDate + ", on a " + + expectedObservationTimeInDays + " jours d'observation mais on a réparti " + observationDaysTotal + + " sur les différents mois " + result + ); + return result; } diff --git a/wao-persistence/src/test/java/fr/ifremer/wao/WaoUtilsTest.java b/wao-persistence/src/test/java/fr/ifremer/wao/WaoUtilsTest.java index 81c9246..816ffb3 100644 --- a/wao-persistence/src/test/java/fr/ifremer/wao/WaoUtilsTest.java +++ b/wao-persistence/src/test/java/fr/ifremer/wao/WaoUtilsTest.java @@ -93,18 +93,6 @@ public class WaoUtilsTest { "La répartition aurait dû se faire sur " + expectedSize + " mois et non " + observationDaysByMonths, expectedSize, observationDaysByMonths.size()); - int expectedObservationTimeInDays = WaoUtils.getObservationTimeInDays(observationBeginDate, observationEndDate); - int actualObservationTimeInDays = 0; - for (Integer monthValue : observationDaysByMonths.values()) { - actualObservationTimeInDays += monthValue; - } - - Assert.assertEquals( - "Il faut répartir " + expectedObservationTimeInDays + " sur les différents mois et non " + actualObservationTimeInDays + ": " + observationDaysByMonths, - expectedObservationTimeInDays, - actualObservationTimeInDays - ); - } } \ No newline at end of file -- To stop receiving notification emails like this one, please contact codelutin.com SCM administrator <admin+scm@codelutin.com>.