public-mirrors
/
composer
mirror of https://github.com/composer/composer
1
0
Fork 0
Code

Revert "Merge remote-tracking branch 'BoShurik/schema'" 

This reverts commit 89c3045e2b, reversing
changes made to 991985792d.
pull/9912/head
Guilliam Xavier 2021-05-24 10:15:16 +02:00
parent 41c66b1a2d
commit 9e2cb30dfb
3 changed files with 2 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
res/composer-schema.json
View File

@ -3,26 +3,7 @@
"title": "Package",
"type": "object",
"additionalProperties": false,
"oneOf": [
{
"properties": {
"type": {
"not": {
"enum": ["project"]
}
}
},
"required": ["name", "description"]
},
{
"properties": {
"type": {
"enum": ["project"]
}
},
"required": ["type"]
}
],
"required": [ "name", "description" ],
"properties": {
"name": {
"type": "string",

2
src/Composer/Json/JsonFile.php
View File

@ -198,7 +198,7 @@ class JsonFile
if ($schema === self::LAX_SCHEMA) {
$schemaData->additionalProperties = true;
$schemaData->oneOf = null;
$schemaData->required = array();
}
$validator = new Validator();

20
tests/Composer/Test/Json/ComposerSchemaTest.php
View File

@ -41,38 +41,18 @@ class ComposerSchemaTest extends TestCase
{
$json = '{ }';
$result = $this->check($json);
$this->assertContains(array('property' => 'type', 'message' => 'The property type is required', 'constraint' => 'required'), $result);
$this->assertContains(array('property' => 'name', 'message' => 'The property name is required', 'constraint' => 'required'), $result);
$this->assertContains(array('property' => 'description', 'message' => 'The property description is required', 'constraint' => 'required'), $result);
$this->assertContains(array('property' => '', 'message' => 'Failed to match exactly one schema', 'constraint' => 'oneOf'), $result);
$json = '{ "name": "vendor/package" }';
$this->assertEquals(array(
array('property' => 'type', 'message' => 'The property type is required', 'constraint' => 'required'),
array('property' => 'description', 'message' => 'The property description is required', 'constraint' => 'required'),
array('property' => '', 'message' => 'Failed to match exactly one schema', 'constraint' => 'oneOf'),
), $this->check($json));
$json = '{ "description": "generic description" }';
$this->assertEquals(array(
array('property' => 'type', 'message' => 'The property type is required', 'constraint' => 'required'),
array('property' => 'name', 'message' => 'The property name is required', 'constraint' => 'required'),
array('property' => '', 'message' => 'Failed to match exactly one schema', 'constraint' => 'oneOf'),
), $this->check($json));
$json = '{ "type": "library" }';
$this->assertEquals(array(
array('property' => 'type', 'message' => 'Does not have a value in the enumeration ["project"]', 'constraint' => 'enum', 'enum' => array('project')),
array('property' => 'name', 'message' => 'The property name is required', 'constraint' => 'required'),
array('property' => 'description', 'message' => 'The property description is required', 'constraint' => 'required'),
array('property' => '', 'message' => 'Failed to match exactly one schema', 'constraint' => 'oneOf'),
), $this->check($json));
$json = '{ "type": "project" }';
$this->assertTrue($this->check($json));
$json = '{ "name": "vendor/package", "description": "description" }';
$this->assertTrue($this->check($json));
}
public function testOptionalAbandonedProperty()