Fix test

cmapi/cmapi_server/handlers/cluster.py

@@ -224,14 +224,16 @@ class ClusterHandler():
                 node, input_config_filename=config,
                 output_config_filename=config
             )
-            with NodeConfig().modify_config(config) as root:
-                update_dbroots_of_readonly_nodes(root)
+
         except Exception as err:
             raise CMAPIBasicError('Error while removing node.') from err
 
         response['node_id'] = node
         active_nodes = get_active_nodes(config)
         if len(active_nodes) > 0:
+            with NodeConfig().modify_config(config) as root:
+                update_dbroots_of_readonly_nodes(root)
+
             update_revision_and_manager(
                 input_config_filename=config, output_config_filename=config
             )

cmapi/cmapi_server/helpers.py

@@ -135,6 +135,10 @@ def start_transaction(
         # this copy will be updated if an optional node can't be reached
         real_active_nodes = set(active_nodes)
         logging.trace(f'Active nodes on start transaction {active_nodes}')
+
+        if not len(active_nodes):
+            logging.warning('No active nodes found, transaction start will not have any effect')
+
         for node in active_nodes:
             url = f'https://{node}:8640/cmapi/{version}/node/begin'
             node_success = False

cmapi/cmapi_server/test/test_node_manip.py

@@ -16,7 +16,6 @@ logging.basicConfig(level='DEBUG')
 SINGLE_NODE_XML = "./cmapi_server/SingleNode.xml"
 
 
-@unittest.skip('Temp skip for testing reason')
 class NodeManipTester(BaseNodeManipTestCase):
 
     def test_add_remove_node(self):