continue instead of return

SUNET · Aug 22, 2024 · da57d0b · da57d0b
1 parent 1c0720f
commit da57d0b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/test_mfazones.py b/test_mfazones.py
@@ -257,7 +257,7 @@ def test_prepared_mfa_folders(self):
         for fullnode in g_drv.fullnodes:
             if g_testtarget=='prod' and fullnode == 'su':
                 self.logger.info(f'We are not testing su in prod right now')
-                return
+                continue
             with self.subTest(mynode=fullnode):
                 self.logger.info(f'TestID: {self._testMethodName} - {fullnode}')
                 prepareOcsMFaShares(fullnode)
@@ -274,7 +274,7 @@ def test_mfa_webdav_folders(self):
         for fullnode in g_drv.fullnodes:
             if g_testtarget=='prod' and fullnode == 'su':
                 self.logger.info(f'We are not testing su in prod right now')
-                return
+                continue
             with self.subTest(mynode=fullnode):
                 self.logger.info(f'TestID: Testing node {fullnode}')
 
@@ -315,7 +315,7 @@ def test_mfa_webdav_shared_folders(self):
         for fullnode in g_drv.fullnodes:
             if g_testtarget=='prod' and fullnode == 'su':
                 self.logger.info(f'We are not testing su in prod right now')
-                return
+                continue
             with self.subTest(mynode=fullnode):
                 self.logger.info(f'TestID: Testing node {fullnode}')
 
@@ -384,7 +384,7 @@ def test_nonmfa_webdav_shared_folders(self):
         for fullnode in g_drv.fullnodes:
             if g_testtarget=='prod' and fullnode == 'su':
                 self.logger.info(f'We are not testing su in prod right now')
-                return
+                continue
             with self.subTest(mynode=fullnode):
                 self.logger.info(f'TestID: Testing node {fullnode}')
 
@@ -469,7 +469,7 @@ def test_mfazones_no_mfauser(self):
         for fullnode in g_drv.fullnodes:
             if g_testtarget=='prod' and fullnode == 'su':
                 self.logger.info(f'We are not testing su in prod right now')
-                return
+                continue
             with self.subTest(mynode=fullnode):
                 self.logger.info(f'TestID: Testing node {fullnode}')
                 nodelogin(fullnode) #No-MFA user
@@ -542,7 +542,7 @@ def test_mfazones_mfauser(self):
         for fullnode in g_drv.fullnodes:
             if g_testtarget=='prod' and fullnode == 'su':
                 self.logger.info(f'We are not testing su in prod right now')
-                return
+                continue
             with self.subTest(mynode=fullnode):
                 self.logger.info(f'TestID: Testing node {fullnode}')
                 actions = ActionChains(g_driver)