updates variable name for new api changes

CiscoZeus · Nov 8, 2016 · 87eadc2 · 87eadc2
1 parent 89dd87a
commit 87eadc2
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/tests/test_zeus_client.py b/tests/test_zeus_client.py
@@ -235,11 +235,11 @@ def test_create_alert(self, mock_requests):
         metric_name = "cpu.value"
         emails = "john1234@gmail.com"
         status = "active"
-        frequency = 60
+        notify_period = 60
 
         self.z.createAlert(alert_name, username, alerts_type,
                            alert_expression, alert_severity, metric_name,
-                           emails, status, frequency)
+                           emails, status, notify_period)
         data = {
             'alert_name': alert_name,
             'username': username,
@@ -250,7 +250,7 @@ def test_create_alert(self, mock_requests):
             'metric_name': metric_name,
             'emails': emails,
             'status': status,
-            'frequency': frequency
+            'notify_period': notify_period
         }
 
         mock_requests.post.assert_called_with(
@@ -272,11 +272,11 @@ def test_modify_alert(self, mock_requests):
         metric_name = "cpu.value"
         emails = "john1234@gmail.com"
         status = "active"
-        frequency = 60
+        notify_period = 60
 
         self.z.modifyAlert(alert_id, alert_name, username, alerts_type,
                            alert_expression, alert_severity, metric_name,
-                           emails, status, frequency)
+                           emails, status, notify_period)
         data = {
             'alert_name': alert_name,
             'username': username,
@@ -287,7 +287,7 @@ def test_modify_alert(self, mock_requests):
             'metric_name': metric_name,
             'emails': emails,
             'status': status,
-            'frequency': frequency
+            'notify_period': notify_period
         }
 
         path = FAKE_SERVER + '/alerts/' + FAKE_TOKEN + '/' + str(alert_id)

diff --git a/zeus/interfaces/alerts.py b/zeus/interfaces/alerts.py
@@ -25,7 +25,7 @@ def __init__(self, user_token, rest_client):
 
     def createAlert(
             self, alert_name, username, alerts_type, alert_expression,
-            alert_severity, metric_name, emails, status, frequency
+            alert_severity, metric_name, emails, status, notify_period
     ):
 
         """Creates a new alert, returns status code of creation
@@ -40,7 +40,7 @@ def createAlert(
         :param metric_name: metric associated with the alert
         :param emails: emails to receive notification when the alert triggers
         :param status: if the alert is active or disable
-        :param frequency: frequency in which the expression is checked
+        :param notify_period: frequency of notifications
 
         :rtype: array
         """
@@ -55,7 +55,7 @@ def createAlert(
             'metric_name': metric_name,
             'emails': emails,
             'status': status,
-            'frequency': frequency
+            'notify_period': notify_period
         }
         path = '/alerts/' + self.token
         header = {'content-type': 'application/json'}
@@ -74,7 +74,7 @@ def getAlerts(self):
     def modifyAlert(
             self, alert_id, alert_name, username, alerts_type,
             alert_expression, alert_severity, metric_name, emails, status,
-            frequency
+            notify_period
     ):
 
         """Modifies an existing alert with new data
@@ -89,7 +89,7 @@ def modifyAlert(
         :param metric_name: metric associated with the alert
         :param emails: emails to receive notification when the alert triggers
         :param status: if the alert is active or disable
-        :param frequency: frequency in which the expression is checked
+        :param notify_period: frequency of notifications
 
         :rtype: array
         """
@@ -104,7 +104,7 @@ def modifyAlert(
             'metric_name': metric_name,
             'emails': emails,
             'status': status,
-            'frequency': frequency
+            'notify_period': notify_period
         }
         path = '/alerts/' + self.token + '/' + str(alert_id)
         header = {'content-type': 'application/json'}