From 7778347aedfb10630317aa329301e8bfaa6a5a78 Mon Sep 17 00:00:00 2001 From: Ben Warzeski Date: Fri, 15 May 2020 11:24:37 -0400 Subject: [PATCH] remove old_format from team config tests --- common/lib/xmodule/xmodule/tests/test_course_module.py | 10 ++++++++-- lms/djangoapps/teams/tests/test_serializers.py | 2 +- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/common/lib/xmodule/xmodule/tests/test_course_module.py b/common/lib/xmodule/xmodule/tests/test_course_module.py index 771a1e1ba4..cd2fdbf7ed 100644 --- a/common/lib/xmodule/xmodule/tests/test_course_module.py +++ b/common/lib/xmodule/xmodule/tests/test_course_module.py @@ -295,7 +295,13 @@ class TeamsConfigurationTestCase(unittest.TestCase): topic_id = "topic_id_{}".format(next_num) name = "Name {}".format(next_num) description = "Description {}".format(next_num) - return {"name": name, "description": description, "id": topic_id} + return { + "name": name, + "description": description, + "id": topic_id, + "type": "open", + "max_team_size": None + } def test_teams_enabled_new_course(self): # Make sure we can detect when no teams exist. @@ -341,7 +347,7 @@ class TeamsConfigurationTestCase(unittest.TestCase): self.add_team_configuration(max_team_size=4, topics=topics) self.assertTrue(self.course.teams_enabled) expected_teamsets_data = [ - teamset.cleaned_data_old_format + teamset.cleaned_data for teamset in self.course.teamsets ] self.assertEqual(expected_teamsets_data, topics) diff --git a/lms/djangoapps/teams/tests/test_serializers.py b/lms/djangoapps/teams/tests/test_serializers.py index 082a802157..b55cc37b6f 100644 --- a/lms/djangoapps/teams/tests/test_serializers.py +++ b/lms/djangoapps/teams/tests/test_serializers.py @@ -198,7 +198,7 @@ class BaseTopicSerializerTestCase(SerializerTestCase): """ with self.assertNumQueries(num_queries): page = Paginator( - self.course.teams_configuration.cleaned_data_old_format['topics'], + self.course.teams_configuration.cleaned_data['teamsets'], self.PAGE_SIZE, ).page(1) # pylint: disable=not-callable