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:25 +0100 |
| commit | cbd2d71fc85f89f322f1d5c85052034b0b57b3b9 (patch) | |
| tree | a98489f7b26c33c1887262543f20a2b44f30d3e6 /smoketest | |
| parent | 344e6d6a9074e2da8e5865f46b4122edfe1ef6a3 (diff) | |
| download | vyos-1x-cbd2d71fc85f89f322f1d5c85052034b0b57b3b9.tar.gz vyos-1x-cbd2d71fc85f89f322f1d5c85052034b0b57b3b9.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.
Diffstat (limited to 'smoketest')
| -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() |