Cleanup, throw exceptions

Author: aarondfrancis

config/torchlight.php

@@ -3,7 +3,7 @@
 return [
     // The Torchlight client caches highlighted code blocks. Here
     // you can define which cache driver you'd like to use.
-    'cache' => env('TORCHLIGHT_CACHE_DRIVER', 'default'),
+    'cache' => env('TORCHLIGHT_CACHE_DRIVER'),
 
     // Which theme you want to use. You can find all of the themes at
     // https://torchlight.dev/themes, or you can provide your own.
@@ -14,4 +14,4 @@
 
     // If you want to register the blade directives, set this to true.
     'blade_directives' => true,
-];
+];

src/Client.php

@@ -5,6 +5,8 @@
 
 namespace Hammerstone\Torchlight;
 
+use Hammerstone\Torchlight\Exceptions\ConfigurationException;
+use Hammerstone\Torchlight\Exceptions\RequestException;
 use Illuminate\Http\Client\Response;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Collection;
@@ -40,12 +42,14 @@ protected function request(Collection $blocks)
         $host = config('torchlight.host', 'https://api.torchlight.dev');
 
         $response = Http::timeout(5)
-            ->withToken(config('torchlight.token'))
+            ->withToken($this->getToken())
             ->post($host . '/highlight', [
                 'blocks' => $this->blocksAsRequestParam($blocks)->values(),
             ])
             ->json();
 
+        $this->potentiallyThrowRequestException($response);
+
         $response = collect(Arr::get($response, 'blocks', []))->keyBy('id');
 
         $blocks->each(function (Block $block) use ($response) {
@@ -60,6 +64,31 @@ protected function request(Collection $blocks)
         return $blocks;
     }
 
+    protected function getToken()
+    {
+        $token = config('torchlight.token');
+
+        if (!$token) {
+            $this->throwUnlessProduction(
+                new ConfigurationException('No Torchlight token configured.')
+            );
+        }
+
+        return $token;
+    }
+
+    protected function potentiallyThrowRequestException($response)
+    {
+        if ($error = Arr::get($response, 'error')) {
+            $this->throwUnlessProduction(new RequestException($error));
+        }
+    }
+
+    protected function throwUnlessProduction($exception)
+    {
+        throw_unless(app()->environment('production'), $exception);
+    }
+
     public function cache()
     {
         $store = config('torchlight.cache');

src/Exceptions/ConfigurationException.php

@@ -0,0 +1,12 @@
+<?php
+/**
+ * @author Aaron Francis <aarondfrancis@gmail.com|https://twitter.com/aarondfrancis>
+ */
+
+namespace Hammerstone\Torchlight\Exceptions;
+
+
+class ConfigurationException extends TorchlightException
+{
+
+}

src/Exceptions/RequestException.php

@@ -0,0 +1,12 @@
+<?php
+/**
+ * @author Aaron Francis <aarondfrancis@gmail.com|https://twitter.com/aarondfrancis>
+ */
+
+namespace Hammerstone\Torchlight\Exceptions;
+
+
+class RequestException extends TorchlightException
+{
+
+}

src/Exceptions/TorchlightException.php

@@ -0,0 +1,11 @@
+<?php
+/**
+ * @author Aaron Francis <aaron@hammerstone.dev|https://twitter.com/aarondfrancis>
+ */
+
+namespace Hammerstone\Torchlight\Exceptions;
+
+class TorchlightException extends \Exception
+{
+
+}