diff options
| author | Iain Lane <iain.lane@canonical.com> | 2018-12-19 12:30:09 +0000 |
|---|---|---|
| committer | Iain Lane <iain.lane@canonical.com> | 2018-12-19 12:30:09 +0000 |
| commit | dfe107070a97e8cf943ccf4f88e7f1c3718de4d6 (patch) | |
| tree | 9417f66719c73f5280cd2cbf8861b3b1ec8b381c /debian/tests | |
| parent | be0549a79eba3b672fde7e9b8702b0747e1ade51 (diff) | |
debian/tests/nm.py: Make assert_iface_down() not check the interface's state
We call nmclient.deactivate_connection() to terminate connections that the testsuite sets up, and according to upstream this is not guaranteed to do anything in particular to the link state. It seems that dnsmasq 2.80 somehow alters the previous assumption that it would be 'state DOWN', so the implementation detail we were checking previously no longer holds. The testsuite does still check that the IPs are removed from the interface, which is logically what we want anyway. (LP: #1805857)
Diffstat (limited to 'debian/tests')
| -rwxr-xr-x | debian/tests/nm.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/debian/tests/nm.py b/debian/tests/nm.py index 64350e08..ead66f5a 100755 --- a/debian/tests/nm.py +++ b/debian/tests/nm.py @@ -192,7 +192,6 @@ class NetworkManagerTest(network_test_base.NetworkTestBase): out = subprocess.check_output(['ip', 'a', 'show', 'dev', iface], universal_newlines=True) - self.assertIn('state DOWN', out) self.assertNotIn('inet 192', out) self.assertNotIn('inet6 2600', out) |