Merge pull request #23078 from edx/adeel/prod_960_fixing_translations_failure

Removing paver command
This commit is contained in:
adeel khan
2020-02-12 12:15:38 +05:00
committed by GitHub
3 changed files with 5 additions and 7 deletions

View File

@@ -51,7 +51,7 @@ pull_translations: ## pull translations from Transifex
git clean -fdX conf/locale/rtl
git clean -fdX conf/locale/eo
i18n_tool validate
paver i18n_compilejs
detect_changed_source_translations: ## check if translation files are up-to-date
i18n_tool changed

View File

@@ -91,17 +91,13 @@ def i18n_dummy():
@task
@needs(
"pavelib.prereqs.install_prereqs",
"pavelib.i18n.i18n_validate_gettext",
)
@cmdopts([
("settings=", "s", "The settings to use (defaults to devstack)"),
])
@timed
def i18n_compilejs(options):
"""
Generating djangojs.mo file using django-statici18n
Generating djangojs.js files using django-statici18n
"""
settings = options.get('settings', DEFAULT_SETTINGS)
settings = 'devstack_docker'
# Generate static i18n JS files.
for system in ['lms', 'cms']:

View File

@@ -185,6 +185,8 @@ class TestI18nCompileJS(PaverTestCase):
"""
Test the "i18n_compilejs" task.
"""
Env.TEST_SETTINGS = 'devstack_docker'
self.reset_task_messages()
os.environ['NO_PREREQ_INSTALL'] = "true"
call_task('pavelib.i18n.i18n_compilejs', options={"settings": Env.TEST_SETTINGS})