Hi Matthieu Looks like I need some help here. I have changed the tests to use junit4. However, I still have issues running UDPTest, which ran fine before.
As you can see, both NettyTest and UDPTest extend the same base class, they just differ in the modules they use. NettyTest runs just fine and when I try to run the UDPTest, it says "No test to run". You can find the tests at https://github.com/kambatla/s4-piper/tree/master/subprojects/s4-comm/src/test/java/org/apache/s4/comm Thanks Karthik On Wed, Oct 26, 2011 at 11:27 AM, Matthieu Morel <[email protected]> wrote: > Hi Karthik, > > I see you've made improvements to the comm layer tests, unfortunately, the > udp test hangs during the build, even from a vanilla checkout from leo's > master branch (and before my last commit). > > I haven't looked into that issue, which is unrelated to classloading, but I > temporarily set the UDPTest as abstract so that it does not block the whole > build. (See my comments in the class headers) > > Can you have a look at it? > > Also, note that tests should use junit4. It will actually allow you to > easily set timeouts for tests! > > Thanks! > > Matthieu > > On Fri, Oct 21, 2011 at 6:31 AM, Karthik Kambatla <[email protected] > >wrote: > > > I am trying to modify s4-comm tests so that they can be run by 'gradle > > test'. However, it fails at the s4-base:test itself. How do I go past > this? > > > > Matthieu, when possible can you please look at the modified test in my > > branch and comment - > > > > > https://github.com/kambatla/s4-piper/tree/fix-comm-test/subprojects/s4-comm/src/test/java/org/apache/s4/comm > > . > > Thank you very much. > > > > Karthik > > > > On Thu, Oct 20, 2011 at 3:33 AM, Karthik Kambatla < > [email protected] > > >wrote: > > > > > Hi Matthieu > > > > > > Agreed. I ll look into that and fix it. > > > > > > Thanks > > > Karthik > > > > > > PS: It might take a little bit as I haven't used JUnit before, this > > > actually is the first time I wrote a test. Perils of academic work :-) > > > > > > > > > On Thu, Oct 20, 2011 at 3:24 AM, Matthieu Morel < > > [email protected]>wrote: > > > > > >> Hi Karthik and Leo, > > >> > > >> I had a quick look at the pull request. It's an excellent idea to add > > such > > >> a > > >> test. However, I am concerned about the approach taken: the test is a > > java > > >> app with a main method. If you expect developers to evaluate that the > > test > > >> passes by reading the output on the console, that's unrealistic. > > >> > > >> Can you refactor the test by using JUnit, as in the other existing > > tests? > > >> The idea is to validate the test through assertions. There are already > > >> examples in the S4 codebase. > > >> > > >> Thanks, > > >> > > >> Matthieu > > >> > > >> On Thu, Oct 20, 2011 at 4:00 AM, Leo Neumeyer <[email protected]> > > >> wrote: > > >> > > >> > DONE. Thanks! > > >> > > > >> > On Oct 19, 2011, at 6:15 PM, Karthik Kambatla wrote: > > >> > > > >> > > Hi Leo > > >> > > > > >> > > I have sent you a pull request - > > >> > https://github.com/leoneu/s4-piper/pull/4 - > > >> > > to test the basic functionality of s4-comm protocols. It has only > a > > >> wee > > >> > bit > > >> > > of code, as I wanted to get a hang of the whole pull and merge > > >> process. > > >> > :) > > >> > > Let me know if I should be any more thorough with anything else. > > >> > > > > >> > > Thanks > > >> > > Karthik > > >> > > > >> > > > >> > > > > > > > > >
