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.

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

[WIP] Edly: migrate tests for native APIs coverage #587

Draft
wants to merge 8 commits into
base: edly/forumv2
Choose a base branch
from

Conversation

Faraz32123
Copy link

This PR is still a work in progress and not yet ready for final review.

  • In this PR, edx-platform tests will be modified/replicated to use forum v2 instead of v1.

Muhammad Faraz Maqsood and others added 8 commits October 18, 2024 16:33
This commit introduces the new Forum V2 application, allowing users to choose between the legacy Forum V1 and the new Forum V2 at the course level.

Key Changes:
- Added waffle flag `forum_v2.enable_forum_v2` to enable Forum V2 for selected courses, allowing coexistence with Forum V1.
- Default data storage for Forum V2 is set to MongoDB, with an option to switch to MySQL using the waffle flag `forum_v2.enable_mysql_backend`.
- Introduced management command `forum_migrate_course_from_mongodb_to_mysql` for per-course data migration from MongoDB to MySQL.

Note: This PR does not include all unit tests for the Forum V2 native API due to ongoing migration efforts. Further updates will follow to ensure full test coverage before final release.

Co-authored-by: [Muhammad Faraz Maqsood] <faraz.maqsood@arbisoft.com>
Co-authored-by: [Ali Salman] <ali.salman@arbisoft.com>
- migrated the tests for native APIs from test_serializers to test_serializers_v2 file.
- tests from test_views to test_views_v2 is still in progress
- tests for native APIs are migrated from test_views.py file to test_views_v2.py file are completed.
@regisb regisb force-pushed the edly/forumv2 branch 5 times, most recently from 400dc29 to 4a75633 Compare November 12, 2024 20:22
@Faraz32123 Faraz32123 force-pushed the edly/forumv2 branch 8 times, most recently from 9f0f7a5 to 9f9b02a Compare November 19, 2024 16:01
@regisb regisb force-pushed the edly/forumv2 branch 5 times, most recently from 1f83f24 to 6a28970 Compare December 3, 2024 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant