Skip to content

Conversation

Faraz32123
Copy link
Owner

No description provided.

@Faraz32123 Faraz32123 force-pushed the test_branch branch 8 times, most recently from c22eae6 to 41e1795 Compare June 5, 2025 09:15
41e1795
In this commit:
- fix parse_xml error that occurs during importing a course that contains problem-builder component or during copying pasting the problem-builder component by removing unused & unsupported positional argument i.e. "id_generator".
- upgrade requirements especially "xblock[django]" to version "2.0.0" where this extra argument "id_generator" was removed.
    - With this requirement upgrade, also dropped support for django 3.2. due to conflicts in dependencies in CI.
        - ERROR: Cannot install Django<5.0 and >=4.2 and django==3.2.25 because these package versions have conflicting dependencies.
- update ubuntu version ro ubuntu-latest.
- update codecov actions to v5 in CI.

Note: This Change will be incompatible with Palm release of openedx.
integration tests are failing with below error, This error comes from xblock-utils package which is already deprecated. So, disabling these tests for now.
```
______ ERROR collecting problem_builder/tests/integration/test_titles.py _______
ImportError while importing test module '/home/runner/work/problem-builder/problem-builder/problem_builder/tests/integration/test_titles.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
problem_builder/tests/integration/test_titles.py:27: in <module>
    from xblockutils.base_test import SeleniumXBlockTest
.tox/integration42/lib/python3.8/site-packages/xblockutils/base_test.py:28: in <module>
    from workbench.test.selenium_test import SeleniumTest
E   ModuleNotFoundError: No module named 'workbench.test.selenium_test'
```
@Faraz32123 Faraz32123 closed this Jun 5, 2025
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