tests: remove logs

Author: Laure-di

scaleway/tests/test_instance.py

@@ -1,5 +1,3 @@
-import logging
-import sys
 from typing import List
 import unittest
 import uuid
@@ -11,10 +9,6 @@
 from scaleway.instance.v1.types import Server, VolumeServerTemplate
 from scaleway.block.v1alpha1.types import Volume, CreateVolumeRequestFromEmpty
 
-logger = logging.getLogger()
-logger.level = logging.DEBUG
-stream_handler = logging.StreamHandler(sys.stdout)
-logger.addHandler(stream_handler)
 
 server_name = f"test-sdk-python-{uuid.uuid4().hex[:6]}"
 max_retry = 10
@@ -39,14 +33,11 @@ def tearDown(self) -> None:
             self.instanceAPI.detach_server_volume(
                 server_id=self._server.id, volume_id=volume.id
             )
-            logger.info(f"✅ Volume {volume.id} has been detach")
 
             self.blockAPI.delete_volume(volume_id=volume.id)
-            logger.info(f"✅ Volume {volume.id} has been deleted")
 
         if self._server:
             self.instanceAPI.delete_server(zone=self.zone, server_id=self._server.id)
-            logger.info(f"🗑️ Deleted server: {self._server.id}")
 
     def wait_test_instance_server(self, server_id):
         interval = interval
@@ -56,7 +47,6 @@ def wait_test_instance_server(self, server_id):
             s = self.instanceAPI.get_server(zone=self.zone, server_id=server_id)
 
             if s.state == "running":
-                logger.info(f"✅ Server {server_id} is running.")
                 break
 
             time.sleep(interval)
@@ -77,7 +67,6 @@ def create_test_instance_server(self) -> Server:
             dynamic_ip_required=True,
             volumes=volume,
         )
-        logger.info(f"✅ Created server: {server.id}")
 
         self._server = server.server
 
@@ -92,7 +81,6 @@ def create_test_from_empty_volume(self, number) -> List[Volume]:
             volume = self.blockAPI.create_volume(
                 from_empty=CreateVolumeRequestFromEmpty(size=10),
             )
-            logger.info(f"✅ Created server: {volume.id}")
 
             self.blockAPI.wait_for_volume(volume_id=volume.id, zone=self.zone)
 
@@ -111,22 +99,16 @@ def test_attach_aditionnal_volume(self):
 
         self.assertIsNotNone(additional_volume.id)
         self.assertEqual(additional_volume.size, 10)
-        logger.info(f"✅ Volume created with ID: {additional_volume.id}")
 
         self.instanceAPI.attach_server_volume(
             server_id=server.id, volume_id=additional_volume.id
         )
 
         self.blockAPI.wait_for_volume(volume_id=additional_volume.id, zone=self.zone)
 
-        logger.info(f"🔗 Attached volume {additional_volume.id} to server {server.id}")
-
         updated_server = self.instanceAPI.get_server(
             zone=self.zone, server_id=server.id
         )
         attached_volumes = updated_server.volumes or {}
         attached_volume_ids = [v.volume.id for v in attached_volumes.values()]
         self.assertIn(additional_volume.id, attached_volume_ids)
-        logger.info(
-            f"✅ Volume {additional_volume.id} is attached to server {server.id}"
-        )

scaleway/tests/test_vpc.py

@@ -1,16 +1,9 @@
-import logging
-import sys
 import unittest
 from scaleway.vpc.v2 import VpcV2API
 from scaleway_core.api import ScalewayException
 from scaleway_core.client import Client
 from scaleway_core.utils import random_name
 
-logger = logging.getLogger()
-logger.level = logging.DEBUG
-stream_handler = logging.StreamHandler(sys.stdout)
-logger.addHandler(stream_handler)
-
 region = "fr-par"
 tags = ["sdk-python", "regression-test"]
 
@@ -31,17 +24,14 @@ def setUpClass(self):
             project_id=self.project_id,
             name=random_name("vpc-test-sdk-python"),
         )
-        logger.info(f"✅ VPC {self._vpc.id} has been created")
 
     @classmethod
     def tearDownClass(self):
         for pn in self._pns_to_cleanup:
             self.vpcAPI.delete_private_network(private_network_id=pn.id)
-            logger.info(f"🧹 Deleted Private Network {pn.id}")
 
         if self._vpc is not None:
             self.vpcAPI.delete_vpc(vpc_id=self._vpc.id, region=self.region)
-            logger.info(f"🧹 Deleted VPC {self._vpc.id}")
 
     def test_delete_vpc(self):
         vpc = self.vpcAPI.create_vpc(
@@ -50,20 +40,17 @@ def test_delete_vpc(self):
             project_id=self.project_id,
             name=random_name("vpc-test-sdk-python"),
         )
-        logger.info(f"✅ VPC {vpc.id} has been created")
+
         self.assertIsNotNone(vpc.id)
         self.assertEqual(vpc.region, self.region)
 
         self.vpcAPI.delete_vpc(vpc_id=vpc.id)
-        logger.info(f"🗑️ VPC {vpc.id} deletion requested")
 
         with self.assertRaises(ScalewayException):
             self.vpcAPI.get_vpc(vpc_id=vpc.id)
-        logger.info(f"✅ VPC {vpc.id} has been deleted successfully")
 
     def test_list_vpcs(self):
         vpcs = self.vpcAPI.list_vp_cs(region=self.region).vpcs
-        logger.info(f"🔎 Listed {len(vpcs)} VPC(s) in region: {self.region}")
         self.assertIsInstance(vpcs, list)
 
     def test_create_private_network(self):
@@ -75,29 +62,26 @@ def test_create_private_network(self):
                 name=random_name(f"pn-{i}"),
             )
             self._pns_to_cleanup.append(pn)
-            logger.info(f"✅ PN {i + 1}/5: {pn.id} created in VPC {self._vpc.id}")
+
             self.assertEqual(pn.vpc_id, self._vpc.id)
 
     def test_list_private_network(self):
         networks = self.vpcAPI.list_private_networks(region=self.region)
-        logger.info(
-            f"🔎 Listed {networks.total_count} private network(s) in region: {self.region}"
-        )
         self.assertIsInstance(networks.private_networks, list)
 
     def test_get_vpc(self):
         vpc = self.vpcAPI.get_vpc(vpc_id=self._vpc.id, region=self.region)
-        logger.info(f"📥 Retrieved VPC {vpc.id}")
+        
         self.assertIsNotNone(vpc)
         self.assertEqual(self._vpc.id, vpc.id)
 
     def test_update_vpc(self):
         vpc = self.vpcAPI.update_vpc(vpc_id=self._vpc.id, tags=tags)
-        logger.info(f"🛠️ Updated VPC {vpc.id} with tags: {tags}")
+        
         self.assertEqual(vpc.tags, tags)
         self.assertEqual(self._vpc.id, vpc.id)
 
     def test_list_vpc_all(self):
         vpcs = self.vpcAPI.list_vp_cs_all()
-        logger.info(f"📥 Retrieved total of {len(vpcs)} VPC(s) across all regions")
+
         self.assertIsInstance(vpcs, list)