Andal.LND/edx-platform
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed all py3 tests with AssertionError: "Error: too few arguments"

Browse Source
This commit is contained in:
Ayub khan
2019-08-22 18:42:58 +05:00
parent 5dc63a9aab
commit f5d97260a5
8 changed files with 45 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
cms/djangoapps/contentstore/management/commands/tests/test_create_course.py
View File

@@ -23,7 +23,10 @@ class TestArgParsing(TestCase):
super(TestArgParsing, self).setUp()
def test_no_args(self):
errstring = "Error: too few arguments"
if six.PY2:
errstring = "Error: too few arguments"
else:
errstring = "Error: the following arguments are required: modulestore, user, org, number, run"
with self.assertRaisesRegexp(CommandError, errstring):
call_command('create_course')

6
cms/djangoapps/contentstore/management/commands/tests/test_delete_orphans.py
View File

@@ -21,7 +21,11 @@ class TestDeleteOrphan(TestOrphanBase):
"""
Test delete_orphans command with no arguments
"""
with self.assertRaisesRegexp(CommandError, 'Error: too few arguments'):
if six.PY2:
errstring = 'Error: too few arguments'
else:
errstring = 'Error: the following arguments are required: course_id'
with self.assertRaisesRegexp(CommandError, errstring):
call_command('delete_orphans')
@ddt.data(ModuleStoreEnum.Type.split, ModuleStoreEnum.Type.mongo)

5
cms/djangoapps/contentstore/management/commands/tests/test_export.py
View File

@@ -25,7 +25,10 @@ class TestArgParsingCourseExport(unittest.TestCase):
"""
Test export command with no arguments
"""
errstring = "Error: too few arguments"
if six.PY2:
errstring = "Error: too few arguments"
else:
errstring = "Error: the following arguments are required: course_id, output_path"
with self.assertRaisesRegexp(CommandError, errstring):
call_command('export')

6
cms/djangoapps/contentstore/management/commands/tests/test_force_publish.py
View File

@@ -29,7 +29,11 @@ class TestForcePublish(SharedModuleStoreTestCase):
"""
Test 'force_publish' command with no arguments
"""
errstring = "Error: too few arguments"
if six.PY2:
errstring = "Error: too few arguments"
else:
errstring = "Error: the following arguments are required: course_key"
with self.assertRaisesRegexp(CommandError, errstring):
call_command('force_publish')

11
cms/djangoapps/contentstore/management/commands/tests/test_git_export.py
View File

@@ -62,8 +62,15 @@ class TestGitExport(CourseTestCase):
with self.assertRaisesRegexp(CommandError, 'Error: unrecognized arguments:*'):
call_command('git_export', 'blah', 'blah', 'blah', stderr=StringIO())
with self.assertRaisesMessage(CommandError, 'Error: too few arguments'):
call_command('git_export', stderr=StringIO())
if six.PY2:
with self.assertRaisesMessage(CommandError, 'Error: too few arguments'):
call_command('git_export', stderr=StringIO())
else:
with self.assertRaisesMessage(
CommandError,
'Error: the following arguments are required: course_loc, git_url'
):
call_command('git_export', stderr=StringIO())
# Send bad url to get course not exported
with self.assertRaisesRegexp(CommandError, six.text_type(GitExportError.URL_BAD)):

7
cms/djangoapps/contentstore/management/commands/tests/test_migrate_to_split.py
View File

@@ -3,6 +3,8 @@ Unittests for migrating a course to split mongo
"""
from __future__ import absolute_import
import six
from django.core.management import CommandError, call_command
from django.test import TestCase
@@ -24,7 +26,10 @@ class TestArgParsing(TestCase):
"""
Test the arg length error
"""
errstring = "Error: too few arguments"
if six.PY2:
errstring = "Error: too few arguments"
else:
errstring = "Error: the following arguments are required: course_key, email"
with self.assertRaisesRegexp(CommandError, errstring):
call_command("migrate_to_split")