Address SonarQube code smells

gso/services/netbox_client.py

@@ -160,7 +160,7 @@ class NetboxClient:
     def create_device(self, device_name: str, site_tier: str) -> Devices:
         """Create a new device in Netbox."""
         # Get device type id
-        tier_info = TierInfo().get_module_by_name(f"Tier{site_tier}")
+        tier_info = TierInfo().get_module_by_name(f"tier{site_tier}")
         device_type = self.netbox.dcim.device_types.get(model=tier_info.device_type)
 
         # Get device role id

gso/utils/device_info.py

@@ -18,14 +18,14 @@ class TierInfo:
 
     def __init__(self) -> None:
         """Initialise the different tiers of sites that exist."""
-        self.Tier1 = ModuleInfo(
+        self.tier1 = ModuleInfo(
             device_type="7750 SR-7s",
             module_bays_slots=[1, 2],
             module_type="XMA2-s-36p-400g",
             breakout_interfaces_per_slot=[36, 35, 34, 33],
             total_10g_interfaces=80,
         )
-        self.Tier2 = ModuleInfo(
+        self.tier2 = ModuleInfo(
             device_type="7750-SR7s",
             module_bays_slots=[1, 2],
             module_type="XMA2-s-36p-400g",

gso/workflows/iptrunk/migrate_iptrunk.py

@@ -40,6 +40,8 @@ from gso.utils.helpers import (
 
 logger = getLogger(__name__)
 
+PLAYBOOK_VERB_NOT_YET_PROPERLY_SET = "Playbook verb is not yet properly set."
+
 
 def initial_input_form_generator(subscription_id: UUIDstr) -> FormGenerator:
     """Gather input from the operator on the new router that the IP trunk should connect to."""
@@ -250,7 +252,7 @@ def deploy_new_config_dry(
         "trunk_interface",
     )
 
-    logger.warning("Playbook verb is not yet properly set.")
+    logger.warning(PLAYBOOK_VERB_NOT_YET_PROPERLY_SET)
 
     return {
         "subscription": subscription,
@@ -286,7 +288,7 @@ def deploy_new_config_real(
         dry_run=False,
     )
 
-    logger.warning("Playbook verb is not yet properly set.")
+    logger.warning(PLAYBOOK_VERB_NOT_YET_PROPERLY_SET)
 
     return {
         "subscription": subscription,
@@ -337,7 +339,7 @@ def deploy_new_isis(
         dry_run=False,
     )
 
-    logger.warning("Playbook verb is not yet properly set.")
+    logger.warning(PLAYBOOK_VERB_NOT_YET_PROPERLY_SET)
 
     return {
         "subscription": subscription,
@@ -409,7 +411,7 @@ def delete_old_config_dry(
         "delete",
     )
 
-    logger.warning("Playbook verb is not yet properly set.")
+    logger.warning(PLAYBOOK_VERB_NOT_YET_PROPERLY_SET)
 
     return {"subscription": subscription}
 
@@ -443,7 +445,7 @@ def delete_old_config_real(
         dry_run=False,
     )
 
-    logger.warning("Playbook verb is not yet properly set.")
+    logger.warning(PLAYBOOK_VERB_NOT_YET_PROPERLY_SET)
 
     return {"subscription": subscription}