Workaround for tests in Python 3.4

related bug: http://bugs.python.org/issue21750
python-jsonschema · Jun 20, 2014 · be5097d · be5097d
1 parent e8ba6a6
commit be5097d
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 19 deletions.
diff --git a/jsonschema/cli.py b/jsonschema/cli.py
@@ -40,8 +40,8 @@ def _json_file(path):
     "-V", "--validator",
     type=_namedAnyWithDefault,
     help="the fully qualified object name of a validator to use, or, for "
-          "validators that are registered with jsonschema, simply the name "
-          "of the class.",
+         "validators that are registered with jsonschema, simply the name "
+         "of the class.",
 )
 parser.add_argument(
     "schema",

diff --git a/jsonschema/tests/test_cli.py b/jsonschema/tests/test_cli.py
@@ -18,14 +18,19 @@ def iter_errors(self, instance):
 
 
 class TestParser(unittest.TestCase):
-
     FakeValidator = fake_validator()
 
     def setUp(self):
-        self.open = mock.mock_open(read_data='{}')
-        patch = mock.patch.object(cli, "open", self.open, create=True)
-        patch.start()
-        self.addCleanup(patch.stop)
+        mock_open = mock.mock_open()
+        patch_open = mock.patch.object(cli, "open", mock_open, create=True)
+        patch_open.start()
+        self.addCleanup(patch_open.stop)
+
+        mock_json_load = mock.Mock()
+        mock_json_load.return_value = {}
+        patch_json_load = mock.patch("json.load")
+        patch_json_load.start()
+        self.addCleanup(patch_json_load.stop)
 
     def test_find_validator_by_fully_qualified_object_name(self):
         arguments = cli.parse_args(
@@ -54,10 +59,10 @@ def test_successful_validation(self):
         stdout, stderr = StringIO(), StringIO()
         exit_code = cli.run(
             {
-                "validator" : fake_validator(),
-                "schema" : {},
-                "instances" : [1],
-                "error_format" : "{error.message}",
+                "validator": fake_validator(),
+                "schema": {},
+                "instances": [1],
+                "error_format": "{error.message}",
             },
             stdout=stdout,
             stderr=stderr,
@@ -71,10 +76,10 @@ def test_unsuccessful_validation(self):
         stdout, stderr = StringIO(), StringIO()
         exit_code = cli.run(
             {
-                "validator" : fake_validator([error]),
-                "schema" : {},
-                "instances" : [1],
-                "error_format" : "{error.instance} - {error.message}",
+                "validator": fake_validator([error]),
+                "schema": {},
+                "instances": [1],
+                "error_format": "{error.instance} - {error.message}",
             },
             stdout=stdout,
             stderr=stderr,
@@ -92,10 +97,10 @@ def test_unsuccessful_validation_multiple_instances(self):
         stdout, stderr = StringIO(), StringIO()
         exit_code = cli.run(
             {
-                "validator" : fake_validator(first_errors, second_errors),
-                "schema" : {},
-                "instances" : [1, 2],
-                "error_format" : "{error.instance} - {error.message}\t",
+                "validator": fake_validator(first_errors, second_errors),
+                "schema": {},
+                "instances": [1, 2],
+                "error_format": "{error.instance} - {error.message}\t",
             },
             stdout=stdout,
             stderr=stderr,