add feedback summary endpoint

Author: Doug Black

tests/resources/call_feedback_summary.json

@@ -0,0 +1,17 @@
+{
+    "start_date": "2014-01-01",
+    "end_date": "2014-01-31",
+    "account_sid": "AC7b659acc64804bd8bc01a2d9d54248e1",
+    "call_count": 10200,
+    "call_feedback_count": 729,
+    "quality_score_average": 4.5,
+    "quality_score_median": 4,
+    "quality_score_standard_deviation": 1,
+    "issues": [
+        {
+            "description": "imperfect-audio",
+            "count": 45,
+            "percentage_of_total_calls": "0.04%"
+        }
+    ]
+}

tests/test_call_feedback.py

@@ -3,7 +3,7 @@
 from nose.tools import assert_equal
 from mock import Mock, patch, ANY
 from tools import create_mock_json
-from twilio.rest.resources import Call
+from twilio.rest.resources import Call, Calls
 
 AUTH = ('foo', 'bar')
 
@@ -50,3 +50,23 @@ def test_create_call_feedback(self, request):
             data=exp_data, auth=AUTH,
             timeout=ANY,
         )
+
+
+class CallFeedbackSummaryTest(unittest.TestCase):
+
+    @patch('twilio.rest.resources.base.make_twilio_request')
+    def test_get_call_feedback_summary(self, request):
+        resp = create_mock_json('tests/resources/call_feedback_summary.json')
+        request.return_value = resp
+
+        base_uri = 'https://api.twilio.com/2010-04-01/Accounts/AC123'
+        account_sid = 'AC123'
+        auth = (account_sid, "token")
+
+        calls = Calls(base_uri, auth)
+        uri = "%s/Calls/Summary" % base_uri
+        feedback = calls.summary.get()
+        assert_equal(10200, feedback.call_count)
+        assert_equal(729, feedback.call_feedback_count)
+
+        request.assert_called_with('GET', uri, params={}, auth=auth)

twilio/rest/resources/call_feedback.py

@@ -9,10 +9,9 @@ class CallFeedback(InstanceResource):
 
     def __init__(self, parent):
         self.parent = parent
-        super(InstanceResource, self).__init__(
-            parent.uri,
-            parent.auth,
-            parent.timeout,
+        super(CallFeedback, self).__init__(
+            parent,
+            None,
         )
 
 
@@ -42,9 +41,10 @@ def create(self, **kwargs):
     def get(self, **kwargs):
         """ Get the feedback for this call
 
-         Usage:
+        Usage:
+
+        .. code-block:: python
 
-         .. code-block:: python
             feedback = client.calls.get("CA123").feedback
             print feedback.issues
 
@@ -57,7 +57,52 @@ def get(self, **kwargs):
 
     def load_instance(self, data):
         # Overridden because CallFeedback instances
-        # don't contain sids :(
+        # don't contain sids
+        instance = self.instance(self)
+        instance.load(data)
+        return instance
+
+
+class CallFeedbackSummaryInstance(InstanceResource):
+
+    def __init__(self, parent):
+        self.parent = parent
+        super(CallFeedbackSummaryInstance, self).__init__(
+            parent,
+            None,
+        )
+
+
+class CallFeedbackSummary(ListResource):
+
+    name = "Summary"
+    key = "Feedback"
+    instance = CallFeedbackSummaryInstance
+
+    def __init__(self, parent, *args, **kwargs):
+        super(CallFeedbackSummary, self).__init__(*args, **kwargs)
+        self.base_uri = parent.uri
+
+    def get(self, **kwargs):
+        """ Get the feedback summary for calls on this account
+
+        Usage:
+
+        .. code-block:: python
+
+            summary = client.calls.summary.get()
+            print summary.quality_score_average
+
+        :rtype: :class:`~twilio.rest.resources.InstanceResource`
+        :raises: a :exc:`~twilio.TwilioRestException` if the request fails
+        """
+        params = transform_params(kwargs)
+        _, data = self.request('GET', self.uri, params=params)
+        return self.load_instance(data)
+
+    def load_instance(self, data):
+        # Overridden because CallFeedback summaries
+        # do not contain sids
         instance = self.instance(self)
         instance.load(data)
         return instance

twilio/rest/resources/calls.py

@@ -1,6 +1,9 @@
 from .notifications import Notifications
 from .recordings import Recordings
-from twilio.rest.resources.call_feedback import CallFeedbackFactory
+from twilio.rest.resources.call_feedback import (
+    CallFeedbackFactory,
+    CallFeedbackSummary,
+)
 from .util import normalize_dates, parse_date
 from . import InstanceResource, ListResource
 
@@ -54,6 +57,11 @@ class Calls(ListResource):
     name = "Calls"
     instance = Call
 
+    def __init__(self, *args, **kwargs):
+        super(Calls, self).__init__(*args, **kwargs)
+
+        self.summary = CallFeedbackSummary(self, *args, **kwargs)
+
     @normalize_dates
     def list(self, from_=None, ended_after=None,
              ended_before=None, ended=None, started_before=None,