fixing a control flow bug

Author: cderici

tests/integration/test_model.py

@@ -341,13 +341,17 @@ def wait_for_network(container, timeout=30):
                 time.sleep(attempt * 5)
             else:
                 break
+        else:
+            # this else will only run when for finishes without a
+            # break (meaning it kept erroring)
+            raise Exception('Unable to add_machine')
 
-            assert len(model.machines) == 1
+        assert len(model.machines) == 1
 
-            res = await machine1.destroy(force=True)
+        res = await machine1.destroy(force=True)
 
-            assert res is None
-            assert len(model.machines) == 0
+        assert res is None
+        assert len(model.machines) == 0
 
         container.stop(wait=True)
         container.delete(wait=True)
@@ -451,9 +455,15 @@ def wait_for_network(container, timeout=30):
                     private_key_path,
                 ))
             except (paramiko.ssh_exception.NoValidConnectionsError, paramiko.ssh_exception.AuthenticationException):
+                # if we get an exception, try again
                 time.sleep(attempt * 5)
             else:
+                # try part finished without exception
                 break
+        else:
+            # this else will only run when for finishes without a
+            # break (meaning it kept erroring)
+            raise Exception('Unable to add_machine')
 
         assert len(model.machines) == 1