SIENTIAPDE-1478

Refactor API class to remove unnecessary blank line and improve code readability. Update test cases to rename 'timeout' parameter to 'request_timeout' for consistency.

scouter/activities/api.py

@@ -136,7 +136,6 @@ class API(SientiaMonitoring):
             )
             raise e
 
-
         latest_values['timestamp'] = latest_values['timestamp'].dt.strftime(DATETIME_FORMAT_WITH_TZ)
 
         self.debug(f'Latest values: {latest_values.to_string()}', metadata=metadata)
@@ -144,7 +143,7 @@ class API(SientiaMonitoring):
         # Normalize the package timestamp
         valid_timestamp_values = latest_values['timestamp'].dropna()
         latest_values['timestamp'] = valid_timestamp_values.max()
-        
+
         self.info(f'Gathered {len(latest_values)} tag values', metadata=metadata)
 
         return latest_values.to_dict(orient='records')

tests/activities/test_api.py

@@ -141,7 +141,7 @@ async def test_get_tag_values_success(api_activity):
         start_time='*-1d',
         max_count=10,
         metadata=metadata['metadata'],
-        timeout=30,
+        request_timeout=30,
     )
 
     assert len(result) == 3
@@ -195,7 +195,7 @@ async def test_get_tag_values_with_default_max_count(api_activity):
         start_time='*-1h',
         max_count=1,
         metadata=metadata['metadata'],
-        timeout=15,
+        request_timeout=15,
     )
 
     assert len(result) == 1