From: Subrata M. <su...@li...> - 2008-02-21 15:05:25
|
> On Tue, 2008-01-15 at 10:30 +0100, Aurélien Charbon wrote: > > Hi, > > > > I have run the whole test suite after having applied this little patch. > > There should not be any difference between the original test run and a > > run post patch applied since there are no new code in runtest scripts to > > launch the new tests for RPC. > > Do you think it is necessary to add an entry in the scripts ? > > Yes, this is absolutely necessary, else, how do people run your new test > cases and be sure that it works fine too. It should also be able to run > (the new test cases) when people invoke ltp/testscripts/networktests.sh. > Hi Aurélien Charbon, I was expecting some work around for my above comments. You need to make changes to files: 1) ltp/runtest/rpc, and, 2) ltp/runtest/stress.part3, so that both rpc-simple (old), and, latest rpc test cases are run. This will automatically take care of: 1) ltp/testscripts/networktests.sh, and, 2) ltp/testscripts/networkstress.sh, I am still to apply any of your Patches to LTP. It would be great if you can send me: 1) An updated complete (everything) Patch settling all the above issues, which also includes the new test cases, (Patch over latest LTP CVS diff),and, 2) Results of test run post patching. Regards-- Subrata > --Subrata > > > > > Regards, > > > > Aurélien > > > > > > Subrata Modak wrote: > > > > >On Mon, 2008-01-07 at 18:55 +0100, Aurélien Charbon wrote: > > > > > > > > >>Hi, > > >> > > >>The patch i sent does not completely delete the existing testcases for RPC. > > >>In fact some simple tests were already in directory testcases/network/rpc. > > >>The patch places the existing tests in testcases/network/rpc/simple_tests > > >>The new ones are in testcases/network/rpc/rpc-tirpc-full-test-suite > > >> > > >>This must disturb the test launcher because I forgot to modify the rpc > > >>script (in runtest directory) > > >>I think this patch should fix the problem. > > >> > > >> > > >> > > > > > > > > >Have you run the tests after applying this patch. If yes, please provide > > >us with the results of the same test run post patch applied. It would be > > >great if you can also highlight the test result for the new additions > > >against the old one. > > > > > > > > > > > > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2008. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ > Ltp-list mailing list > Ltp...@li... > https://lists.sourceforge.net/lists/listinfo/ltp-list |