openedx / openedx/openedx-platform

More unit tests for migration of legacy library_content block

Open
#37,837 0 comments 0 reactions 1 assignee View on GitHub

@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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.