Refactor GetBrewery to remove caching logic (#101)

Co-authored-by: Alex Justesen <1144087+alexjustesen@users.noreply.github.com>

Author: alexjustesen

app/Http/Controllers/Api/V1/GetBrewery.php

@@ -8,7 +8,6 @@
 use Illuminate\Http\JsonResponse;
 use Illuminate\Http\Request;
 use Illuminate\Http\Response;
-use Illuminate\Support\Facades\Cache;
 
 class GetBrewery extends Controller
 {
@@ -17,9 +16,7 @@ class GetBrewery extends Controller
      */
     public function __invoke(Request $request, string $id): JsonResponse
     {
-        $brewery = Cache::remember('brewery_'.$id, config('platform.cache_control_max_age'), function () use ($id) {
-            return new BreweryResource(Brewery::findOrFail($id));
-        });
+        $brewery = new BreweryResource(Brewery::findOrFail($id));
 
         return response()
             ->json(

tests/Feature/Api/V1/GetBreweryTest.php

@@ -39,7 +39,7 @@
     $response = $this->getJson("/v1/breweries/{$brewery->id}");
     $hasCache = Cache::has('brewery_'.$brewery->id);
     expect($hasCache)->toBeTrue();
-});
+})->skip(message: 'Caching was disabled for testing.');
 
 test('response has correct json api structure', function () {
     $brewery = createBrewery();