diff options
| author | Christian Poessinger <christian@poessinger.com> | 2021-01-05 21:15:02 +0100 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2021-01-05 21:16:45 +0100 |
| commit | 13a58d38b3dc8065a8ba71904e143e3d69aab638 (patch) | |
| tree | 421ee63e76daccf776a95bcc29ab1356e71be090 /smoketest/scripts/cli | |
| parent | 4ebe908b2d824a3cb7579de66efd20bdcbd401fc (diff) | |
| download | vyos-1x-13a58d38b3dc8065a8ba71904e143e3d69aab638.tar.gz vyos-1x-13a58d38b3dc8065a8ba71904e143e3d69aab638.zip | |
smoketest: tunnel: fix setUp() ordering for reference base class members
With commit ce809eee ("smoketest: mirror: T3169: re-add mirror / SPAN test case")
the setUp() method has been re-organized not taking into account that there will
already be a reference to self.session which will only be created by the base
class.
(cherry picked from commit cbd2d71fc85f89f322f1d5c85052034b0b57b3b9)
Diffstat (limited to 'smoketest/scripts/cli')
| -rwxr-xr-x | smoketest/scripts/cli/test_interfaces_tunnel.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/smoketest/scripts/cli/test_interfaces_tunnel.py b/smoketest/scripts/cli/test_interfaces_tunnel.py index f7b7f99ca..ca68cb8ba 100755 --- a/smoketest/scripts/cli/test_interfaces_tunnel.py +++ b/smoketest/scripts/cli/test_interfaces_tunnel.py @@ -67,9 +67,6 @@ class TunnelInterfaceTest(BasicInterfaceTest.BaseTest): self.local_v4 = '192.0.2.1' self.local_v6 = '2001:db8::1' - self.session.set(['interfaces', 'dummy', source_if, 'address', self.local_v4 + '/32']) - self.session.set(['interfaces', 'dummy', source_if, 'address', self.local_v6 + '/128']) - self._options = { 'tun10': ['encapsulation ipip', 'remote-ip 192.0.2.10', 'local-ip ' + self.local_v4], 'tun20': ['encapsulation gre', 'remote-ip 192.0.2.20', 'local-ip ' + self.local_v4], @@ -78,6 +75,9 @@ class TunnelInterfaceTest(BasicInterfaceTest.BaseTest): self._interfaces = list(self._options) super().setUp() + self.session.set(['interfaces', 'dummy', source_if, 'address', self.local_v4 + '/32']) + self.session.set(['interfaces', 'dummy', source_if, 'address', self.local_v6 + '/128']) + def tearDown(self): self.session.delete(['interfaces', 'dummy', source_if]) super().tearDown() |