From 5fa44123bfaccc8b6acd7cf8fd198642a8380540 Mon Sep 17 00:00:00 2001 From: Don Mitchell Date: Mon, 28 Oct 2013 12:27:04 -0400 Subject: [PATCH] Allow 'course' as course's id keyword --- common/lib/xmodule/xmodule/modulestore/tests/factories.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/common/lib/xmodule/xmodule/modulestore/tests/factories.py b/common/lib/xmodule/xmodule/modulestore/tests/factories.py index 7d887bb81b..5fe0dc8849 100644 --- a/common/lib/xmodule/xmodule/modulestore/tests/factories.py +++ b/common/lib/xmodule/xmodule/modulestore/tests/factories.py @@ -46,7 +46,8 @@ class CourseFactory(XModuleFactory): # passed in via **kwargs. However, some of those aren't actual field values, # so pop those off for use separately org = kwargs.pop('org', None) - number = kwargs.pop('number', kwargs.pop('course', None)) + # because the factory provides a default 'number' arg, prefer the non-defaulted 'course' arg if any + number = kwargs.pop('course', kwargs.pop('number', None)) store = kwargs.pop('modulestore') location = Location('i4x', org, number, 'course', Location.clean(kwargs.get('display_name')))