Hi, Unitary tests fixed
Antonio Cet e-mail a été envoyé depuis un ordinateur protégé par Avast. www.avast.com <https://www.avast.com/fr-fr/lp-esg-fav?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=OA-2109-B> <#DDB4FAA8-2DD7-40BB-A1B8-4E2AA1F9FDF2> 2016-03-17 12:38 GMT+01:00 sebb <[email protected]>: > On 17 March 2016 at 11:20, Antonio Gomes Rodrigues <[email protected]> > wrote: > > I will fix the unit test asap > > > > Can you provide the checkstyle configuration to run it in local before > > commit the change? > > It's in SVN. > > But as already noted it does not catch everything. > > The best is to follow the layout convention for the file that you are > working on. > This may vary between files (and certainly does for source file types). > > > Antonio > > > > Cet e-mail a été envoyé depuis un ordinateur protégé par Avast. > > www.avast.com > > < > https://www.avast.com/fr-fr/lp-esg-fav?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=OA-2109-B > > > > <#DDB4FAA8-2DD7-40BB-A1B8-4E2AA1F9FDF2> > > > > 2016-03-17 12:15 GMT+01:00 Vladimir Sitnikov < > [email protected]>: > > > >> Antonio>Do we have a checkstyle configuration or something like that > >> to avoid waste > >> Antonio>time (your time and mine) in how to code to JMeter? > >> > >> There's a checkstyle (it is run as a part of Travis job), however it > >> would never check 100% of the issues. > >> My "else branch swap" request was just to make sure the diff does not > >> bring unintentional modifications. > >> > >> By the way, have you seen that Travis job fails? Any chances your fix > that? > >> > >> Vladimir > >> >
