Merge pull request #98 from dynata/6-get-sections

added get survey sections function

Author: dynata-bradley

CmixAPIClient/api.py

@@ -235,6 +235,19 @@ def get_survey_status(self, survey_id):
             raise CmixError('Get Survey Status returned without a status. Response: {}'.format(status_response.json()))
         return status.lower()
 
+    def get_survey_sections(self, survey_id):
+        self.check_auth_headers()
+        sections_url = '{}/surveys/{}/sections'.format(CMIX_SERVICES['survey'][self.url_type], survey_id)
+        sections_response = requests.get(sections_url, headers=self._authentication_headers)
+        if sections_response.status_code != 200:
+            raise CmixError(
+                'CMIX returned a non-200 response code while getting sections: {} and error {}'.format(
+                    sections_response.status_code,
+                    sections_response.text
+                )
+            )
+        return sections_response.json()
+
     def get_survey_sources(self, survey_id):
         self.check_auth_headers()
         sources_url = '{}/surveys/{}/sources'.format(CMIX_SERVICES['survey'][self.url_type], survey_id)

README.md

@@ -33,6 +33,7 @@ A Python client library for the [Dynata Cmix API](https://wiki2.criticalmix.net/
     get_survey_definition(survey_id)
     get_survey_locales(survey_id)
     get_survey_xml(survey_id)
+    get_survey_sections(survey_id)
     get_survey_simulations(survey_id)
     get_survey_sources(survey_id)
     get_survey_test_url(survey_id)

tests/test_api.py

@@ -169,6 +169,32 @@ def test_get_survey_status_error_handled(self):
             with self.assertRaises(CmixError):
                 self.cmix_api.get_survey_status(self.survey_id)
 
+    def test_get_survey_sections(self):
+        self.cmix_api._authentication_headers = {'Authentication': 'Bearer test'}
+
+        # success case
+        with mock.patch('CmixAPIClient.api.requests') as mock_request:
+            mock_get = mock.Mock()
+            mock_get.status_code = 200
+            mock_get.json.return_value = {}
+            mock_request.get.return_value = mock_get
+
+            self.cmix_api.get_survey_sections(self.survey_id)
+
+            base_url = CMIX_SERVICES['survey']['BASE_URL']
+            surveys_url = '{}/surveys/{}/sections'.format(base_url, self.survey_id)
+            mock_request.get.assert_any_call(surveys_url, headers=self.cmix_api._authentication_headers)
+
+        # error case (survey not found)
+        with mock.patch('CmixAPIClient.api.requests') as mock_request:
+            mock_get = mock.Mock()
+            mock_get.status_code = 404
+            mock_get.json.return_value = {}
+            mock_request.get.return_value = mock_get
+
+            with self.assertRaises(CmixError):
+                self.cmix_api.get_survey_sections(self.survey_id)
+
     def test_get_survey_locales(self):
         self.cmix_api._authentication_headers = {'Authentication': 'Bearer test'}