Fix failing tests

VertisanPRO · May 30, 2022 · 9300e11 · 9300e11
1 parent a5521ec
commit 9300e11
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/app/Http/Controllers/Api/Client/ApiKeyController.php b/app/Http/Controllers/Api/Client/ApiKeyController.php
@@ -60,13 +60,14 @@ public function store(StoreApiKeyRequest $request)
      */
     public function delete(ClientApiRequest $request, string $identifier)
     {
+        /** @var \Pterodactyl\Models\ApiKey $key */
         $key = $request->user()->apiKeys()
             ->where('key_type', ApiKey::TYPE_ACCOUNT)
             ->where('identifier', $identifier)
-            ->first();
+            ->firstOrFail();
 
         Activity::event('user:api-key.delete')
-            ->property('identifer', $key->identifer)
+            ->property('identifer', $key->identifier)
             ->log();
 
         $key->delete();

diff --git a/tests/Integration/Api/Client/SSHKeyControllerTest.php b/tests/Integration/Api/Client/SSHKeyControllerTest.php
@@ -46,8 +46,8 @@ public function testSSHKeyCanBeDeleted()
         $this->assertSoftDeleted($key);
         $this->assertNotSoftDeleted($key2);
 
-        $this->deleteJson($this->link($key))->assertNoContent();
-        $this->deleteJson($this->link($key2))->assertNoContent();
+        $this->deleteJson($this->link($key))->assertNotFound();
+        $this->deleteJson($this->link($key2))->assertNotFound();
 
         $this->assertNotSoftDeleted($key2);
     }

diff --git a/tests/Integration/IntegrationTestCase.php b/tests/Integration/IntegrationTestCase.php
@@ -13,6 +13,10 @@ abstract class IntegrationTestCase extends TestCase
 
     protected array $connectionsToTransact = ['mysql'];
 
+    protected $defaultHeaders = [
+        'Accept' => 'application/json',
+    ];
+
     /**
      * Return an ISO-8601 formatted timestamp to use in the API response.
      */