Use InvalidArgumentTypeException for index option validation

Author: jmikola

src/Model/IndexInput.php

@@ -4,7 +4,7 @@
 
 use MongoDB\BSON\Serializable;
 use MongoDB\Exception\InvalidArgumentException;
-use MongoDB\Exception\UnexpectedTypeException;
+use MongoDB\Exception\InvalidArgumentTypeException;
 
 /**
  * Index input model class.
@@ -32,12 +32,12 @@ public function __construct(array $index)
         }
 
         if ( ! is_array($index['key']) && ! is_object($index['key'])) {
-            throw new UnexpectedTypeException($index['key'], 'array or object');
+            throw new InvalidArgumentTypeException('"key" option', $index['key'], 'array or object');
         }
 
-        foreach ($index['key'] as $order) {
+        foreach ($index['key'] as $fieldName => $order) {
             if ( ! is_int($order) && ! is_float($order) && ! is_string($order)) {
-                throw new UnexpectedTypeException($order, 'numeric or string');
+                throw new InvalidArgumentTypeException(sprintf('order value for "%s" field within "key" option', $fieldName), $order, 'numeric or string');
             }
         }
 
@@ -46,15 +46,15 @@ public function __construct(array $index)
         }
 
         if ( ! is_string($index['ns'])) {
-            throw new UnexpectedTypeException($index['ns'], 'string');
+            throw new InvalidArgumentTypeException('"ns" option', $index['ns'], 'string');
         }
 
         if ( ! isset($index['name'])) {
             $index['name'] = \MongoDB\generate_index_name($index['key']);
         }
 
         if ( ! is_string($index['name'])) {
-            throw new UnexpectedTypeException($index['name'], 'string');
+            throw new InvalidArgumentTypeException('"name" option', $index['name'], 'string');
         }
 
         $this->index = $index;

src/Operation/CreateIndexes.php

@@ -7,7 +7,7 @@
 use MongoDB\Driver\BulkWrite as Bulk;
 use MongoDB\Driver\WriteConcern;
 use MongoDB\Exception\InvalidArgumentException;
-use MongoDB\Exception\UnexpectedTypeException;
+use MongoDB\Exception\InvalidArgumentTypeException;
 use MongoDB\Model\IndexInput;
 
 /**
@@ -42,7 +42,7 @@ public function __construct($databaseName, $collectionName, array $indexes)
 
         foreach ($indexes as $index) {
             if ( ! is_array($index)) {
-                throw new UnexpectedTypeException($index, 'array');
+                throw new InvalidArgumentTypeException(sprintf('$index[%d]', $i), $index, 'array');
             }
 
             if ( ! isset($index['ns'])) {

tests/Model/IndexInputTest.php

@@ -4,6 +4,7 @@
 
 use MongoDB\Model\IndexInput;
 use MongoDB\Tests\TestCase;
+use stdClass;
 
 class IndexInputTest extends TestCase
 {
@@ -16,19 +17,25 @@ public function testConstructorShouldRequireKey()
     }
 
     /**
-     * @expectedException MongoDB\Exception\UnexpectedTypeException
+     * @expectedException MongoDB\Exception\InvalidArgumentTypeException
      */
     public function testConstructorShouldRequireKeyToBeArrayOrObject()
     {
         new IndexInput(['key' => 'foo']);
     }
 
     /**
-     * @expectedException MongoDB\Exception\UnexpectedTypeException
+     * @expectedException MongoDB\Exception\InvalidArgumentTypeException
+     * @dataProvider provideInvalidFieldOrderValues
      */
-    public function testConstructorShouldRequireKeyOrderToBeScalar()
+    public function testConstructorShouldRequireKeyFieldOrderToBeNumericOrString($order)
     {
-        new IndexInput(['key' => ['x' => []]]);
+        new IndexInput(['key' => ['x' => $order]]);
+    }
+
+    public function provideInvalidFieldOrderValues()
+    {
+        return $this->wrapValuesForDataProvider([true, [], new stdClass]);
     }
 
     /**
@@ -40,15 +47,15 @@ public function testConstructorShouldRequireNamespace()
     }
 
     /**
-     * @expectedException MongoDB\Exception\UnexpectedTypeException
+     * @expectedException MongoDB\Exception\InvalidArgumentTypeException
      */
     public function testConstructorShouldRequireNamespaceToBeString()
     {
         new IndexInput(['key' => ['x' => 1], 'ns' => 1]);
     }
 
     /**
-     * @expectedException MongoDB\Exception\UnexpectedTypeException
+     * @expectedException MongoDB\Exception\InvalidArgumentTypeException
      */
     public function testConstructorShouldRequireNameToBeString()
     {

tests/Operation/CreateIndexesTest.php

@@ -8,6 +8,7 @@ class CreateIndexesTest extends TestCase
 {
     /**
      * @expectedException MongoDB\Exception\InvalidArgumentException
+     * @expectedExceptionMessage $indexes is empty
      */
     public function testCreateIndexesRequiresAtLeastOneIndex()
     {