-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix missing require_level in Juniper #771
Conversation
bba9111
to
9f0a97f
Compare
edX refactored it into HasAccessRule
5b2af09
to
5aa947f
Compare
def test_allow_beta(self, mock_get_user): | ||
""" Test allow beta ag | ||
mock_get_user.return_value = userainst list beta. """ | ||
def test_allow_beta(self, _mock_get_user): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just clean up for merge leftovers
@@ -3948,7 +3948,6 @@ def test_skip_entrance_exam_student(self): | |||
|
|||
|
|||
@patch('bulk_email.models.html_to_text', Mock(return_value='Mocking CourseEmail.text_message', autospec=True)) | |||
@override_settings(DEFAULT_SITE_THEME='edx-theme-codebase') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not needed
unique_id_for_user, | ||
is_email_retired | ||
is_email_retired, | ||
unique_id_for_user |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
match upstream
edX refactored it into HasAccessRule.