Skip to content

Commit c02c732

Browse files
committed
Improve code readability
1 parent c40babe commit c02c732

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

scaleway/tests/test_instance.py

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
volume_size = 10
2323
commercial_type = "DEV1-S"
2424
zone = "fr-par-1"
25-
timeout_attach = 10
2625

2726

2827
class TestE2EServerCreation(unittest.TestCase):
@@ -43,19 +42,24 @@ def tearDown(self) -> None:
4342

4443
self.blockAPI.delete_volume(volume_id=volume.id)
4544
logger.info("✅ Volume {volume.id} has been deleted")
45+
4646
if self._server:
4747
self.instanceAPI.delete_server(zone=self.zone, server_id=self._server.id)
4848
logger.info(f"🗑️ Deleted server: {self._server.id}")
4949

5050
def wait_test_instance_server(self, server_id):
5151
interval = interval
52+
5253
for i in range(1, max_retry):
5354
interval *= i
5455
s = self.instanceAPI.get_server(zone=self.zone, server_id=server_id)
56+
5557
if s.state == "running":
5658
logger.info(f"✅ Server {server_id} is running.")
5759
break
60+
5861
time.sleep(interval)
62+
5963
self.fail("Server did not reach 'running' state in time.")
6064

6165
def create_test_instance_server(self) -> Server:
@@ -64,6 +68,7 @@ def create_test_instance_server(self) -> Server:
6468
volume_type="sbs_volume", name="my-volume", size=volume_size
6569
)
6670
}
71+
6772
server = self.instanceAPI._create_server(
6873
commercial_type=commercial_type,
6974
zone=self.zone,
@@ -72,18 +77,24 @@ def create_test_instance_server(self) -> Server:
7277
volumes=volume,
7378
)
7479
logger.info(f"✅ Created server: {server.id}")
80+
7581
self._server = server.server
82+
7683
self.wait_test_instance_server(server_id=server.server.id)
84+
7785
return server.server
7886

7987
def create_test_from_empty_volume(self, number) -> List[Volume]:
8088
volumes: List[Volume] = {}
89+
8190
for i in range(number):
8291
volume = self.blockAPI.create_volume(
8392
from_empty=CreateVolumeRequestFromEmpty(size=10),
8493
)
8594
logger.info("✅ Created server: {volume.id}")
95+
8696
self.blockAPI.wait_for_volume(volume_id=volume.id, zone=self.zone)
97+
8798
self._volumes.append(volume) # Ensure cleanup in tearDown
8899
volumes.append(volume)
89100

0 commit comments

Comments
 (0)