raharper, I have updated the docstrings

Diff comments:

> diff --git a/tests/vmtests/test_network_disabled.py 
> b/tests/vmtests/test_network_disabled.py
> new file mode 100644
> index 0000000..cf55030
> --- /dev/null
> +++ b/tests/vmtests/test_network_disabled.py
> @@ -0,0 +1,73 @@
> +# This file is part of curtin. See LICENSE file for copyright and license 
> info.
> +
> +from .releases import base_vm_classes as relbase
> +from .test_network import TestNetworkBaseTestsAbs
> +
> +from unittest import SkipTest
> +
> +import os
> +
> +
> +class CurtinDisableNetworkRendering(TestNetworkBaseTestsAbs):
> +    """ Test that curtin does not passthrough network config when
> +    networking is disabled."""
> +    conf_file = "examples/tests/network_disabled.yaml"
> +
> +    def test_cloudinit_network_not_created(self):
> +        cc_passthrough = "cloud.cfg.d/50-curtin-networking.cfg"
> +
> +        pt_file = os.path.join(self.td.collect, 'etc_cloud',
> +                               cc_passthrough)
> +        self.assertFalse(os.path.exists(pt_file))
> +
> +    def test_cloudinit_network_passthrough(self):
> +        raise SkipTest('not available on %s' % self.__class__)
> +
> +    def test_static_routes(self):
> +        raise SkipTest('not available on %s' % self.__class__)
> +
> +    def test_ip_output(self):
> +        raise SkipTest('not available on %s' % self.__class__)
> +
> +    def test_etc_resolvconf(self):
> +        raise SkipTest('not available on %s' % self.__class__)
> +
> +
> +class CurtinDisableCloudInitNetworking(TestNetworkBaseTestsAbs):
> +    """ Test disabling networking with network-config

Fixed

> +    with no version key."""
> +    conf_file = "examples/tests/network_config_disabled.yaml"
> +
> +    def test_etc_resolvconf(self):
> +        raise SkipTest('not available on %s' % self.__class__)
> +
> +    def test_ip_output(self):
> +        raise SkipTest('not available on %s' % self.__class__)
> +
> +
> +class CurtinDisableCloudInitNetworkingVersion1(
> +    CurtinDisableCloudInitNetworking
> +):
> +    """ Test disabling networking with network-config
> +    with version key."""

Fixed

> +    conf_file = "examples/tests/network_config_disabled_with_version.yaml"
> +
> +
> +class FocalCurtinDisableNetworkRendering(relbase.focal,
> +                                         CurtinDisableNetworkRendering):
> +    __test__ = True
> +
> +
> +class FocalCurtinDisableCloudInitNetworkingVersion1(
> +    relbase.focal,
> +    CurtinDisableCloudInitNetworkingVersion1
> +):
> +    __test__ = True
> +
> +
> +class FocalCurtinDisableCloudInitNetworking(relbase.focal,
> +                                            
> CurtinDisableCloudInitNetworking):
> +    __test__ = True
> +
> +
> +# vi: ts=4 expandtab syntax=python


-- 
https://code.launchpad.net/~lamoura/curtin/+git/curtin/+merge/383785
Your team curtin developers is subscribed to branch curtin:master.

-- 
Mailing list: https://launchpad.net/~curtin-dev
Post to     : curtin-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~curtin-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to