improve validateOptions

Author: bshaffer

Core/src/ApiHelperTrait.php

@@ -272,25 +272,28 @@ private function splitOptionalArgs(array $input, array $extraAllowedKeys = []):
      * the CallOptions classname. Parameters are split and returned in the order
      * that the options types are provided.
      */
-    private function validateOptions(array $options, array|string ...$optionTypes): array
+    private function validateOptions(array $options, array|Message|string ...$optionTypes): array
     {
         $splitOptions = [];
         foreach ($optionTypes as $optionType) {
             if (is_array($optionType)) {
                 $splitOptions[] = $this->pluckArray($optionType, $options);
-            } elseif (is_string($optionType)) {
-                if (is_subclass_of($optionType, Message::class)) {
+            } else {
+                if ($optionType === CallOptions::class) {
+                    $callOptionKeys = array_keys((new CallOptions([]))->toArray());
+                    $splitOptions[] = $this->pluckArray($callOptionKeys, $options);
+                } else {
                     $messageKeys = array_map(
                         fn ($method) => lcfirst(substr($method, 3)),
                         array_filter(
                             get_class_methods($optionType),
                             fn ($m) => 0 === strpos($m, 'get')
                         )
                     );
-                    $splitOptions[] = $this->pluckArray($messageKeys, $options);
-                } elseif ($optionType === CallOptions::class) {
-                    $callOptionKeys = array_keys((new CallOptions([]))->toArray());
-                    $splitOptions[] = $this->pluckArray($callOptionKeys, $options);
+                    $messageOptions = $this->pluckArray($messageKeys, $options);
+                    $splitOptions[] = $optionType instanceof Message
+                        ? $this->serializer->decodeMessage($optionType, $messageOptions)
+                        : $messageOptions;
                 }
             }
         }

Core/tests/Unit/ApiHelperTraitTest.php

@@ -18,10 +18,11 @@
 namespace Google\Cloud\Core\Tests\Unit;
 
 use Google\ApiCore\Options\CallOptions;
+use Google\ApiCore\Serializer;
+use Google\ApiCore\Testing\MockRequest;
 use Google\Cloud\Core\Duration;
 use Google\Cloud\Core\Testing\GrpcTestTrait;
 use Google\Cloud\Core\Tests\Unit\Stubs\ApiHelpersTraitImpl;
-use Google\Cloud\Core\Tests\Unit\Stubs\TestMessage;
 use PHPUnit\Framework\TestCase;
 use Prophecy\PhpUnit\ProphecyTrait;
 
@@ -38,6 +39,7 @@ class ApiHelperTraitTest extends TestCase
     public function setUp(): void
     {
         $this->implementation = new ApiHelpersTraitImpl();
+        $this->implementation->serializer = new Serializer();
     }
 
     public function testFormatsTimestamp()
@@ -294,18 +296,18 @@ public function validateOptionsProvider()
             ],
             [
                 [
-                    'baz' => 'bat',
+                    'pageToken' => 'bat',
                     'qux' => 'quux',
                     'timeoutMillis' => 123,
                 ],
                 [
                     CallOptions::class,
-                    TestMessage::class,
+                    MockRequest::class,
                     ['qux'],
                 ],
                 [
                     ['timeoutMillis' => 123],
-                    ['baz' => 'bat'],
+                    ['pageToken' => 'bat'],
                     ['qux' => 'quux'],
                 ]
             ],
@@ -315,7 +317,7 @@ public function validateOptionsProvider()
                 ],
                 [
                     ['baz'],
-                    TestMessage::class,
+                    MockRequest::class,
                     CallOptions::class,
                 ],
                 [
@@ -324,6 +326,20 @@ public function validateOptionsProvider()
                     [],
                 ]
             ],
+            [
+                [
+                    'baz' => 'bat',
+                    'pageToken' => 'foo1',
+                ],
+                [
+                    ['baz'],
+                    new MockRequest(),
+                ],
+                [
+                    ['baz' => 'bat'],
+                    (new MockRequest())->setPageToken('foo1'),
+                ]
+            ],
         ];
     }
 

Core/tests/Unit/Stubs/ApiHelpersTraitImpl.php

@@ -33,4 +33,6 @@ class ApiHelpersTraitImpl
         unpackValue as public;
         validateOptions as public;
     }
+
+    public $serializer;
 }