On Fri, Jun 27, 2008 at 10:37 AM, Damitha Kumarage <[EMAIL PROTECTED]> wrote:
> Hi, > I don't think we need to design new test suites at this time. What we need > to do is adding comprehensive number of tests into the existing test suite > sufficient for testing the focused code of refactoring. As I remember we > worked on two test suites Can you point where the test sources are available now ? -Rajika > in the past but did not proceed adding more tests to them. > > thanks, > Damitha > > Rajika Kumarasiri wrote: > >> >> >> On Fri, Jun 27, 2008 at 9:29 AM, Samisa Abeysinghe <[EMAIL PROTECTED]<mailto: >> [EMAIL PROTECTED]>> wrote: >> >> Supun Kamburugamuva wrote: >> >> Hi all, >> >> +1 for refactoring the HTTP transport sections and +1 for >> having a good set of test cases. >> >> Since we are agreeing to doing this, I suggest we should have >> a good design before going on. I think we have enough >> knowledge and experience to get this right at this point. So >> I'm looking forward to see a very good discussion in the >> mailing lists for making this right at this time. >> >> >> But testing and test suite has to be priority even beyond design >> discussion. That is one of the key areas that we lag. So I would >> like for us to discuss testing first. >> >> Adding a well formed test suite is important, because this will helps a >> developer to locate a bug easily, May be we can add test suite based on >> Deja GNU. >> >> -Rajika >> Then to discuss design. >> >> So how best can we test the transport code that we have? >> >> Thanks, >> Samisa... >> >> >> Regards, >> Supun.. >> On Fri, Jun 27, 2008 at 9:04 AM, Samisa Abeysinghe >> <[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]> >> <mailto:[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>>> wrote: >> >> Manjula Peiris wrote: >> >> Hi devs, >> >> Most of the code in Axis2/C transport is not organized >> properly. This >> makes it very difficult to do some changes or adding new >> functionality >> to the code. One main reason for this is REST handling and >> SOAP handling >> are mixed in the cod,e in a very ugly manner. >> Axis2/JAVA has a >> separate >> section called MessageBuilder which happened before the >> Transport stuff >> in order to separate out REST Invocations, SOAP invocations >> and other >> different messaging mechanisms. I propose we do >> something like >> that or >> at least separate out REST handling inside HTTP >> transport.(I >> mean at >> least separate REST code to different functions or .c >> files. ) >> This can >> be a post 1.5 fix. >> >> I am always +1 for refactoring and improvements. Mind you, >> one of >> the key pre-requisites of refactoring is testing, to >> guarantee pre >> and post refactoring code bases have the same behavior. >> Do we have a good enough test for this, if not, IMHO, we should >> build a good set of tests before we endeavor on this >> re-factoring >> mission. Else would we have tough time fixing >> post-refactoring bugs. >> >> Thanks, >> Samisa... >> >> Thanks, >> -Manjula. >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: >> [EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]> >> <mailto:[EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]>> >> >> For additional commands, e-mail: >> [EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]> >> <mailto:[EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]>> >> >> >> ------------------------------------------------------------------------ >> >> >> No virus found in this incoming message. >> Checked by AVG. Version: 8.0.101 / Virus Database: >> 270.4.1/1519 - Release Date: 6/25/2008 4:13 PM >> >> >> -- Samisa Abeysinghe Director, Engineering; WSO2 Inc. >> >> http://www.wso2.com/ - "The Open Source SOA Company" >> >> >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: >> [EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]> >> <mailto:[EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]>> >> >> For additional commands, e-mail: >> [EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]> >> <mailto:[EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]>> >> >> >> >> >> ------------------------------------------------------------------------ >> >> >> No virus found in this incoming message. >> Checked by AVG. Version: 8.0.101 / Virus Database: >> 270.4.1/1521 - Release Date: 6/26/2008 11:20 AM >> >> >> >> -- Samisa Abeysinghe Director, Engineering; WSO2 Inc. >> >> http://www.wso2.com/ - "The Open Source SOA Company" >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]> >> For additional commands, e-mail: [EMAIL PROTECTED] >> <mailto:[EMAIL PROTECTED]> >> >> >> >> >> -- >> comp.lang.c - http://groups.google.com/group/comp.lang.c/topics >> > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- comp.lang.c - http://groups.google.com/group/comp.lang.c/topics
