diff options
author | Daniil Baturin <daniil@vyos.io> | 2023-10-07 18:13:41 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-10-07 18:13:41 +0100 |
commit | 0d975350d0a953ef7dc311c165335f7f2977e107 (patch) | |
tree | 980b5cdd88e185dbd4303495caf6fdda3fa82e6a /smoketest/scripts | |
parent | f1eac571f22af7779ef7264425e0cc51d1aaae11 (diff) | |
parent | e357258e645cf85de0035d4ecfbf99db4dd90f7e (diff) | |
download | vyos-1x-0d975350d0a953ef7dc311c165335f7f2977e107.tar.gz vyos-1x-0d975350d0a953ef7dc311c165335f7f2977e107.zip |
Merge pull request #2335 from c-po/t5630-pppoe-mru
pppoe: T5630: allow to specify MRU in addition to already configurable MTU
Diffstat (limited to 'smoketest/scripts')
-rwxr-xr-x | smoketest/scripts/cli/test_interfaces_pppoe.py | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/smoketest/scripts/cli/test_interfaces_pppoe.py b/smoketest/scripts/cli/test_interfaces_pppoe.py index 0ce5e2fe0..7b702759f 100755 --- a/smoketest/scripts/cli/test_interfaces_pppoe.py +++ b/smoketest/scripts/cli/test_interfaces_pppoe.py @@ -59,10 +59,12 @@ class PPPoEInterfaceTest(VyOSUnitTestSHIM.TestCase): user = f'VyOS-user-{interface}' passwd = f'VyOS-passwd-{interface}' mtu = '1400' + mru = '1300' self.cli_set(base_path + [interface, 'authentication', 'username', user]) self.cli_set(base_path + [interface, 'authentication', 'password', passwd]) self.cli_set(base_path + [interface, 'mtu', mtu]) + self.cli_set(base_path + [interface, 'mru', '9000']) self.cli_set(base_path + [interface, 'no-peer-dns']) # check validate() - a source-interface is required @@ -70,6 +72,11 @@ class PPPoEInterfaceTest(VyOSUnitTestSHIM.TestCase): self.cli_commit() self.cli_set(base_path + [interface, 'source-interface', self._source_interface]) + # check validate() - MRU needs to be less or equal then MTU + with self.assertRaises(ConfigSessionError): + self.cli_commit() + self.cli_set(base_path + [interface, 'mru', mru]) + # commit changes self.cli_commit() @@ -80,6 +87,8 @@ class PPPoEInterfaceTest(VyOSUnitTestSHIM.TestCase): tmp = get_config_value(interface, 'mtu')[1] self.assertEqual(tmp, mtu) + tmp = get_config_value(interface, 'mru')[1] + self.assertEqual(tmp, mru) tmp = get_config_value(interface, 'user')[1].replace('"', '') self.assertEqual(tmp, user) tmp = get_config_value(interface, 'password')[1].replace('"', '') |