From 6191d0fe9171405f84b3a7a75b797f85156f0aca Mon Sep 17 00:00:00 2001
From: Karel van Klink <karel.vanklink@geant.org>
Date: Tue, 2 Jan 2024 17:26:59 +0100
Subject: [PATCH] fix one unit test, and remove some unused code
---
gso/workflows/iptrunk/migrate_iptrunk.py | 5 -----
test/workflows/iptrunk/test_migrate_iptrunk.py | 2 +-
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/gso/workflows/iptrunk/migrate_iptrunk.py b/gso/workflows/iptrunk/migrate_iptrunk.py
index 2968ff86..1c18cf12 100644
--- a/gso/workflows/iptrunk/migrate_iptrunk.py
+++ b/gso/workflows/iptrunk/migrate_iptrunk.py
@@ -7,7 +7,6 @@ configured to run from A to C. B is then no longer associated with this IP trunk
import copy
import json
import re
-from logging import getLogger
from typing import NoReturn
from uuid import uuid4
@@ -41,10 +40,6 @@ from gso.utils.helpers import (
)
from gso.utils.workflow_steps import set_isis_to_90000
-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."""
diff --git a/test/workflows/iptrunk/test_migrate_iptrunk.py b/test/workflows/iptrunk/test_migrate_iptrunk.py
index 524624ff..9a791b65 100644
--- a/test/workflows/iptrunk/test_migrate_iptrunk.py
+++ b/test/workflows/iptrunk/test_migrate_iptrunk.py
@@ -107,7 +107,7 @@ def interface_lists_are_equal(list1, list2):
indirect=True,
)
@pytest.mark.workflow()
-@patch("gso.services.provisioning_proxy.execute_playbook")
+@patch("gso.services.provisioning_proxy._send_request")
@patch("gso.services.netbox_client.NetboxClient.get_available_interfaces")
@patch("gso.services.netbox_client.NetboxClient.get_available_lags")
@patch("gso.services.netbox_client.NetboxClient.create_interface")
--
GitLab