Fixed, sorry about that. Fwiw, the problems were not related to the tests in the PR, which I fixed before committing, but from a later commit.
2016-09-12 11:00 GMT+02:00 Jean-Baptiste Onofré <[email protected]>: > Hi all, > > following the last commits on the shell grep action, the build is broken > on master due to test failures: > > Tests in error: > GrepTest.testGrep:47->systemInOutDecorator:300 » NullPointer > GrepTest.testGrepCount:86->systemInOutDecorator:300 » NullPointer > GrepTest.testGrepCountInvert:96->systemInOutDecorator:300 » NullPointer > > I gonna fix that. > > Sorry for the inconvenience. > > FYI: I started to review the Pull Request and mentioned these tests > failures to the contributor. So please, as reminder: > 1/ in a pull request, when you see a "R: @..." comments, it means a > reviewer started to work on the pull request > 2/ merge a pull request only when LGTM is set and the full build is OK > > Thanks > Regards > JB > -- > Jean-Baptiste Onofré > [email protected] > http://blog.nanthrax.net > Talend - http://www.talend.com > -- ------------------------ Guillaume Nodet ------------------------ Red Hat, Open Source Integration Email: [email protected] Web: http://fusesource.com Blog: http://gnodet.blogspot.com/
