ReviewOS

also looking at this

meemalabs/meema-client-php

add test case

#37
Merged glennmichael123 wants to merge test-environment into main
8 files +106 -88
src/Client.phpmodified+6-3
Changes to src/Client.php
@@ -26,13 +26,16 @@ class Client
2626 *
2727 * @param string $accessKey
2828 */
29 public function __construct($accessKey)
29 public function __construct($accessKey, $baseUrl = null)
3030 {
3131 $this->initializeEnv();
3232
3333 $this->accessKey = $accessKey;
34
35 $url = $baseUrl ?? $_ENV['BASE_URL'];
36
3437 $this->client = new GuzzleClient([
35 'base_uri' => $_ENV['BASE_URL'] ?? 'https://api.mee.ma',
38 'base_uri' => $url ?? 'https://api.mee.ma',
3639 ]);
3740 }
3841
@@ -56,7 +59,7 @@ class Client
5659 public function request($method, $path, $data = null)
5760 {
5861 $content = $this->client->request($method, $path, [
59 'json' => $data,
62 'query' => $data,
6063 'headers' => [
6164 'Content-Type' => 'application/json',
6265 'Authorization' => "Bearer {$this->accessKey}",