Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
979ee684
Commit
979ee684
authored
Jun 07, 2013
by
Ned Batchelder
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Add to the test, a check that the XML file truly has a non-ASCII character in it.
parent
4896e445
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
10 additions
and
0 deletions
+10
-0
common/lib/xmodule/xmodule/modulestore/tests/test_xml.py
+10
-0
No files found.
common/lib/xmodule/xmodule/modulestore/tests/test_xml.py
View file @
979ee684
import
os.path
from
xmodule.course_module
import
CourseDescriptor
from
xmodule.modulestore.xml
import
XMLModuleStore
from
nose.tools
import
assert_raises
from
.test_modulestore
import
check_path_to_location
from
.
import
DATA_DIR
...
...
@@ -19,6 +23,12 @@ class TestXMLModuleStore(object):
# edX/full/6.002_Spring_2012 has non-ASCII chars, and during
# uniquification of names, would raise a UnicodeError. It no longer does.
# Ensure that there really is a non-ASCII character in the course.
with
open
(
os
.
path
.
join
(
DATA_DIR
,
"full/sequential/Administrivia_and_Circuit_Elements.xml"
))
as
xmlf
:
xml
=
xmlf
.
read
()
with
assert_raises
(
UnicodeDecodeError
):
xml
.
decode
(
'ascii'
)
# Load the course, but don't make error modules. This will succeed,
# but will record the errors.
modulestore
=
XMLModuleStore
(
DATA_DIR
,
course_dirs
=
[
'full'
],
load_error_modules
=
False
)
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment