updating func names

Author: pmajali

linode_api4/groups/monitor_service.py

@@ -15,7 +15,7 @@ class MonitorGroup(Group):
     This group contains all features beneath the `/monitor` group in the API v4.
     """
 
-    def list_monitor_dashboards(self, *filters) -> list[MonitorDashboard]:
+    def dashboards(self, *filters) -> list[MonitorDashboard]:
         """
         Returns a list of dashboards.
 
@@ -36,7 +36,7 @@ def list_monitor_dashboards(self, *filters) -> list[MonitorDashboard]:
 
         return self.client._get_and_filter(MonitorDashboard, *filters)
 
-    def list_dashboards_by_service(
+    def dashboards_by_service(
         self, service_type: str, *filters
     ) -> list[MonitorDashboard]:
         """
@@ -64,7 +64,7 @@ def list_dashboards_by_service(
             endpoint=f"/monitor/services/{service_type}/dashboards",
         )
 
-    def list_supported_services(self, *filters) -> list[MonitorService]:
+    def services(self, *filters) -> list[MonitorService]:
         """
         Returns a list of services supported by ACLP.
 
@@ -84,7 +84,7 @@ def list_supported_services(self, *filters) -> list[MonitorService]:
 
         return self.client._get_and_filter(MonitorService, *filters)
 
-    def list_service_by_type(
+    def service_by_type(
         self, service_type: str, *filters
     ) -> list[MonitorService]:
         """
@@ -111,7 +111,7 @@ def list_service_by_type(
             endpoint=f"/monitor/services/{service_type}",
         )
 
-    def list_metric_definitions(
+    def metric_definitions(
         self, service_type: str, *filters
     ) -> list[MonitorMetricsDefinition]:
         """

test/integration/models/monitor/test_monitor.py

@@ -18,7 +18,7 @@
 # List all dashboards
 def test_get_all_dashboards(test_linode_client):
     client = test_linode_client
-    dashboards = client.monitor_service.list_monitor_dashboards()
+    dashboards = client.monitor_service.dashboards()
     assert isinstance(dashboards[0], MonitorDashboard)
 
     dashboard_get = dashboards[0]
@@ -30,7 +30,7 @@ def test_get_all_dashboards(test_linode_client):
     assert dashboard_by_id.id == 1
 
     # #Fetch Dashboard by service_type
-    dashboards_by_svc = client.monitor_service.list_dashboards_by_service(
+    dashboards_by_svc = client.monitor_service.dashboards_by_service(
         service_type=get_service_type
     )
     assert isinstance(dashboards_by_svc[0], MonitorDashboard)
@@ -40,20 +40,20 @@ def test_get_all_dashboards(test_linode_client):
 # List supported services
 def test_get_supported_services(test_linode_client):
     client = test_linode_client
-    supported_services = client.monitor_service.list_supported_services()
+    supported_services = client.monitor_service.services()
     assert isinstance(supported_services[0], MonitorService)
 
     get_supported_service = supported_services[0].service_type
 
     # Get details for a particular service
-    service_details = client.monitor_service.list_service_by_type(
+    service_details = client.monitor_service.service_by_type(
         service_type=get_supported_service
     )
     assert isinstance(service_details[0], MonitorService)
     assert service_details[0].service_type == get_supported_service
 
     # Get Metric definition details for that particular service
-    metric_definitions = client.monitor_service.list_metric_definitions(
+    metric_definitions = client.monitor_service.metric_definitions(
         service_type=get_supported_service
     )
     assert isinstance(metric_definitions[0], MonitorMetricsDefinition)

test/unit/objects/monitor_service_test.py

@@ -13,7 +13,7 @@ def test_supported_services(self):
         """
         Test the services supported by monitor
         """
-        service = self.client.monitor_service.list_supported_services()
+        service = self.client.monitor_service.services()
         self.assertEqual(len(service), 1)
         self.assertEqual(service[0].label, "Databases")
         self.assertEqual(service[0].service_type, "dbaas")
@@ -43,7 +43,7 @@ def test_dashboard_by_ID(self):
         self.assertEqual(dashboard.widgets[0].y_label, "cpu_usage")
 
     def test_dashboard_by_service_type(self):
-        dashboards = self.client.monitor_service.list_dashboards_by_service(
+        dashboards = self.client.monitor_service.dashboards_by_service(
             service_type="dbaas"
         )
         self.assertEqual(dashboards[0].type, "standard")
@@ -66,7 +66,7 @@ def test_dashboard_by_service_type(self):
         self.assertEqual(dashboards[0].widgets[0].y_label, "cpu_usage")
 
     def test_get_all_dashboards(self):
-        dashboards = self.client.monitor_service.list_monitor_dashboards()
+        dashboards = self.client.monitor_service.dashboards()
         self.assertEqual(dashboards[0].type, "standard")
         self.assertEqual(
             dashboards[0].created, datetime.datetime(2024, 10, 10, 5, 1, 58)
@@ -87,15 +87,15 @@ def test_get_all_dashboards(self):
         self.assertEqual(dashboards[0].widgets[0].y_label, "cpu_usage")
 
     def test_specific_service_details(self):
-        data = self.client.monitor_service.list_service_by_type(
+        data = self.client.monitor_service.service_by_type(
             service_type="dbaas"
         )
         self.assertEqual(data[0].label, "Databases")
         self.assertEqual(data[0].service_type, "dbaas")
 
     def test_metric_definitions(self):
 
-        metrics = self.client.monitor_service.list_metric_definitions(
+        metrics = self.client.monitor_service.metric_definitions(
             service_type="dbaas"
         )
         self.assertEqual(