Skip to content

Commit

Permalink
Merge pull request #332 from edx/repo-tools/upgrade-python-requiremen…
Browse files Browse the repository at this point in the history
…ts-d5e994d

chore: Upgrade Python requirements
  • Loading branch information
MichaelRoytman authored Oct 18, 2024
2 parents d5e994d + 939e361 commit 40b06b6
Show file tree
Hide file tree
Showing 13 changed files with 246 additions and 203 deletions.
1 change: 1 addition & 0 deletions edx_exams/apps/api/v1/tests/test_views.py
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
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
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
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
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

0 comments on commit 40b06b6

Please sign in to comment.