fix: address tests failures after refactor

openedx · Apr 23, 2024 · 0d240e3 · 0d240e3
1 parent 0a192bd
commit 0d240e3
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 24 deletions.
diff --git a/cms/djangoapps/contentstore/tests/test_course_settings.py b/cms/djangoapps/contentstore/tests/test_course_settings.py
@@ -1753,29 +1753,6 @@ def _set_request_user_to_staff(self):
         self.request.user = self.user
         set_current_request(self.request)
 
-    def test_get_team_sets_from_settings(self):
-        """
-        Tests that get_team_sets_from_settings returns the correct team sets.
-        """
-        settings_dict = {
-            "teams_configuration": {
-                "value": {
-                    "team_sets": [
-                        {
-                            "team_id": "team1",
-                        },
-                        {
-                            "team_id": "team2",
-                        }
-                    ]
-                }
-            }
-        }
-
-        team_sets = CourseMetadata.get_team_sets_from_settings(settings_dict)
-
-        self.assertEqual(team_sets, settings_dict["teams_configuration"]["value"]["team_sets"])
-
     def test_team_content_groups_off(self):
         """
         Tests that user_partition_id is not added to the model when content groups for teams are off.

diff --git a/lms/djangoapps/teams/team_partition_scheme.py b/lms/djangoapps/teams/team_partition_scheme.py
@@ -40,7 +40,7 @@ def groups(self):
         if not CONTENT_GROUPS_FOR_TEAMS.is_enabled(course_key):
             return []
 
-        # Get the team-set for this partition via the team_sets_mapping and then get the teams in that team-set
+        # Get the team-set for this partition via the partition parameters and then get the teams in that team-set
         # to create the groups for this partition.
         team_sets = TeamsConfigurationService().get_teams_configuration(course_key).teamsets
         team_set_id = self.parameters["team_set_id"]