Deprecate get_node_info() in favor of get_node_details()

pydactyl/api/nodes.py

@@ -19,7 +19,7 @@ def list_nodes(self, include=None):
         response = self._api_request(endpoint=endpoint, params=params)
         return PaginatedResponse(self, endpoint, response)
 
-    def get_node_info(self, node_id):
+    def get_node_details(self, node_id):
         """Get detailed info for the specified node.
 
         Args:
@@ -29,6 +29,16 @@ def get_node_info(self, node_id):
             endpoint='application/nodes/{}'.format(node_id))
         return response
 
+    def get_node_info(self, node_id):
+        """Get detailed info for the specified node.
+
+        DEPRECATED: Use get_node_details
+
+        Args:
+            node_id(int): Pterodactyl Node ID.
+        """
+        return self.get_node_details(node_id)
+
     def create_node(self, name, description, location_id, fqdn, memory, disk,
                     memory_overallocate=0,
                     disk_overallocate=0, use_ssl=True, behind_proxy=False,
@@ -165,3 +175,4 @@ def delete_allocation(self, node_id, allocation_id):
         response = self._api_request(
             endpoint=endpoint, mode='DELETE', json=False)
         return response
+

tests/application/nodes_test.py

@@ -27,11 +27,11 @@ def test_list_nodes_with_includes(self, mock_api):
         mock_api.assert_called_with(**expected)
 
     @mock.patch('pydactyl.api.base.PterodactylAPI._api_request')
-    def test_get_node_info(self, mock_api):
+    def test_get_node_details(self, mock_api):
         expected = {
             'endpoint': 'application/nodes/11',
         }
-        self.client.nodes.get_node_info(11)
+        self.client.nodes.get_node_details(11)
         mock_api.assert_called_with(**expected)
 
     @mock.patch('pydactyl.api.base.PterodactylAPI._api_request')