Merge pull request kitetail#60 from kitetail/add-configure-method-to-…

…pending-zttp-response Add ability to set configuration all at once
pixelfed · Mar 11, 2018 · 3875188 · 3875188
2 parents 9c5508e + c95d154
commit 3875188
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/src/Zttp.php b/src/Zttp.php
@@ -26,6 +26,13 @@ static function new(...$args)
         return new self(...$args);
     }
 
+    function withOptions($options)
+    {
+        return tap($this, function ($request) use ($options) {
+            return $this->options = array_merge_recursive($this->options, $options);
+        });
+    }
+
     function withoutRedirecting()
     {
         return tap($this, function ($request) {
@@ -174,6 +181,7 @@ function buildHandlerStack()
             $stack->push($this->buildBeforeSendingHandler());
         });
     }
+
     function buildBeforeSendingHandler()
     {
         return function ($handler) {

diff --git a/tests/ZttpTest.php b/tests/ZttpTest.php
@@ -63,6 +63,22 @@ function query_parameters_in_urls_can_be_combined_with_array_parameters()
         ], $response->json());
     }
 
+    /** @test */
+    function options_can_be_set_all_at_once()
+    {
+        $response = Zttp::withOptions([
+            'headers' => [
+                'accept' => ['text/xml'],
+            ]
+        ])->get($this->url('/get'));
+
+        $this->assertArraySubset([
+            'headers' => [
+                'accept' => ['text/xml'],
+            ]
+        ], $response->json());
+    }
+
     /** @test */
     function post_content_is_json_by_default()
     {