Skip to content

Commit

Permalink
temp: fix TestDiscussionCourseEnrollmentSerializer.test_discussion_ta…
Browse files Browse the repository at this point in the history
…b_url failed tests for Palm

Tests will fail again after Palm upgrade
  • Loading branch information
OmarIthawi committed Jun 4, 2024
1 parent 37b7549 commit 83ead13
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions lms/djangoapps/mobile_api/users/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@


import datetime
import unittest
from unittest.mock import patch
from urllib.parse import parse_qs

Expand Down Expand Up @@ -36,6 +37,7 @@
)
from lms.djangoapps.mobile_api.utils import API_V1, API_V05, API_V2, API_V3
from openedx.core.lib.courses import course_image_url
from openedx.core.release import RELEASE_LINE
from openedx.core.djangoapps.discussions.models import DiscussionsConfiguration
from openedx.features.course_duration_limits.models import CourseDurationLimitConfig
from openedx.features.course_experience.tests.views.helpers import add_course_mode
Expand Down Expand Up @@ -752,6 +754,10 @@ def get_serialized_data(self, api_version):
context={'request': self.request, 'api_version': api_version},
).data

@unittest.skipIf(
condition=RELEASE_LINE == 'palm',
reason='Temporarily disable in NELC Palm because of Redwood mobile api cherry-picks',
)
@ddt.data(True, False)
def test_discussion_tab_url(self, discussion_tab_enabled):
"""
Expand Down

0 comments on commit 83ead13

Please sign in to comment.