From 14f9e1ad43e3725acba92bef8befb413eaac430f Mon Sep 17 00:00:00 2001 From: Dane Everitt Date: Sun, 14 Jan 2018 12:11:04 -0600 Subject: [PATCH] More permission removal cleanup --- app/Models/ApiKey.php | 19 +++--- .../Services/Api/PermissionServiceTest.php | 62 ------------------- 2 files changed, 9 insertions(+), 72 deletions(-) delete mode 100644 tests/Unit/Services/Api/PermissionServiceTest.php diff --git a/app/Models/ApiKey.php b/app/Models/ApiKey.php index 3aa7edcea..ed53a3181 100644 --- a/app/Models/ApiKey.php +++ b/app/Models/ApiKey.php @@ -14,6 +14,15 @@ class ApiKey extends Model implements CleansAttributes, ValidableContract { use Eloquence, Validable; + /** + * Different API keys that can exist on the system. + */ + const TYPE_NONE = 0; + const TYPE_USER = 1; + const TYPE_APPLICATION = 2; + const TYPE_DAEMON_USER = 3; + const TYPE_DAEMON_APPLICATION = 4; + /** * The length of API key identifiers. */ @@ -124,14 +133,4 @@ class ApiKey extends Model implements CleansAttributes, ValidableContract { return app()->make(Encrypter::class)->decrypt($this->token); } - - /** - * Gets the permissions associated with a key. - * - * @return \Illuminate\Database\Eloquent\Relations\HasMany - */ - public function permissions() - { - return $this->hasMany(APIPermission::class, 'key_id'); - } } diff --git a/tests/Unit/Services/Api/PermissionServiceTest.php b/tests/Unit/Services/Api/PermissionServiceTest.php deleted file mode 100644 index 384acec72..000000000 --- a/tests/Unit/Services/Api/PermissionServiceTest.php +++ /dev/null @@ -1,62 +0,0 @@ -. - * - * This software is licensed under the terms of the MIT license. - * https://opensource.org/licenses/MIT - */ - -namespace Tests\Unit\Services; - -use Mockery as m; -use Tests\TestCase; -use Pterodactyl\Models\APIPermission; -use Pterodactyl\Services\Api\PermissionService; -use Pterodactyl\Contracts\Repository\ApiPermissionRepositoryInterface; - -class PermissionServiceTest extends TestCase -{ - /** - * @var \Pterodactyl\Contracts\Repository\ApiPermissionRepositoryInterface - */ - protected $repository; - - /** - * @var \Pterodactyl\Services\Api\PermissionService - */ - protected $service; - - /** - * Setup tests. - */ - public function setUp() - { - parent::setUp(); - - $this->repository = m::mock(ApiPermissionRepositoryInterface::class); - $this->service = new PermissionService($this->repository); - } - - /** - * Test that a new API permission can be assigned to a key. - */ - public function test_create_function() - { - $this->repository->shouldReceive('withoutFreshModel')->withNoArgs()->once()->andReturnSelf() - ->shouldReceive('create')->with(['key_id' => 1, 'permission' => 'test-permission']) - ->once()->andReturn(true); - - $this->assertTrue($this->service->create(1, 'test-permission')); - } - - /** - * Test that function returns an array of all the permissions available as defined on the model. - */ - public function test_get_permissions_function() - { - $this->repository->shouldReceive('getModel')->withNoArgs()->once()->andReturn(new APIPermission()); - - $this->assertEquals(APIPermission::CONST_PERMISSIONS, $this->service->getPermissions()); - } -}