Fix: Mark check() and coerce() as deprecated

jsonrainbow · Dec 30, 2017 · 3b34779 · 3b34779
1 parent dfdcb5b
commit 3b34779
Show file tree

Hide file tree

Showing 8 changed files with 17 additions and 13 deletions.
diff --git a/bin/validate-json b/bin/validate-json
@@ -217,7 +217,7 @@ if (isset($arOptions['--dump-schema'])) {
 
 try {
     $validator = new JsonSchema\Validator();
-    $validator->check($data, $schema);
+    $validator->validate($data, $schema);
 
     if ($validator->isValid()) {
         if(isset($arOptions['--verbose'])) {

diff --git a/demo/demo.php b/demo/demo.php
@@ -6,7 +6,7 @@
 
 // Validate
 $validator = new JsonSchema\Validator();
-$validator->check($data, (object) array('$ref' => 'file://' . realpath('schema.json')));
+$validator->validate($data, (object) array('$ref' => 'file://' . realpath('schema.json')));
 
 if ($validator->isValid()) {
     echo "The supplied JSON validates against the schema.\n";

diff --git a/src/JsonSchema/Validator.php b/src/JsonSchema/Validator.php
@@ -74,6 +74,8 @@ public function validate(&$value, $schema = null, $checkMode = null)
 
     /**
      * Alias to validate(), to maintain backwards-compatibility with the previous API
+     *
+     * @deprecated
      */
     public function check($value, $schema)
     {
@@ -82,6 +84,8 @@ public function check($value, $schema)
 
     /**
      * Alias to validate(), to maintain backwards-compatibility with the previous API
+     *
+     * @deprecated
      */
     public function coerce(&$value, $schema)
     {

diff --git a/tests/Constraints/LongArraysTest.php b/tests/Constraints/LongArraysTest.php
@@ -41,7 +41,7 @@ public function testLongStringArray()
 
         $validator = new Validator(new Factory($schemaStorage));
         $checkValue = json_decode($input);
-        $validator->check($checkValue, $schema);
+        $validator->validate($checkValue, $schema);
         $this->assertTrue($validator->isValid(), print_r($validator->getErrors(), true));
     }
 
@@ -69,7 +69,7 @@ public function testLongNumberArray()
 
         $validator = new Validator(new Factory($schemaStorage));
         $checkValue = json_decode($input);
-        $validator->check($checkValue, $schema);
+        $validator->validate($checkValue, $schema);
         $this->assertTrue($validator->isValid(), print_r($validator->getErrors(), true));
     }
 
@@ -97,7 +97,7 @@ public function testLongIntegerArray()
 
         $validator = new Validator(new Factory($schemaStorage));
         $checkValue = json_decode($input);
-        $validator->check($checkValue, $schema);
+        $validator->validate($checkValue, $schema);
         $this->assertTrue($validator->isValid(), print_r($validator->getErrors(), true));
     }
 

diff --git a/tests/Constraints/PointerTest.php b/tests/Constraints/PointerTest.php
@@ -83,7 +83,7 @@ public function testVariousPointers()
 
         $validator = new Validator();
         $checkValue = json_decode(json_encode($value));
-        $validator->check($checkValue, json_decode(json_encode($schema)));
+        $validator->validate($checkValue, json_decode(json_encode($schema)));
 
         $this->assertEquals(
             array(

diff --git a/tests/Constraints/SelfDefinedSchemaTest.php b/tests/Constraints/SelfDefinedSchemaTest.php
@@ -71,6 +71,6 @@ public function testInvalidArgumentException()
     {
         $v = new Validator();
         $this->setExpectedException('\JsonSchema\Exception\InvalidArgumentException');
-        $v->check(json_decode('{}'), json_decode(''));
+        $v->validate(json_decode('{}'), json_decode(''));
     }
 }
diff --git a/tests/Uri/UriRetrieverTest.php b/tests/Uri/UriRetrieverTest.php
@@ -55,7 +55,7 @@ public function testChildExtendsParentValidTest($childSchema, $parentSchema)
         $decodedJson = json_decode($json);
         $decodedJsonSchema = json_decode($childSchema);
 
-        $this->validator->check($decodedJson, $decodedJsonSchema);
+        $this->validator->validate($decodedJson, $decodedJsonSchema);
         $this->assertTrue($this->validator->isValid());
     }
 
@@ -70,7 +70,7 @@ public function testChildExtendsParentInvalidChildTest($childSchema, $parentSche
         $decodedJson = json_decode($json);
         $decodedJsonSchema = json_decode($childSchema);
 
-        $this->validator->check($decodedJson, $decodedJsonSchema);
+        $this->validator->validate($decodedJson, $decodedJsonSchema);
         $this->assertFalse($this->validator->isValid());
     }
 
@@ -85,7 +85,7 @@ public function testChildExtendsParentInvalidParentTest($childSchema, $parentSch
         $decodedJson = json_decode($json);
         $decodedJsonSchema = json_decode($childSchema);
 
-        $this->validator->check($decodedJson, $decodedJsonSchema);
+        $this->validator->validate($decodedJson, $decodedJsonSchema);
         $this->assertFalse($this->validator->isValid());
     }
 
@@ -101,7 +101,7 @@ public function testResolveRelativeUri($childSchema, $parentSchema)
         $decodedJson = json_decode($json);
         $decodedJsonSchema = json_decode($childSchema);
 
-        $this->validator->check($decodedJson, $decodedJsonSchema);
+        $this->validator->validate($decodedJson, $decodedJsonSchema);
         $this->assertTrue($this->validator->isValid());
     }
 

diff --git a/tests/ValidatorTest.php b/tests/ValidatorTest.php
@@ -36,7 +36,7 @@ public function testBadAssocSchemaInput()
         $validator->validate($data, $schema);
     }
 
-    public function testCheck()
+    public function testDeprecatedCheckStillValidates()
     {
         $schema = json_decode('{"type":"string"}');
         $data = json_decode('42');
@@ -47,7 +47,7 @@ public function testCheck()
         $this->assertFalse($validator->isValid(), 'Validation succeeded, but should have failed.');
     }
 
-    public function testCoerce()
+    public function testDeprecatedCoerceStillValidates()
     {
         $schema = json_decode('{"type":"integer"}');
         $data = json_decode('"42"');