Skip to content

Commit

Permalink
⬆️(backend) upgrade to Django 4.1
Browse files Browse the repository at this point in the history
Some admin view tests required some adjustments.
The count of strings in the admin template of Django
has changed from version 4 to 4.1 of Django.
  • Loading branch information
jonathanreveille committed May 13, 2024
1 parent 0b06154 commit 7a48859
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 22 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ Versioning](https://semver.org/spec/v2.0.0.html).

## Changed

- Upgrade to Django 4.1 version
- Upgrade to Django 4.0 version and postgres to version 12 in
docker compose file
- Migrate to Sentry SDK 2.0
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ classifiers = [
]
dependencies = [
"arrow",
"Django==4.0",
"Django==4.1",
"djangocms-file",
"djangocms-googlemap",
"djangocms-link",
Expand Down
42 changes: 21 additions & 21 deletions tests/apps/courses/test_admin_course_run.py
Original file line number Diff line number Diff line change
Expand Up @@ -52,15 +52,15 @@ def test_admin_course_run_add_view(self):
response = self.client.get(url, follow=True)

# Check that the page includes all our fields
self.assertContains(response, "id_title")
self.assertContains(response, "id_resource_link")
self.assertContains(response, "id_start_", count=3)
self.assertContains(response, "id_end_", count=3)
self.assertContains(response, "id_enrollment_start_", count=3)
self.assertContains(response, "id_enrollment_end_", count=3)
self.assertContains(response, "id_languages")
self.assertContains(response, "id_enrollment_count")
self.assertContains(response, "id_sync_mode")
self.assertContains(response, "id_title", count=2)
self.assertContains(response, "id_resource_link", count=2)
self.assertContains(response, "id_start_", count=2)
self.assertContains(response, "id_end_", count=2)
self.assertContains(response, "id_enrollment_start_", count=2)
self.assertContains(response, "id_enrollment_end_", count=2)
self.assertContains(response, "id_languages", count=3)
self.assertContains(response, "id_enrollment_count", count=3)
self.assertContains(response, "id_sync_mode", count=2)

# List

Expand Down Expand Up @@ -208,12 +208,12 @@ def test_admin_course_run_change_view_get_superuser_draft(self):
# Check that the page includes all our fields
self.assertContains(response, "id_title", count=2)
self.assertContains(response, "id_resource_link", count=2)
self.assertContains(response, "id_start_", count=3)
self.assertContains(response, "id_end_", count=3)
self.assertContains(response, "id_enrollment_start_", count=3)
self.assertContains(response, "id_enrollment_end_", count=3)
self.assertContains(response, "id_languages", count=2)
self.assertContains(response, "id_enrollment_count", count=2)
self.assertContains(response, "id_start_", count=2)
self.assertContains(response, "id_end_", count=2)
self.assertContains(response, "id_enrollment_start_", count=2)
self.assertContains(response, "id_enrollment_end_", count=2)
self.assertContains(response, "id_languages", count=3)
self.assertContains(response, "id_enrollment_count", count=3)
self.assertContains(response, "id_sync_mode", count=2)

def test_admin_course_run_change_view_get_superuser_public(self):
Expand Down Expand Up @@ -319,12 +319,12 @@ def test_admin_course_run_change_view_get_staff_all_permissions(self):
# Check that the page includes all our fields
self.assertContains(response, "id_title", count=2)
self.assertContains(response, "id_resource_link", count=2)
self.assertContains(response, "id_start_", count=3)
self.assertContains(response, "id_end_", count=3)
self.assertContains(response, "id_enrollment_start_", count=3)
self.assertContains(response, "id_enrollment_end_", count=3)
self.assertContains(response, "id_languages", count=2)
self.assertContains(response, "id_enrollment_count", count=2)
self.assertContains(response, "id_start_", count=2)
self.assertContains(response, "id_end_", count=2)
self.assertContains(response, "id_enrollment_start_", count=2)
self.assertContains(response, "id_enrollment_end_", count=2)
self.assertContains(response, "id_languages", count=3)
self.assertContains(response, "id_enrollment_count", count=3)
self.assertContains(response, "id_sync_mode", count=2)

# Add
Expand Down

0 comments on commit 7a48859

Please sign in to comment.