smoketest: T5447: wpa_supplicant is only run if required

1 files changed, 4 insertions, 19 deletions

diff --git a/smoketest/scripts/cli/test_interfaces_macsec.py b/smoketest/scripts/cli/test_interfaces_macsec.py
index 30d1ad659..ea0f00071 100755
--- a/smoketest/scripts/cli/test_interfaces_macsec.py
+++ b/smoketest/scripts/cli/test_interfaces_macsec.py
@@ -139,15 +139,9 @@ class MACsecInterfaceTest(BasicInterfaceTest.TestCase):
         # final commit and verify
         self.cli_commit()
         self.assertIn(interface, interfaces())
-        self.assertIn(interface, interfaces())
-        self.assertEqual(cipher, get_cipher(interface))
 
-        # check that we use the new macsec_csindex option (T4537)
-        tmp = get_config_value(src_interface, 'macsec_csindex')
-        self.assertIn("0", tmp)
-
-        # Check for running process
-        self.assertTrue(process_named_running(PROCESS_NAME))
+        # Verify proper cipher suite (T4537)
+        self.assertEqual(cipher, get_cipher(interface))
 
     def test_macsec_gcm_aes_256(self):
         src_interface = 'eth0'
@@ -168,18 +162,12 @@ class MACsecInterfaceTest(BasicInterfaceTest.TestCase):
         # final commit and verify
         self.cli_commit()
         self.assertIn(interface, interfaces())
-        self.assertEqual(cipher, get_cipher(interface))
-
-        # check that we use the new macsec_csindex option (T4537)
-        tmp = get_config_value(src_interface, 'macsec_csindex')
-        self.assertIn("1", tmp)
 
-        # Check for running process
-        self.assertTrue(process_named_running(PROCESS_NAME))
+        # Verify proper cipher suite (T4537)
+        self.assertEqual(cipher, get_cipher(interface))
 
     def test_macsec_source_interface(self):
         # Ensure source-interface can bot be part of any other bond or bridge
-
         base_bridge = ['interfaces', 'bridge', 'br200']
         base_bond = ['interfaces', 'bonding', 'bond200']
 
@@ -205,9 +193,6 @@ class MACsecInterfaceTest(BasicInterfaceTest.TestCase):
             self.cli_commit()
             self.assertIn(interface, interfaces())
 
-        # Check for running process
-        self.assertTrue(process_named_running(PROCESS_NAME))
-
     def test_macsec_static_keys(self):
         src_interface = 'eth0'
         interface = 'macsec5'