Commit f60923c6 by Clinton Blackburn

Removed submission_counts method

This method/endpoint will be replaced by a course-problems endpoint.
parent b12ac07b
import analyticsclient.constants.data_format as DF
from analyticsclient.exceptions import InvalidRequestError
class Module(object):
......@@ -29,22 +28,6 @@ class Module(object):
return self.client.get(path, data_format=data_format)
def submission_counts(self, module_ids, data_format=DF.JSON):
"""
Get submission counts data for multiple modules.
Arguments:
module_ids (list[str]): IDs of modules for which data should be returned
data_format (str): Format in which to return data (default is JSON)
"""
if not module_ids:
raise InvalidRequestError('At least one module ID must be supplied.')
module_ids = ','.join(module_ids)
path = 'problems/submission_counts/?problem_ids={}'.format(module_ids)
return self.client.get(path, data_format=data_format)
def grade_distribution(self, data_format=DF.JSON):
"""
Get grade distribution data for a module.
......
import json
import httpretty
from analyticsclient.exceptions import InvalidRequestError
from analyticsclient.tests import ClientTestCase
......@@ -95,19 +95,3 @@ class ModulesTests(ClientTestCase):
uri = self.get_api_url('problems/{0}/grade_distribution/'.format(self.module_id))
httpretty.register_uri(httpretty.GET, uri, body=json.dumps(body))
self.assertEqual(body, self.module.grade_distribution())
@httpretty.activate
def test_submission_counts(self):
self.assertRaises(InvalidRequestError, self.module.submission_counts, None)
body = [
{
'module_id': self.module_id,
'total': 100,
'correct': 80
}
]
uri = self.get_api_url('problems/submission_counts/')
httpretty.register_uri(httpretty.GET, uri, body=json.dumps(body))
self.assertEqual(body, self.module.submission_counts([self.module_id]))
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment