[sr-dev] http_client api test

Olle E. Johansson oej at edvina.net
Thu Jun 2 09:26:42 CEST 2016


> On 02 Jun 2016, at 09:15, Daniel-Constantin Mierla <miconda at gmail.com> wrote:
> 
> 
> 
> On 02/06/16 09:09, Olle E. Johansson wrote:
>> 
>>> On 01 Jun 2016, at 08:43, Daniel-Constantin Mierla <miconda at gmail.com <mailto: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 :-)
> There is a folder 'test' now where are also the current units. Perhaps is the best place where to create the new folder for testing modules.
Ok
test/modules/src for source code
test/modules/cfg for test configurations

?

/O
> 
> Cheers,
> Daniel
> 
>> 
>> /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/#%21/miconda> - http://www.linkedin.com/in/miconda <http://www.linkedin.com/in/miconda>_______________________________________________
>>> 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>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20160602/2eca3c3c/attachment.html>


More information about the sr-dev mailing list