From f4552e7a7d77ec272d641c2ee45635253c98bd78 Mon Sep 17 00:00:00 2001
From: Karel van Klink <karel.vanklink@geant.org>
Date: Tue, 11 Jul 2023 17:18:14 +0200
Subject: [PATCH] reformat strings in workflow steps

---
 gso/workflows/device/create_device.py       | 14 ++++----------
 gso/workflows/iptrunk/modify_generic.py     |  8 ++++----
 gso/workflows/iptrunk/modify_isis_metric.py | 18 ++++--------------
 gso/workflows/iptrunk/terminate_iptrunk.py  |  4 ++--
 4 files changed, 14 insertions(+), 30 deletions(-)

diff --git a/gso/workflows/device/create_device.py b/gso/workflows/device/create_device.py
index e0a865f7..c669e6d0 100644
--- a/gso/workflows/device/create_device.py
+++ b/gso/workflows/device/create_device.py
@@ -128,11 +128,8 @@ def provision_device_dry(subscription: DeviceProvisioning, process_id: UUIDstr)
     return {
         "subscription": subscription,
         "label_text": (
-            "Dry run for the deployment of base config on a"
-            f"new {subscription.device_type}. Deployment is "
-            "done  by the provisioning proxy, please "
-            "wait for the results to come back before "
-            "continuing."
+            f"Dry run for the deployment of base config on a new {subscription.device_type}. Deployment is done by the"
+            f" provisioning proxy, please wait for the results to come back before continuing."
         ),
     }
 
@@ -144,11 +141,8 @@ def provision_device_real(subscription: DeviceProvisioning, process_id: UUIDstr)
     return {
         "subscription": subscription,
         "label_text": (
-            "Deployment of base config for a new "
-            f"{subscription.device_type}. Deployment is being "
-            "taken care of by the provisioning proxy, please "
-            "wait for the results to come back before "
-            "continuing."
+            f"Deployment of base config for a new {subscription.device_type}. Deployment is being taken care of by the"
+            f" provisioning proxy, please wait for the results to come back before continuing."
         ),
     }
 
diff --git a/gso/workflows/iptrunk/modify_generic.py b/gso/workflows/iptrunk/modify_generic.py
index 68294b2e..cd86f6ef 100644
--- a/gso/workflows/iptrunk/modify_generic.py
+++ b/gso/workflows/iptrunk/modify_generic.py
@@ -108,7 +108,7 @@ def provision_ip_trunk_iface_dry(subscription: Iptrunk, process_id: UUIDstr) ->
 
     return {
         "subscription": subscription,
-        "label_text": "Provision of the Trunk interface [DRY]" "Please refresh to get the results of the playbook",
+        "label_text": "[DRY RUN] Provisioning trunk interface, please refresh to get the results of the playbook.",
     }
 
 
@@ -118,7 +118,7 @@ def provision_ip_trunk_iface_real(subscription: Iptrunk, process_id: UUIDstr) ->
 
     return {
         "subscription": subscription,
-        "label_text": "Provision of the Trunk interface [REAL]" "Please refresh to get the results of the playbook",
+        "label_text": "Provisioning trunk interface, please refresh to get the results of the playbook.",
     }
 
 
@@ -128,7 +128,7 @@ def provision_ip_trunk_lldp_iface_dry(subscription: Iptrunk, process_id: UUIDstr
 
     return {
         "subscription": subscription,
-        "label_text": "Provision of the LLDP interface [DRY]" "Please refresh to get the results of the playbook",
+        "label_text": "[DRY RUN] Provisioning LLDP interface, please refresh to get the results of the playbook.",
     }
 
 
@@ -138,7 +138,7 @@ def provision_ip_trunk_lldp_iface_real(subscription: Iptrunk, process_id: UUIDst
 
     return {
         "subscription": subscription,
-        "label_text": "Provision of the LLDP interface [REAL]" "Please refresh to get the results of the playbook",
+        "label_text": "Provisioning LLDP interface, please refresh to get the results of the playbook.",
     }
 
 
diff --git a/gso/workflows/iptrunk/modify_isis_metric.py b/gso/workflows/iptrunk/modify_isis_metric.py
index d7affd6a..6edaf8a4 100644
--- a/gso/workflows/iptrunk/modify_isis_metric.py
+++ b/gso/workflows/iptrunk/modify_isis_metric.py
@@ -34,13 +34,8 @@ def provision_ip_trunk_isis_iface_dry(subscription: Iptrunk, process_id: UUIDstr
 
     return {
         "subscription": subscription,
-        "label_text": (
-            "This is a dry run for the deployment of a new IP "
-            "trunk ISIS interface. "
-            "Deployment is being taken care of by the "
-            "provisioning proxy, please wait for the results to "
-            "come back before continuing."
-        ),
+        "label_text": "This is a dry run for the deployment of a new IP trunk ISIS interface. Deployment is being taken"
+        " care of by the provisioning proxy, please wait for the results to come back before continuing.",
     }
 
 
@@ -50,13 +45,8 @@ def provision_ip_trunk_isis_iface_real(subscription: Iptrunk, process_id: UUIDst
 
     return {
         "subscription": subscription,
-        "label_text": (
-            "This is a live deployment of a new IP trunk "
-            "ISIS interface. "
-            "Deployment is being taken care of by the "
-            "provisioning proxy, please wait for the results to "
-            "come back before continuing."
-        ),
+        "label_text": "This is a live deployment of a new IP trunk ISIS interface. Deployment is being taken care of by"
+        " the provisioning proxy, please wait for the results to come back before continuing.",
     }
 
 
diff --git a/gso/workflows/iptrunk/terminate_iptrunk.py b/gso/workflows/iptrunk/terminate_iptrunk.py
index 474ae707..e67f638c 100644
--- a/gso/workflows/iptrunk/terminate_iptrunk.py
+++ b/gso/workflows/iptrunk/terminate_iptrunk.py
@@ -49,7 +49,7 @@ def deprovision_ip_trunk_dry(subscription: Iptrunk, process_id: UUIDstr) -> Stat
 
     return {
         "subscription": subscription,
-        "label_text": "This is a dry run for the termination of an IP " "trunk. " "Press refresh to get the results",
+        "label_text": "[DRY RUN] Terminating IP trunk, please refresh to get the results of the playbook.",
     }
 
 
@@ -59,7 +59,7 @@ def deprovision_ip_trunk_real(subscription: Iptrunk, process_id: UUIDstr) -> Sta
 
     return {
         "subscription": subscription,
-        "label_text": "This is a termination of an IP trunk. " "Press refresh to get the results",
+        "label_text": "Terminating IP trunk, please refresh to get the results of the playbook.",
     }
 
 
-- 
GitLab