Fix Python and JavaScript (Karma) tests for Problem XBlock by irtazaakram · Pull Request #151 · openedx/xblocks-contrib

irtazaakram · 2026-02-03T05:44:49Z

Summary

This PR fixes the Python and JavaScript (Karma) tests for the newly extracted Problem (CAPA) XBlock.

Context

Extraction PR: Adds Problem XBlock extracting from edx-platform repo #63
Platform Enablement PR: Ensure Problem Block tests run on built-in block and integration tests cover both Problem Blocks openedx-platform#37136

What Changed

Python Tests: Fixed import paths and module references in tests/ to align with the new directory structure.
Javascript Tests: Added Karma configuration and Webpack config.
Setup: Changed how xblocks-contribute is installed by requiring npm build.

Notes

Pylint cleanup and documentation updates are out of scope for this PR and will be addressed in Fix Quality and Docs tests for Problem XBlock #153 and that will be merged into this branch for it to be ready.
The primary goal of this PR is to get the test suite green to unblock integration.

.github/workflows/javascript-tests.yml

.github/workflows/pypi-publish.yml

xblocks_contrib/problem/assets/webpack.config.js

package.json

kdmccormick

I'm still working through the review, here are my comments so far

xblocks_contrib/problem/capa/xqueue_interface.py

.github/workflows/javascript-tests.yml

MANIFEST.in

xblocks_contrib/problem/assets/webpack.config.js

xblocks_contrib/problem/tests/__init__.py

xblocks_contrib/problem/tests/test_capa_block.py

xblocks_contrib/problem/capa_block.py

xblocks_contrib/problem/public/css/ProblemBlockDisplay.css

xblocks_contrib/problem/public/js/ProblemBlockDisplay.js

kdmccormick

@irtazaakram , glad to see tests are passing. This looks fine to merge into the problemblock branch if that would be helpful for development, just please incorporate my XBlockUserState TODO comment, if you agree. No need to wait for my approval on this one.

However, I don't want to add XModuleMixin, XModuleToXBlockMixin, or XmlMixin to this repo if possible. They are legacy tech debt and I think that, with some refactoring, we will be able to get everything working without relying on them. Or, if we do merge them in temporarily, I'd like to refactor them away as soon as possible.

If you want to keep them in this PR, could you add TODO comments about removing each of them? Let me know if you walk to talk about strategies for factoring them out later.

xblocks_contrib/problem/assets/karma.conf.js

xblocks_contrib/problem/tests/test_capa_block.py

test_settings.py

.gitignore

xblocks_contrib/problem/assets/webpack.config.js

xblocks_contrib/problem/capa_block.py

xblocks_contrib/problem/assets/package.json

xblocks_contrib/problem/capa_block.py

salman2013

LGTM

* fix: add problem xblock * fix: python and javascript tests for problem xblock (#151) * fix: quality & doc tests (#153)

irtazaakram changed the title ~~fix: python & javascript tests~~ Fix Python and JavaScript (Karma) tests for Problem Xblock Feb 3, 2026

irtazaakram changed the title ~~Fix Python and JavaScript (Karma) tests for Problem Xblock~~ Fix Python and JavaScript (Karma) tests for Problem XBlock Feb 3, 2026

irtazaakram self-assigned this Feb 3, 2026

irtazaakram moved this to 👀 In review in Aximprovements Team Feb 3, 2026

irtazaakram added this to Aximprovements Team Feb 3, 2026

irtazaakram force-pushed the fix-problemblock branch 2 times, most recently from 3c433b8 to e0497fd Compare February 3, 2026 14:33

irtazaakram mentioned this pull request Feb 4, 2026

Fix Quality and Docs tests for Problem XBlock #153

Merged

kdmccormick self-requested a review February 4, 2026 14:16

salman2013 requested changes Feb 6, 2026

View reviewed changes

.github/workflows/javascript-tests.yml Outdated Show resolved Hide resolved

.github/workflows/pypi-publish.yml Outdated Show resolved Hide resolved

xblocks_contrib/problem/assets/webpack.config.js Show resolved Hide resolved

package.json Outdated Show resolved Hide resolved

kdmccormick reviewed Feb 9, 2026

View reviewed changes

xblocks_contrib/problem/capa/xqueue_interface.py Outdated Show resolved Hide resolved

.github/workflows/javascript-tests.yml Outdated Show resolved Hide resolved

irtazaakram force-pushed the fix-problemblock branch from 9c42194 to c52b875 Compare February 9, 2026 14:30

farhan requested changes Feb 9, 2026

View reviewed changes

MANIFEST.in Outdated Show resolved Hide resolved

xblocks_contrib/problem/assets/webpack.config.js Show resolved Hide resolved

xblocks_contrib/problem/tests/__init__.py Outdated Show resolved Hide resolved

xblocks_contrib/problem/tests/test_capa_block.py Outdated Show resolved Hide resolved

farhan requested changes Feb 9, 2026

View reviewed changes

xblocks_contrib/problem/capa_block.py Show resolved Hide resolved

xblocks_contrib/problem/capa_block.py Outdated Show resolved Hide resolved

xblocks_contrib/problem/capa_block.py Outdated Show resolved Hide resolved

xblocks_contrib/problem/capa_block.py Show resolved Hide resolved

irtazaakram mentioned this pull request Feb 10, 2026

Use runtime-provided XQueueService instead of constructing it in ProblemBlock openedx/openedx-platform#37998

Merged

farhan mentioned this pull request Feb 11, 2026

Code Review | Extracted Problem Block PR's openedx/openedx-platform#38002

Closed

salman2013 reviewed Feb 16, 2026

View reviewed changes

xblocks_contrib/problem/capa_block.py Outdated Show resolved Hide resolved

irtazaakram force-pushed the problemblock branch from 264fadc to 9435393 Compare February 19, 2026 07:36

irtazaakram force-pushed the fix-problemblock branch 2 times, most recently from 6599a44 to 363ce85 Compare February 23, 2026 09:18

irtazaakram mentioned this pull request Feb 23, 2026

Adds Problem XBlock extracting from edx-platform repo #63

Merged

irtazaakram requested review from farhan and salman2013 February 23, 2026 09:29

irtazaakram commented Feb 23, 2026

View reviewed changes

xblocks_contrib/problem/public/css/ProblemBlockDisplay.css Outdated Show resolved Hide resolved

irtazaakram commented Feb 23, 2026

View reviewed changes

xblocks_contrib/problem/public/js/ProblemBlockDisplay.js Outdated Show resolved Hide resolved

kdmccormick reviewed Feb 23, 2026

View reviewed changes

xblocks_contrib/problem/assets/karma.conf.js Show resolved Hide resolved

xblocks_contrib/problem/tests/test_capa_block.py Show resolved Hide resolved

test_settings.py Outdated Show resolved Hide resolved

kdmccormick mentioned this pull request Feb 23, 2026

Extract XBlockUserState class to XBlock repo openedx/openedx-platform#38043

Open

irtazaakram force-pushed the problemblock branch from 9435393 to f51be3d Compare February 24, 2026 14:24

irtazaakram force-pushed the fix-problemblock branch from 9721222 to c50800d Compare February 24, 2026 14:44

irtazaakram force-pushed the problemblock branch from f51be3d to 254a524 Compare February 25, 2026 08:01

irtazaakram force-pushed the fix-problemblock branch from f771708 to fe1810d Compare February 25, 2026 08:18

irtazaakram force-pushed the problemblock branch from 254a524 to f0ba9ee Compare February 25, 2026 10:41

irtazaakram force-pushed the fix-problemblock branch from fe1810d to c4cd4db Compare February 25, 2026 10:41

farhan requested changes Feb 27, 2026

View reviewed changes

.gitignore Outdated Show resolved Hide resolved

xblocks_contrib/problem/assets/webpack.config.js Outdated Show resolved Hide resolved

xblocks_contrib/problem/capa_block.py Show resolved Hide resolved

xblocks_contrib/problem/assets/package.json Outdated Show resolved Hide resolved

farhan requested changes Feb 27, 2026

View reviewed changes

xblocks_contrib/problem/capa_block.py Show resolved Hide resolved

xblocks_contrib/problem/capa_block.py Show resolved Hide resolved

farhan requested changes Feb 28, 2026

View reviewed changes

xblocks_contrib/problem/capa_block.py Outdated Show resolved Hide resolved

xblocks_contrib/problem/capa_block.py Show resolved Hide resolved

salman2013 approved these changes Mar 2, 2026

View reviewed changes

irtazaakram force-pushed the problemblock branch from f0ba9ee to ad06e4e Compare March 2, 2026 06:30

fix: python & javascript & openedx-platform tests

1999b3d

irtazaakram force-pushed the fix-problemblock branch from 9e05b31 to 1999b3d Compare March 2, 2026 06:36

fix: add package-lock.json

6b215e3

farhan approved these changes Mar 2, 2026

View reviewed changes

fix: quality & doc tests (#153)

3bb9e4a

irtazaakram merged commit eef1737 into problemblock Mar 2, 2026
12 of 13 checks passed

irtazaakram deleted the fix-problemblock branch March 2, 2026 11:28

github-project-automation bot moved this from 👀 In review to ✅ Done in Aximprovements Team Mar 2, 2026

irtazaakram added a commit that referenced this pull request Mar 2, 2026

fix: adds Problem XBlock from edx-platform repo (#63)

5b1b088

* fix: add problem xblock * fix: python and javascript tests for problem xblock (#151) * fix: quality & doc tests (#153)

irtazaakram mentioned this pull request Mar 25, 2026

Remove legacy XModule mixins from xblocks-contrib XBlocks #197

Open

Conversation

irtazaakram commented Feb 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Context

What Changed

Notes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kdmccormick left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

kdmccormick left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

salman2013 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

irtazaakram commented Feb 3, 2026 •

edited

Loading

kdmccormick left a comment •

edited

Loading