[sr-dev] http_client api test

[Olle E. Johansson]

> On 01 Jun 2016, at 08:43, Daniel-Constantin Mierla <miconda at gmail.com> wrote:
> 
> Hello,
> 
> this probably needs a more elaborate discussion and planning, being somehow related to unit testing that is planned to be approached for 5.0.
> 
> IMO, anything for testing a module should be in a subfolder inside the module directory or inside a directory dedicated for testing. It won't be that nice to have them at the same level with module directories, will be a lot of noise there as a module may have more than one unit test.
> 
> 
Let’s create a module_test directory that hosts various test modules then. Naming the directory “test” makes it clear for users that it’s
not a normal production-use directory. I think creating test modules for modules that implement API is a good thing. If we can automate
tests that’s even better. I’ll use this module as a test case and we’ll figure it out.

If we come up with something else and need to move it, users should not be affected, only developers. I know that we are a grumpy
group of persons that doesn’t want to change anything, but let’s hope that we can survive us messing around with test code :-)

/O
> Cheers,
> Daniel
> 
> On 31/05/16 22:46, Olle E. Johansson wrote:
>> Hi!
>> 
>> In order to be able to test the API for the http_client I have created a separate module. I couldn’t let it live in the same directory, so I created a new module directory for it. If I keep it in the same directory the Makefile automatically links it with the http_client so-file, which is not what I want.
>> 
>> I would like to store it somewhere - I can either commit is a module that doesn’t compile by default, store it in a hidden branch and forget about it or get help in fixing the Makefile so it can optionally compile in the same directory.
>> 
>> Which would be the way forward?
>> 
>> Right now it doesn’t do much more than bind to the API, but hopefully it will soon be able to test all the provided API functions.
>> 
>> /O
>> _______________________________________________
>> sr-dev mailing list
>> sr-dev at lists.sip-router.org <mailto:sr-dev at lists.sip-router.org>
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev <http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev>
> 
> -- 
> Daniel-Constantin Mierla
> http://www.asipto.com <http://www.asipto.com/> - http://www.kamailio.org <http://www.kamailio.org/>
> http://twitter.com/#!/miconda <http://twitter.com/#!/miconda> - http://www.linkedin.com/in/miconda <http://www.linkedin.com/in/miconda>_______________________________________________
> sr-dev mailing list
> sr-dev at lists.sip-router.org
> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20160602/fa611941/attachment.html>