Throw exception for invalid headers (#23)

Author: Attia-Ahmed

Diff for: src/Models/Request.php

@@ -50,6 +50,12 @@ public static function create(array $data): self
         if (!empty($data['headers'])) {
             $request->headers = collect($data['headers'])
                 ->mapWithKeys(function ($header) {
+                    if (!str_contains($header, ':')) {
+                        throw new InvalidArgumentException(
+                            sprintf('The "%s" header must be a key/value pair separated by ":".', $header)
+                        );
+                    }
+
                     [$key, $value] = explode(':', $header, 2);
 
                     return [trim($key) => self::convertDataType(trim($value))];

Diff for: tests/Feature/Console/Commands/CurlCommandTest.php

@@ -29,6 +29,14 @@ public function test_it_throw_exception_when_for_invalid_url()
         Artisan::call('shift:curl -X GET "https://{domain:port}/api/{id}/"');
     }
 
+    public function test_it_throw_exception_when_for_invalid_headers()
+    {
+        $this->expectException(InvalidArgumentException::class);
+        $this->expectExceptionMessage('The "foo" header must be a key/value pair separated by ":".');
+
+        Artisan::call("shift:curl https://example.com --header 'foo'");
+    }
+
     public function curlCommandFixtures()
     {
         return [