Read the config file contents as bytes, it's just for debugging anyway. #990

Author: nedbat

CHANGES.rst

@@ -28,6 +28,11 @@ Unreleased
   ``--precision`` option to control the number of decimal points displayed.
   Thanks, Teake Nutma.
 
+- TOML configuration files with non-ASCII characters would cause errors on
+  Windows (`issue 990`_).  This is now fixed.
+
+.. _issue 990: https://github.com/nedbat/coveragepy/issues/990
+
 
 .. _changes_51:
 

coverage/config.py

@@ -325,7 +325,7 @@ def from_file(self, filename, our_file):
 
         if used:
             self.config_file = os.path.abspath(filename)
-            with open(filename) as f:
+            with open(filename, "rb") as f:
                 self._config_contents = f.read()
 
         return used

tests/test_config.py

@@ -3,6 +3,7 @@
 # For details: https://github.com/nedbat/coveragepy/blob/master/NOTICE.txt
 
 """Test the config file handling for coverage.py"""
+
 from collections import OrderedDict
 
 import mock
@@ -62,6 +63,8 @@ def test_toml_config_file(self):
         # A .coveragerc file will be read into the configuration.
         self.make_file("pyproject.toml", """\
             # This is just a bogus toml file for testing.
+            [tool.somethingelse]
+            authors = ["Joe D'Ávila <[email protected]>"]
             [tool.coverage.run]
             concurrency = ["a", "b"]
             timid = true
@@ -70,20 +73,23 @@ def test_toml_config_file(self):
             [tool.coverage.report]
             precision = 3
             fail_under = 90.5
+            [tool.coverage.html]
+            title = "tabblo & «ταБЬℓσ»"
             [tool.coverage.plugins.a_plugin]
             hello = "world"
             """)
         cov = coverage.Coverage(config_file="pyproject.toml")
         self.assertTrue(cov.config.timid)
         self.assertFalse(cov.config.branch)
-        self.assertEqual(cov.config.concurrency, ["a", "b"])
-        self.assertEqual(cov.config.data_file, ".hello_kitty.data")
-        self.assertEqual(cov.config.plugins, ["plugins.a_plugin"])
+        self.assertEqual(cov.config.concurrency, [u"a", u"b"])
+        self.assertEqual(cov.config.data_file, u".hello_kitty.data")
+        self.assertEqual(cov.config.plugins, [u"plugins.a_plugin"])
         self.assertEqual(cov.config.precision, 3)
+        self.assertEqual(cov.config.html_title, u"tabblo & «ταБЬℓσ»")
         self.assertAlmostEqual(cov.config.fail_under, 90.5)
         self.assertEqual(
             cov.config.get_plugin_options("plugins.a_plugin"),
-            {'hello': 'world'}
+            {u"hello": u"world"}
         )
 
         # Test that our class doesn't reject integers when loading floats