Make sure rst files are read with UTF-8 encoding so as to avoid issues when locale is C (restricting the charset to ASCII)

astrofrog · astrofrog · commit 462dea47d340 · 2017-07-05T00:27:57.000+01:00
diff --git a/.travis.yml b/.travis.yml
@@ -33,6 +33,7 @@ env:
     - PYTHON_VERSION=3.5 SPHINX_VERSION=1.5
     - PYTHON_VERSION=3.6 SPHINX_VERSION=1.6 CONDA_CHANNELS="conda-forge"
     - PYTHON_VERSION=3.6 SPHINX_VERSION=dev CONDA_DEPENDENCIES="setuptools cython pytest-cov"
+    - PYTHON_VERSION=2.7 LOCALE=C
     - PYTHON_VERSION=3.6 LOCALE=C
   global:
     - LOCALE=default
diff --git a/sphinx_automodapi/automodsumm.py b/sphinx_automodapi/automodsumm.py
@@ -86,6 +86,7 @@ class members that are inherited from a base class. This value can be
 import inspect
 import os
 import re
+import io
 
 from sphinx.ext.autosummary import Autosummary
 from sphinx.ext.inheritance_diagram import InheritanceDiagram
@@ -305,7 +306,7 @@ def automodsumm_to_autosummary_lines(fn, app):
 
     fullfn = os.path.join(app.builder.env.srcdir, fn)
 
-    with open(fullfn) as fr:
+    with io.open(fullfn, encoding='utf8') as fr:
         # Note: we use __name__ here instead of just writing the module name in
         #       case this extension is bundled into another package
         from . import automodapi
diff --git a/sphinx_automodapi/tests/test_cases.py b/sphinx_automodapi/tests/test_cases.py
@@ -4,6 +4,7 @@
 # We store different cases in the cases sub-directory of the tests directory
 
 import os
+import io
 import sys
 import glob
 import shutil
@@ -109,6 +110,6 @@ def test_run_full_case(tmpdir, case_dir):
             path_relative = os.path.relpath(path_reference, output_dir)
             path_actual = os.path.join(docs_dir, path_relative)
             assert os.path.exists(path_actual)
-            actual = open(path_actual).read()
-            reference = open(path_reference).read()
+            actual = io.open(path_actual, encoding='utf8').read()
+            reference = io.open(path_reference, encoding='utf8').read()
             assert actual.strip() == reference.strip()
