Skip to content
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.

Sign up for GitHub

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

chore: Upgrade Python requirements #332

Merged
merged 2 commits into from
Oct 18, 2024
Merged

chore: Upgrade Python requirements #332

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
de21193
chore: Upgrade Python requirements
edx-requirements-bot Oct 18, 2024
939e361
style: fix pylint errors
MichaelRoytman Oct 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions edx_exams/apps/api/v1/tests/test_views.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1789,6 +1789,7 @@ def setUp(self):
course_id=self.course_id,
)

# pylint: disable=too-many-positional-arguments
def request_api(self, method, user, course_id, data=None, allowance_id=None):
"""
Helper function to make API request
Expand Down
8 changes: 4 additions & 4 deletions edx_exams/apps/core/models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -380,16 +380,16 @@ def create_or_update(cls, course_id, provider, escalation_email):
# and duplicates made with the new provider.
if provider != existing_provider:
count = cls._sync_exams_with_new_provider(existing_config.course_id, provider)
log.info(f'Updated course exam configuration course_id={course_id} '
+ f'to provider={provider_name} and recreated {count} exams')
log.info('Updated course exam configuration course_id=%s to provider=%s and recreated %d exams',
course_id, provider_name, count)
else:
CourseExamConfiguration.objects.create(
course_id=course_id,
escalation_email=escalation_email,
provider=provider,
)
log.info(f'Created course exam configuration course_id={course_id}, provider={provider_name}, '
+ f'escalation_email={escalation_email}')
log.info('Created course exam configuration course_id=%s, provider=%s, escalation_email=%s',
course_id, provider_name, escalation_email)

@classmethod
def update_course_config(cls, existing_config, new_provider, escalation_email):
Expand Down
3 changes: 3 additions & 0 deletions edx_exams/apps/lti/tests/test_views.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -158,6 +158,7 @@ def make_post_request(self, request_body, token):
@patch.object(Lti1p3ApiAuthentication, 'authenticate', return_value=(AnonymousUser(), None))
@patch('edx_exams.apps.lti.views.LtiProctoringAcsPermissions.has_permission')
@patch('edx_exams.apps.lti.views.get_attempt_for_user_with_attempt_number_and_resource_id')
# pylint: disable=too-many-positional-arguments
def test_acs_attempt_status(self,
attempt_status,
expected_response_status,
Expand Down Expand Up @@ -210,6 +211,7 @@ def test_acs_no_attempt_found(self,
@patch.object(Lti1p3ApiAuthentication, 'authenticate', return_value=(AnonymousUser(), None))
@patch('edx_exams.apps.lti.views.LtiProctoringAcsPermissions.has_permission')
@patch('edx_exams.apps.lti.views.get_attempt_for_user_with_attempt_number_and_resource_id')
# pylint: disable=too-many-positional-arguments
def test_acs_base_parameter_missing_errors(self,
acs_parameter,
acs_sub_parameter,
Expand Down Expand Up @@ -317,6 +319,7 @@ def test_acs_invalid_action(self,
@patch.object(Lti1p3ApiAuthentication, 'authenticate', return_value=(AnonymousUser(), None))
@patch('edx_exams.apps.lti.views.LtiProctoringAcsPermissions.has_permission')
@patch('edx_exams.apps.lti.views.get_attempt_for_user_with_attempt_number_and_resource_id')
# pylint: disable=too-many-positional-arguments
def test_acs_terminate(self,
reason_code,
incident_severity,
Expand Down
36 changes: 18 additions & 18 deletions requirements/base.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,9 @@ attrs==24.2.0
# openedx-events
bleach==6.1.0
# via lti-consumer-xblock
boto3==1.35.22
boto3==1.35.44
# via fs-s3fs
botocore==1.35.22
botocore==1.35.44
# via
# boto3
# s3transfer
Expand All @@ -29,7 +29,7 @@ cffi==1.17.1
# via
# cryptography
# pynacl
charset-normalizer==3.3.2
charset-normalizer==3.4.0
# via requests
click==8.1.7
# via
Expand All @@ -43,7 +43,7 @@ coreapi==2.3.3
# openapi-codec
coreschema==0.0.4
# via coreapi
cryptography==43.0.1
cryptography==43.0.3
# via
# pyjwt
# social-auth-core
Expand Down Expand Up @@ -87,7 +87,7 @@ django-appconf==1.0.6
# via django-statici18n
django-config-models==2.7.0
# via lti-consumer-xblock
django-cors-headers==4.4.0
django-cors-headers==4.5.0
# via -r requirements/base.in
django-crum==0.7.9
# via
Expand Down Expand Up @@ -120,11 +120,11 @@ djangorestframework==3.15.2
# drf-yasg
# edx-api-doc-tools
# edx-drf-extensions
dnspython==2.6.1
dnspython==2.7.0
# via pymongo
drf-jwt==1.19.2
# via edx-drf-extensions
drf-yasg==1.21.7
drf-yasg==1.21.8
# via edx-api-doc-tools
edx-api-doc-tools==2.0.0
# via -r requirements/base.in
Expand All @@ -136,7 +136,7 @@ edx-ccx-keys==1.3.0
# openedx-events
edx-django-release-util==1.4.0
# via -r requirements/base.in
edx-django-utils==5.15.0
edx-django-utils==7.0.0
# via
# -r requirements/base.in
# django-config-models
Expand All @@ -150,7 +150,7 @@ edx-drf-extensions==10.4.0
# via -r requirements/base.in
edx-event-bus-kafka==5.8.1
# via -r requirements/base.in
edx-event-bus-redis==0.5.0
edx-event-bus-redis==0.5.1
# via -r requirements/base.in
edx-opaque-keys[django]==2.11.0
# via
Expand Down Expand Up @@ -206,14 +206,14 @@ mako==1.3.5
# via
# lti-consumer-xblock
# xblock
markupsafe==2.1.5
markupsafe==3.0.2
# via
# jinja2
# mako
# xblock
mysqlclient==2.2.4
# via -r requirements/base.in
newrelic==9.13.0
newrelic==10.2.0
# via edx-django-utils
oauthlib==3.2.2
# via
Expand All @@ -224,22 +224,22 @@ openapi-codec==1.3.2
# via django-rest-swagger
openedx-django-pyfs==3.7.0
# via lti-consumer-xblock
openedx-events==9.14.1
openedx-events==9.15.0
# via
# -r requirements/base.in
# edx-event-bus-kafka
# edx-event-bus-redis
openedx-filters==1.9.0
openedx-filters==1.11.0
# via lti-consumer-xblock
packaging==24.1
# via drf-yasg
pbr==6.1.0
# via stevedore
psutil==6.0.0
psutil==6.1.0
# via edx-django-utils
pycparser==2.22
# via cffi
pycryptodomex==3.20.0
pycryptodomex==3.21.0
# via
# lti-consumer-xblock
# pyjwkest
Expand All @@ -256,7 +256,7 @@ pyjwt[crypto]==2.9.0
# social-auth-core
pymemcache==4.0.0
# via -r requirements/base.in
pymongo==4.9.1
pymongo==4.10.1
# via edx-opaque-keys
pynacl==1.5.0
# via edx-django-utils
Expand All @@ -279,7 +279,7 @@ pyyaml==6.0.2
# drf-yasg
# edx-django-release-util
# xblock
redis==5.0.8
redis==5.1.1
# via walrus
requests==2.32.3
# via
Expand All @@ -291,7 +291,7 @@ requests==2.32.3
# social-auth-core
requests-oauthlib==2.0.0
# via social-auth-core
s3transfer==0.10.2
s3transfer==0.10.3
# via boto3
semantic-version==2.10.0
# via edx-drf-extensions
Expand Down
8 changes: 4 additions & 4 deletions requirements/ci.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,9 @@ chardet==5.2.0
# via tox
colorama==0.4.6
# via tox
coverage==7.6.1
coverage==7.6.3
# via -r requirements/ci.in
distlib==0.3.8
distlib==0.3.9
# via virtualenv
filelock==3.16.1
# via
Expand All @@ -30,7 +30,7 @@ pluggy==1.5.0
# via tox
pyproject-api==1.8.0
# via tox
tox==4.20.0
tox==4.23.0
# via -r requirements/ci.in
virtualenv==20.26.5
virtualenv==20.27.0
# via tox
Loading
Loading