Yes. That's right.
Also there is a unit test here [1] which mocks "ip netns exec".
[1] -
https://review.openstack.org/#/c/145005/5/neutron_vpnaas/tests/unit/services/vpn/device_drivers/test_ipsec.py
Thanks
Numan
On 01/21/2015 02:22 PM, Kevin Benton wrote:
So the test wouldn't make much sense then without the creation of the
namespace, right? If that's the case, it sounds like it is a very low
level functional test making sure that routes can be installed into
namespaces.
On Wed, Jan 21, 2015 at 12:19 AM, Numan Siddique
<numan.siddi...@enovance.com <mailto:numan.siddi...@enovance.com>> wrote:
It is asserting the return value of "ip netns exec <ns> ip route
get <ip_address>".
Thanks
Numan
On 01/21/2015 12:34 PM, Kevin Benton wrote:
Is the test asserting things about interactions with the system,
or does it just happen to use a system call as a side effect of
one of the setups?
On Tue, Jan 20, 2015 at 1:24 PM, Paul Michali <p...@michali.net
<mailto:p...@michali.net>> wrote:
My question is whether the tests proposed should be unit
tests or functional tests. They only test one method, and
it's not a complete piece of functionality - like creating a
VPN connection.
If that one system call is mocked, these could all be treated
as unit tests. So I'm wondering if there is an advantage in
actually testing the system call (getaddrinfo), as part of
this work?
Thoughts?
PCM (Paul Michali)
IRC............ pc_m (irc.freenode.com <http://irc.freenode.com>)
Twitter....... @pmichali
On Tue, Jan 20, 2015 at 3:02 PM, Kevin Benton
<blak...@gmail.com <mailto:blak...@gmail.com>> wrote:
I don't believe we have any unit tests that create
namespaces or veth pairs. This sounds like it belongs
with functional tests.
On Tue, Jan 20, 2015 at 10:20 AM, Numan Siddique
<numan.siddi...@enovance.com
<mailto:numan.siddi...@enovance.com>> wrote:
Hello,
I am working on a bug [1] on neutron vpnaas and
submitted the patch here [2].
The test code to test the fix does the following
- creates a namespace
- creates a veth pair and add one interface into
the namespace
- configures the interface with an ip address and
- adds a default gateway
- and of course tests the code.
This test code only tests a specific function
(OpenSwanProcess._get_nexthop())
Reviewers of this patch are not clear if this should
be part of functional tests or unit tests.
Can unit tests create linux namespaces, interfaces
etc or it falls under functional tests?
Please let me know your thoughts on this.
[1] - https://bugs.launchpad.net/neutron/+bug/1405413
[2] - https://review.openstack.org/#/c/145005/5
Regards
Numan
__________________________________________________________________________
OpenStack Development Mailing List (not for usage
questions)
Unsubscribe:
openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
<http://openstack-dev-requ...@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
--
Kevin Benton
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe:
openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
<http://openstack-dev-requ...@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe:
openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
<http://openstack-dev-requ...@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
--
Kevin Benton
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe:openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
<mailto:openstack-dev-requ...@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe:
openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
<http://openstack-dev-requ...@lists.openstack.org?subject:unsubscribe>
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
--
Kevin Benton
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
__________________________________________________________________________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev