Skip to content

Commit 027deab

Browse files
committed
temp: disable broken test suites for Webpack-built JS
Should be re-enabled in: openedx#35956
1 parent 78e9b3a commit 027deab

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

pavelib/utils/envs.py

+12-8
Original file line numberDiff line numberDiff line change
@@ -88,26 +88,30 @@ class Env:
8888
KARMA_BROWSER = 'FirefoxNoUpdates'
8989

9090
# Files used to run each of the js test suites
91-
# TODO: Store this as a dict. Order seems to matter for some
92-
# reason. See issue TE-415.
91+
# TODO: We have [temporarily disabled] the three Webpack-based tests suites. They have been silently
92+
# broken for a long time; after noticing they were broken, we added the DieHardPlugin to
93+
# webpack.common.config.js to prevent future silent breakage, but have not yet been able to
94+
# fix and re-enable the suites. Note that the LMS suite is all Webpack-based even though it's
95+
# not in the name.
96+
# Issue: https://github.com/openedx/edx-platform/issues/35956
9397
KARMA_CONFIG_FILES = [
9498
REPO_ROOT / 'cms/static/karma_cms.conf.js',
9599
REPO_ROOT / 'cms/static/karma_cms_squire.conf.js',
96-
REPO_ROOT / 'cms/static/karma_cms_webpack.conf.js',
97-
REPO_ROOT / 'lms/static/karma_lms.conf.js',
100+
## [temporarily disabled] REPO_ROOT / 'cms/static/karma_cms_webpack.conf.js',
101+
## [temporarily disabled] REPO_ROOT / 'lms/static/karma_lms.conf.js',
98102
REPO_ROOT / 'xmodule/js/karma_xmodule.conf.js',
99-
REPO_ROOT / 'xmodule/js/karma_xmodule_webpack.conf.js',
103+
## [temporarily disabled] REPO_ROOT / 'xmodule/js/karma_xmodule_webpack.conf.js',
100104
REPO_ROOT / 'common/static/karma_common.conf.js',
101105
REPO_ROOT / 'common/static/karma_common_requirejs.conf.js',
102106
]
103107

104108
JS_TEST_ID_KEYS = [
105109
'cms',
106110
'cms-squire',
107-
'cms-webpack',
108-
'lms',
111+
## [temporarily-disabled] 'cms-webpack',
112+
## [temporarily-disabled] 'lms',
109113
'xmodule',
110-
'xmodule-webpack',
114+
## [temporarily-disabled] 'xmodule-webpack',
111115
'common',
112116
'common-requirejs',
113117
'jest-snapshot'

0 commit comments

Comments
 (0)