Type: mergeDefaults() are disabled by default (BC BREAK) [Closes #28, C…

…loses #31]

readme.md

@@ -169,7 +169,7 @@ The parameter can also be a schema, so we can write:
 Expect::arrayOf(Expect::bool())
 ```
 
-The default value is an empty array. If you specify default value, it will be merged with the passed data. This can be disabled using `mergeDefaults(false)`.
+The default value is an empty array. If you specify default value and call `mergeDefaults()`, it will be merged with the passed data.
 
 
 Enumeration: anyOf()

src/Schema/Elements/Type.php

@@ -37,7 +37,7 @@ final class Type implements Schema
 	private $pattern;
 
 	/** @var bool */
-	private $merge = true;
+	private $merge = false;
 
 
 	public function __construct(string $type)

tests/Schema/Expect.array.phpt

@@ -42,7 +42,7 @@ test('not merging', function () {
 		'key2' => 'val2',
 		'val3',
 		'arr' => ['item'],
-	])->mergeDefaults(false);
+	]);
 
 	Assert::same([], (new Processor)->process($schema, []));
 
@@ -59,7 +59,7 @@ test('merging', function () {
 		'key2' => 'val2',
 		'val3',
 		'arr' => ['item'],
-	]);
+	])->mergeDefaults(true);
 
 	Assert::same([
 		'key1' => 'val1',
@@ -136,7 +136,7 @@ test('merging & other items validation', function () {
 		'key1' => 'val1',
 		'key2' => 'val2',
 		'val3',
-	])->items('string');
+	])->mergeDefaults(true)->items('string');
 
 	Assert::same([
 		'key1' => 'val1',
@@ -204,11 +204,9 @@ test('merging & other items validation', function () {
 
 
 test('items() & scalar', function () {
-	$schema = Expect::array([
-		'a' => 'defval',
-	])->items('string');
+	$schema = Expect::array()->items('string');
 
-	Assert::same(['a' => 'defval'], (new Processor)->process($schema, []));
+	Assert::same([], (new Processor)->process($schema, []));
 
 	checkValidationErrors(function () use ($schema) {
 		(new Processor)->process($schema, [1, 2, 3]);
@@ -232,16 +230,14 @@ test('items() & scalar', function () {
 		(new Processor)->process($schema, ['b' => null]);
 	}, ["The item 'b' expects to be string, null given."]);
 
-	Assert::same(['a' => 'defval', 'b' => 'val'], (new Processor)->process($schema, ['b' => 'val']));
+	Assert::same(['b' => 'val'], (new Processor)->process($schema, ['b' => 'val']));
 });
 
 
 test('items() & structure', function () {
-	$schema = Expect::array([
-		'a' => 'defval',
-	])->items(Expect::structure(['k' => Expect::string()]));
+	$schema = Expect::array([])->items(Expect::structure(['k' => Expect::string()]));
 
-	Assert::same(['a' => 'defval'], (new Processor)->process($schema, []));
+	Assert::same([], (new Processor)->process($schema, []));
 
 	checkValidationErrors(function () use ($schema) {
 		(new Processor)->process($schema, ['a' => 'val']);
@@ -264,7 +260,7 @@ test('items() & structure', function () {
 	}, ["Unexpected item 'b › a', did you mean 'k'?"]);
 
 	Assert::equal(
-		['a' => 'defval', 'b' => (object) ['k' => 'val']],
+		['b' => (object) ['k' => 'val']],
 		(new Processor)->process($schema, ['b' => ['k' => 'val']])
 	);
 });

tests/Schema/Expect.list.phpt

@@ -38,7 +38,7 @@ test('without default value', function () {
 
 
 test('not merging', function () {
-	$schema = Expect::list([1, 2, 3])->mergeDefaults(false);
+	$schema = Expect::list([1, 2, 3]);
 
 	Assert::same([], (new Processor)->process($schema, []));
 
@@ -49,7 +49,7 @@ test('not merging', function () {
 
 
 test('merging', function () {
-	$schema = Expect::list([1, 2, 3]);
+	$schema = Expect::list([1, 2, 3])->mergeDefaults(true);
 
 	Assert::same([1, 2, 3], (new Processor)->process($schema, []));
 
@@ -60,7 +60,7 @@ test('merging', function () {
 
 
 test('merging & other items validation', function () {
-	$schema = Expect::list([1, 2, 3])->items('string');
+	$schema = Expect::list([1, 2, 3])->mergeDefaults(true)->items('string');
 
 	Assert::same([1, 2, 3], (new Processor)->process($schema, []));