Skip to content

Ensure Problem Block tests run on built-in block and integration tests cover both Problem Blocks - #37136

Merged
irtazaakram merged 2 commits into
masterfrom
problem-extracted
Mar 17, 2026
Merged

Ensure Problem Block tests run on built-in block and integration tests cover both Problem Blocks#37136
irtazaakram merged 2 commits into
masterfrom
problem-extracted

Conversation

@irtazaakram

@irtazaakram irtazaakram commented Aug 7, 2025

Copy link
Copy Markdown
Member

This PR:

  • Ensures xmodule/tests/test_capa_block.py always runs against the built-in problem block, since these tests were moved to xblocks_contrib/problem/tests/test_capa_block.py.
  • Updates the integration test lms/djangoapps/grades/tests/integration/test_problems.py to run against both the built-in and extracted problem blocks, regardless of the global flag value.

@irtazaakram
irtazaakram force-pushed the problem-extracted branch 2 times, most recently from b27565c to bbc29bc Compare September 8, 2025 06:53
@irtazaakram
irtazaakram force-pushed the problem-extracted branch 3 times, most recently from 8e1648c to c98f428 Compare February 2, 2026 19:45
@irtazaakram irtazaakram added the create-sandbox open-craft-grove should create a sandbox environment from this PR label Feb 2, 2026
@irtazaakram irtazaakram moved this to 👀 In review in Aximprovements Team Feb 3, 2026
@irtazaakram irtazaakram self-assigned this Feb 3, 2026
@irtazaakram
irtazaakram force-pushed the problem-extracted branch 3 times, most recently from 95bca31 to eeee4c6 Compare February 4, 2026 10:16
@irtazaakram irtazaakram removed the create-sandbox open-craft-grove should create a sandbox environment from this PR label Feb 11, 2026
@irtazaakram
irtazaakram changed the base branch from master to purge-xmodule/capa February 25, 2026 12:47
@irtazaakram
irtazaakram force-pushed the problem-extracted branch 2 times, most recently from 736de78 to 57ff387 Compare February 26, 2026 15:45
@irtazaakram
irtazaakram force-pushed the purge-xmodule/capa branch 2 times, most recently from c1c0a13 to 64d2479 Compare March 3, 2026 08:49
Base automatically changed from purge-xmodule/capa to master March 10, 2026 13:58
@irtazaakram irtazaakram changed the title Test PR | Enable Extracted Problem XBlock Ensure Problem Block tests run on built-in block and integration tests cover both Problem Blocks Mar 11, 2026
@irtazaakram
irtazaakram marked this pull request as ready for review March 11, 2026 11:22
@irtazaakram
irtazaakram requested a review from kdmccormick March 11, 2026 14:19
@farhan

farhan commented Mar 17, 2026

Copy link
Copy Markdown
Contributor

@farhan farhan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add link of the moved test cases.
Rest all seems good

@kdmccormick kdmccormick left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@irtazaakram
irtazaakram merged commit 056acff into master Mar 17, 2026
51 checks passed
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Aximprovements Team Mar 17, 2026
@irtazaakram
irtazaakram deleted the problem-extracted branch March 17, 2026 16:12
taimoor-ahmed-1 pushed a commit to edly-io/edx-platform that referenced this pull request Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants