diff options
| author | Christian Poessinger <christian@poessinger.com> | 2021-01-26 20:07:21 +0100 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2021-01-26 20:49:55 +0100 |
| commit | b48890396dcff074d1e9336ca1db9d3eafe2a12f (patch) | |
| tree | 323856c78e25d9d4676a52ed9d19072a03845d1a /smoketest/scripts/cli/test_interfaces_dummy.py | |
| parent | cec848e4a668ed51868d5fa50504a2f8bb1b4eb7 (diff) | |
| download | vyos-1x-b48890396dcff074d1e9336ca1db9d3eafe2a12f.tar.gz vyos-1x-b48890396dcff074d1e9336ca1db9d3eafe2a12f.zip | |
smoketest: refactor setUp() for all interface based testcases
It does not make sense (at all!) to re-initialize the entire class with every
call to setUp(). We neither change the enabled/disabled tastcases dynamically,
not do we adjust the testinterfaces during a run.
Remove the runtime overhead and place one-time init calls into setUpClass()
instead.
Diffstat (limited to 'smoketest/scripts/cli/test_interfaces_dummy.py')
| -rwxr-xr-x | smoketest/scripts/cli/test_interfaces_dummy.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/smoketest/scripts/cli/test_interfaces_dummy.py b/smoketest/scripts/cli/test_interfaces_dummy.py index 60465a1d5..6e462bccf 100755 --- a/smoketest/scripts/cli/test_interfaces_dummy.py +++ b/smoketest/scripts/cli/test_interfaces_dummy.py @@ -19,10 +19,10 @@ import unittest from base_interfaces_test import BasicInterfaceTest class DummyInterfaceTest(BasicInterfaceTest.BaseTest): - def setUp(self): - self._base_path = ['interfaces', 'dummy'] - self._interfaces = ['dum435', 'dum8677', 'dum0931', 'dum089'] - super().setUp() + @classmethod + def setUpClass(cls): + cls._base_path = ['interfaces', 'dummy'] + cls._interfaces = ['dum435', 'dum8677', 'dum0931', 'dum089'] if __name__ == '__main__': unittest.main(verbosity=2) |