updating with make format changes

Author: pmajali

linode_api4/groups/monitor.py

@@ -24,10 +24,10 @@ class MonitorGroup(Group):
 
     def dashboards(self, *filters):
         """
-        Returns a list of dashboards on your account. 
+        Returns a list of dashboards on your account.
 
         .. note:: This endpoint is in beta. This will only function if base_url is set to `https://api.linode.com/v4beta`.
-        
+
         API Documentation: https://techdocs.akamai.com/linode-api/reference/get-dashboards-all
 
         :param filters: Any number of filters to apply to this query.
@@ -39,14 +39,12 @@ def dashboards(self, *filters):
         """
         return self.client._get_and_filter(Dashboard, *filters)
 
-
-    
     def dashboards_by_service(self, service_type: str, *filters):
         """
-        Returns a dashboards on your account based on the service passed. 
+        Returns a dashboards on your account based on the service passed.
 
         .. note:: This endpoint is in beta. This will only function if base_url is set to `https://api.linode.com/v4beta`.
-        
+
         API Documentation: https://techdocs.akamai.com/linode-api/reference/get-dashboards
 
         :param filters: Any number of filters to apply to this query.
@@ -56,20 +54,19 @@ def dashboards_by_service(self, service_type: str, *filters):
         :returns: A Dashboards filtered by Service Type.
         :rtype: PaginatedList of the Dashboards
         """
-        
+
         return self.client._get_and_filter(
             DashboardByService,
             *filters,
             endpoint=f"/monitor/services/{service_type}/dashboards",
         )
 
-
     def supported_services(self, *filters):
         """
-        Returns a list of services supported by ACLP. 
+        Returns a list of services supported by ACLP.
 
         .. note:: This endpoint is in beta. This will only function if base_url is set to `https://api.linode.com/v4beta`.
-        
+
         API Documentation: https://techdocs.akamai.com/linode-api/reference/get-monitor-services
 
         :param filters: Any number of filters to apply to this query.
@@ -82,54 +79,54 @@ def supported_services(self, *filters):
 
         return self.client._get_and_filter(MonitorServiceSupported, *filters)
 
-    def details_by_service(self, service_type: str,*filters):
+    def details_by_service(self, service_type: str, *filters):
         """
-        Returns a details about a particular service. 
+        Returns a details about a particular service.
 
         .. note:: This endpoint is in beta. This will only function if base_url is set to `https://api.linode.com/v4beta`.
-        
+
         API Documentation: https://techdocs.akamai.com/linode-api/reference/get-monitor-services-for-service-type
 
         :param filters: Any number of filters to apply to this query.
                         See :doc:`Filtering Collections</linode_api4/objects/filtering>`
                         for more details on filtering.
 
         :returns: Details about a  Supported Services
-        :rtype: PaginatedList of the Service 
+        :rtype: PaginatedList of the Service
         """
         return self.client._get_and_filter(
             ServiceDetails,
             *filters,
             endpoint=f"/monitor/services/{service_type}",
         )
-    
-    def metric_definitions(self, service_type: str,*filters):
+
+    def metric_definitions(self, service_type: str, *filters):
         """
-        Returns metrics for a specific service type. 
+        Returns metrics for a specific service type.
 
         .. note:: This endpoint is in beta. This will only function if base_url is set to `https://api.linode.com/v4beta`.
-        
+
         API Documentation: https://techdocs.akamai.com/linode-api/reference/get-monitor-information
 
         :param filters: Any number of filters to apply to this query.
                         See :doc:`Filtering Collections</linode_api4/objects/filtering>`
                         for more details on filtering.
 
         :returns: Returns a  List of metrics for a service
-        :rtype: PaginatedList of metrics 
+        :rtype: PaginatedList of metrics
         """
         return self.client._get_and_filter(
             MetricDefinition,
             *filters,
             endpoint=f"/monitor/services/{service_type}/metric-definitions",
         )
-    
+
     def create_token(self, service_type: str, entity_ids: list, *filters):
         """
-        Returns a JWE Token for a specific service type. 
+        Returns a JWE Token for a specific service type.
 
         .. note:: This endpoint is in beta. This will only function if base_url is set to `https://api.linode.com/v4beta`.
-        
+
         API Documentation: https://techdocs.akamai.com/linode-api/reference/post-get-token
         :param filters: Any number of filters to apply to this query.
                         See :doc:`Filtering Collections</linode_api4/objects/filtering>`
@@ -141,20 +138,12 @@ def create_token(self, service_type: str, entity_ids: list, *filters):
 
         params = {"entity_ids": entity_ids}
 
-        result = self.client.post(f"/monitor/services/{service_type}/token", data=params)
+        result = self.client.post(
+            f"/monitor/services/{service_type}/token", data=params
+        )
 
         if "token" not in result:
             raise UnexpectedResponseError(
                 "Unexpected response when creating token!", json=result
             )
         return CreateToken(self.client, result["token"], result)
-    
-   
-      
-    
-    
-
-
-
-    
-   

linode_api4/objects/monitor.py

@@ -15,11 +15,9 @@ class Dashboard(Base):
         "type": Property(),
         "widgets": Property(mutable=True),
         "updated": Property(is_datetime=True),
-     
     }
 
 
-
 class DashboardByService(Base):
     """
     Get a dashboard: https://techdocs.akamai.com/linode-api/reference/get-dashboards
@@ -33,7 +31,6 @@ class DashboardByService(Base):
         "type": Property(),
         "widgets": Property(mutable=True),
         "updated": Property(is_datetime=True),
-     
     }
 
 
@@ -43,47 +40,43 @@ class MonitorServiceSupported(Base):
     id_attribute = "service_type"
     properties = {
         "service_type": Property(),
-        "label": Property(mutable=True), 
-              
+        "label": Property(mutable=True),
     }
 
+
 class ServiceDetails(Base):
     """
     API Documentation: https://techdocs.akamai.com/linode-api/reference/get-monitor-services-for-service-type
     """
+
     id_attribute = "service_type"
     properties = {
-        "label": Property(), 
-        "service_type": Property(),      
+        "label": Property(),
+        "service_type": Property(),
     }
 
 
-
 class MetricDefinition(Base):
     """
     API Documentation: https://techdocs.akamai.com/linode-api/reference/get-monitor-information
     """
 
     id_attribute = "metric"
     properties = {
-        "available_aggregate_functions": Property(), 
-        "dimensions": Property(mutable=True),   
-        "label": Property(), 
+        "available_aggregate_functions": Property(),
+        "dimensions": Property(mutable=True),
+        "label": Property(),
         "is_alertable": Property(),
         "metric": Property(),
         "metric_type": Property(),
         "scrape_interval": Property(),
         "unit": Property(),
     }
 
+
 class CreateToken(Base):
     """
     API Documentation: https://techdocs.akamai.com/linode-api/reference/post-get-token
     """
 
-    properties = {
-        "token": Property(mutable=True)
-    }
-
-
-
+    properties = {"token": Property(mutable=True)}

test/integration/models/monitor/test_monitor.py

@@ -21,37 +21,45 @@
 def test_get_all_dashboards(test_linode_client):
     client = test_linode_client
     dashboards = client.monitor.dashboards()
-    assert isinstance(dashboards[0],Dashboard)
+    assert isinstance(dashboards[0], Dashboard)
 
     dashboard_get = dashboards[0]
     get_service_type = dashboard_get.service_type
 
-    #Fetch Dashboard by ID
+    # Fetch Dashboard by ID
     dashboard_by_id = client.load(Dashboard, 1)
     assert isinstance(dashboard_by_id, Dashboard)
     assert dashboard_by_id.id == 1
 
     # #Fetch Dashboard by service_type
-    dashboards_by_svc = client.monitor.dashboards_by_service(service_type=get_service_type)
+    dashboards_by_svc = client.monitor.dashboards_by_service(
+        service_type=get_service_type
+    )
     assert isinstance(dashboards_by_svc[0], DashboardByService)
     assert dashboards_by_svc[0].service_type == get_service_type
 
-# List supported services   
+
+# List supported services
 def test_get_supported_services(test_linode_client):
     client = test_linode_client
     supported_services = client.monitor.supported_services()
     assert isinstance(supported_services[0], MonitorServiceSupported)
 
     get_supported_service = supported_services[0].service_type
 
-    #Get details for a particular service
-    service_details = client.monitor.details_by_service(service_type=get_supported_service)
-    assert isinstance(service_details[0],ServiceDetails)
+    # Get details for a particular service
+    service_details = client.monitor.details_by_service(
+        service_type=get_supported_service
+    )
+    assert isinstance(service_details[0], ServiceDetails)
     assert service_details[0].service_type == get_supported_service
 
-    #Get Metric definition details for that particular service
-    metric_definitions = client.monitor.metric_definitions(service_type=get_supported_service)
-    assert isinstance(metric_definitions[0],MetricDefinition)
+    # Get Metric definition details for that particular service
+    metric_definitions = client.monitor.metric_definitions(
+        service_type=get_supported_service
+    )
+    assert isinstance(metric_definitions[0], MetricDefinition)
+
 
 # Test Helpers
 def get_db_engine_id(client: LinodeClient, engine: str):
@@ -63,6 +71,7 @@ def get_db_engine_id(client: LinodeClient, engine: str):
 
     return str(engine_id)
 
+
 @pytest.fixture(scope="session")
 def test_create_and_test_db(test_linode_client):
     client = test_linode_client
@@ -71,7 +80,6 @@ def test_create_and_test_db(test_linode_client):
     engine_id = get_db_engine_id(client, "mysql")
     dbtype = "g6-standard-1"
 
-
     db = client.database.mysql_create(
         label=label,
         region=region,
@@ -89,27 +97,17 @@ def get_db_status():
     yield db
     send_request_when_resource_available(300, db.delete)
 
+
 def test_my_db_functionality(test_linode_client, test_create_and_test_db):
     client = test_linode_client
     assert test_create_and_test_db.status == "active"
-    
-    entity_id = test_create_and_test_db.id
 
+    entity_id = test_create_and_test_db.id
 
     # create token for the particular service
-    token = client.monitor.create_token(service_type="dbaas", entity_ids=[entity_id])
+    token = client.monitor.create_token(
+        service_type="dbaas", entity_ids=[entity_id]
+    )
     assert isinstance(token, CreateToken)
     assert len(token.token) > 0, "Token should not be empty"
     assert hasattr(token, "token"), "Response object has no 'token' attribute"
-
-
-    
-  
-   
-
-
-
-
-
-
-