openedx / openedx/openedx-platform
More unit tests for migration of legacy library_content block
Open
@kdmccormick is already working on this.
Since Jan 5, 2026.
- Dominant language
- Python
- Stars
- 8.2k
- Forks
- 4.4k
- Avg merge
- 6d 18h
- Merged PRs (30d)
- 42
Description
See the docstring of the unit test added by: https://github.com/openedx/edx-platform/pull/37804
It would be good to also test more cases, including:
- When migration occurs which is non-forwarding, it does not affect the childen of this block.
- When the library migration HAS happend but the LC block migration HASN'T YET, then the fields of
the block and its children will be unchanged, but the user will be prompted to upgrade.- When some or all of the blocks already exist in the target library before the migration, then
the migration target versions will NOT all be 1, and the upstream_versions should reflect that.- When the target library blocks have been edited and published AFTER the legacy library migration
but BEFORE the LC block migration, then executing the LC block migration will set upstream_version
based on the migration target versions, NOT the latest versions.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.