From e00a8afc6b2e3fae9a1ec1b237736757d5d9166b Mon Sep 17 00:00:00 2001 From: Calen Pennington Date: Thu, 14 Aug 2014 16:22:19 -0400 Subject: [PATCH] Remove qualifiers from get_courses --- .../lib/xmodule/xmodule/modulestore/split_mongo/split.py | 3 +-- .../xmodule/modulestore/tests/test_split_modulestore.py | 7 ------- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/common/lib/xmodule/xmodule/modulestore/split_mongo/split.py b/common/lib/xmodule/xmodule/modulestore/split_mongo/split.py index 9a64c61cd5..e536cddb64 100644 --- a/common/lib/xmodule/xmodule/modulestore/split_mongo/split.py +++ b/common/lib/xmodule/xmodule/modulestore/split_mongo/split.py @@ -703,7 +703,7 @@ class SplitMongoModuleStore(BulkWriteMixin, ModuleStoreWriteBase): } return envelope - def get_courses(self, branch, qualifiers=None, **kwargs): + def get_courses(self, branch, **kwargs): ''' Returns a list of course descriptors matching any given qualifiers. @@ -714,7 +714,6 @@ class SplitMongoModuleStore(BulkWriteMixin, ModuleStoreWriteBase): To get specific versions via guid use get_course. :param branch: the branch for which to return courses. - :param qualifiers: an optional dict restricting which elements should match ''' matching_indexes = self.find_matching_course_indexes(branch) diff --git a/common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py b/common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py index 25b73786df..266cf582b9 100644 --- a/common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py +++ b/common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py @@ -608,13 +608,6 @@ class SplitModuleCourseTests(SplitModuleTest): _verify_published_course(modulestore().get_courses(branch=BRANCH_NAME_PUBLISHED)) - def test_search_qualifiers(self): - # query w/ search criteria - courses = modulestore().get_courses(branch=BRANCH_NAME_DRAFT, qualifiers={'org': 'testx'}) - self.assertEqual(len(courses), 2) - self.assertIsNotNone(self.findByIdInResult(courses, "head12345")) - self.assertIsNotNone(self.findByIdInResult(courses, "head23456")) - def test_has_course(self): ''' Test the various calling forms for has_course