Make some final changes to tidy up the code

Author: ebebbington

src/Browserless/Client.php

@@ -21,6 +21,10 @@ trait Client
      */
     private $apiUrl = 'https://chrome.browserless.io';
 
+    public const EUROPE_REGION_URL = 'production-lon.browserless.io';
+
+    public const US_REGION_URL = 'production-sfo.browserless.io';
+
     /**
      * @param string $apiKey api key from browserless.io
      * @param \GuzzleHttp\Client $client custom Guzzle client

src/Browserless/Screenshot.php

@@ -19,10 +19,12 @@ class Screenshot
     public function render(string $url, array $options = [])
     {
         $options = array_merge([
-            'quality' => 75,
             'type' => 'jpeg',
             'fullPage' => false,
         ], $options);
+        if ($options['type'] === 'jpeg' && ! isset($options['quality'])) {
+            $options['quality'] = 75;
+        }
 
         return $this->request(
             endpoint: '/screenshot',

test/Browserless/ScreenshotTest.php

@@ -11,13 +11,6 @@
 
 class ScreenshotTest extends TestCase
 {
-    private function getMockedClient()
-    {
-        return $this->getMockBuilder(Chrome::class)
-            ->setMethods(['post'])
-            ->getMock();
-    }
-
     public function test_render()
     {
         $client = $this->getMockedClient();

test/BrowserlessTest.php

@@ -17,13 +17,6 @@
 
 class BrowserlessTest extends TestCase
 {
-    private function getMockedClient()
-    {
-        return $this->getMockBuilder(Chrome::class)
-            ->setMethods(['post'])
-            ->getMock();
-    }
-
     public function test_rotation()
     {
         $client = $this->getMockedClient();

test/TestCase.php

@@ -3,11 +3,19 @@
 namespace SynergiTech\ChromePDF\Test;
 
 use PHPUnit\Framework\TestCase as PHPUnitTestCase;
+use SynergiTech\ChromePDF\Chrome;
 
 class TestCase extends PHPUnitTestCase
 {
     public function hasKeyValue($key, $constraint = null)
     {
         return new Constraint\ArrayHasKeyValue($key, $constraint);
     }
+
+    protected function getMockedClient()
+    {
+        return $this->getMockBuilder(Chrome::class)
+            ->setMethods(['post'])
+            ->getMock();
+    }
 }