From 8c0871c9adc67cdf2560b7cf8d7319105bc66b09 Mon Sep 17 00:00:00 2001 From: Shishir Suvarna Date: Wed, 10 Jan 2024 17:23:53 +0530 Subject: [PATCH] Fix PHPCS fully_qualified_strict_types, nullable_type_declaration_for_default_null_value and unused import --- .../AppCredentialControllerTestBase.php | 58 +++++++++---------- .../AppGroupAppCredentialControllerTest.php | 3 +- 2 files changed, 30 insertions(+), 31 deletions(-) diff --git a/tests/Api/ApigeeX/Controller/AppCredentialControllerTestBase.php b/tests/Api/ApigeeX/Controller/AppCredentialControllerTestBase.php index ca0be7a0..ca61bab3 100644 --- a/tests/Api/ApigeeX/Controller/AppCredentialControllerTestBase.php +++ b/tests/Api/ApigeeX/Controller/AppCredentialControllerTestBase.php @@ -43,13 +43,13 @@ abstract class AppCredentialControllerTestBase extends EntityControllerTestBase // The order of these trait matters. Check @depends in test methods. use AttributesAwareEntityControllerTestTrait; - /** @var \Apigee\Edge\Api\ApigeeX\Entity\ApiProductInterface */ + /** @var ApiProductInterface */ protected static $testApiProduct; - /** @var \Apigee\Edge\Api\Management\Entity\AppOwnerInterface */ + /** @var AppOwnerInterface */ protected static $testAppOwner; - /** @var \Apigee\Edge\Api\Management\Entity\AppInterface */ + /** @var AppInterface */ protected static $testApp; /** @@ -66,12 +66,12 @@ public static function setUpBeforeClass(): void public function testCreatedAppHasAnEmptyCredential(): void { - /** @var \Apigee\Edge\Api\Management\Entity\DeveloperAppInterface $entity */ - /** @var \Apigee\Edge\Api\Management\Controller\AppByOwnerControllerInterface $controller */ + /** @var DeveloperAppInterface $entity */ + /** @var AppByOwnerControllerInterface $controller */ $entity = static::appByOwnerController()->load(static::$testApp->id()); $credentials = $entity->getCredentials(); $this->assertCount(1, $credentials); - /** @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /** @var AppCredentialInterface $credential */ $credential = reset($credentials); $this->assertCount(0, $credential->getApiProducts()); $this->assertNotEmpty($credential->getConsumerKey()); @@ -82,7 +82,7 @@ public function testCreatedAppHasAnEmptyCredential(): void /** * @depends testCreatedAppHasAnEmptyCredential * - * @return \Apigee\Edge\Api\Management\Entity\AppCredentialInterface + * @return AppCredentialInterface */ public function testCreate(): AppCredentialInterface { @@ -102,13 +102,13 @@ public function testCreate(): AppCredentialInterface /** * @depends testCreate * - * @param \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $created + * @param AppCredentialInterface $created * * @return string */ public function testLoad(AppCredentialInterface $created) { - /** @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $loaded */ + /** @var AppCredentialInterface $loaded */ $loaded = static::entityController()->load($created->id()); $this->assertCount(count($loaded->getApiProducts()), $created->getApiProducts()); $this->assertEquals($created->getConsumerKey(), $loaded->getConsumerKey()); @@ -126,7 +126,7 @@ public function testLoad(AppCredentialInterface $created) */ public function testAddProducts(string $entityId): void { - /** @var \Apigee\Edge\Api\Management\Controller\AppCredentialControllerInterface $controller */ + /** @var AppCredentialControllerInterface $controller */ $controller = $this->entityController(); $credential = $controller->addProducts($entityId, [static::$testApiProduct->id()]); $productNames = $this->getCredentialProducts($credential); @@ -140,9 +140,9 @@ public function testAddProducts(string $entityId): void */ public function testOverrideScopes(string $entityId): void { - /** @var \Apigee\Edge\Api\Management\Controller\AppCredentialControllerInterface $controller */ + /** @var AppCredentialControllerInterface $controller */ $controller = $this->entityController(); - /** @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /** @var AppCredentialInterface $credential */ $credential = $controller->load($entityId); $this->assertEmpty($credential->getScopes()); $credential = $controller->overrideScopes($entityId, ['scope 1']); @@ -165,9 +165,9 @@ public function testOverrideScopes(string $entityId): void public function testStatusChange(string $entityId): void { static::markOnlineTestSkipped(__FUNCTION__); - /** @var \Apigee\Edge\Api\Management\Controller\AppCredentialControllerInterface $controller */ + /** @var AppCredentialControllerInterface $controller */ $controller = static::entityController(); - /* @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /* @var AppCredentialInterface $credential */ $controller->setStatus($entityId, AppCredentialControllerInterface::STATUS_REVOKE); $credential = $controller->load($entityId); $this->assertEquals($credential->getStatus(), AppCredentialInterface::STATUS_REVOKED); @@ -186,16 +186,16 @@ public function testStatusChange(string $entityId): void public function testApiProductStatusChange(string $entityId): void { static::markOnlineTestSkipped(__FUNCTION__); - /** @var \Apigee\Edge\Api\Management\Controller\AppCredentialControllerInterface $controller */ + /** @var AppCredentialControllerInterface $controller */ $controller = static::entityController(); - /* @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /* @var AppCredentialInterface $credential */ $controller->setApiProductStatus( $entityId, static::$testApiProduct->id(), AppCredentialControllerInterface::STATUS_REVOKE ); $credential = $controller->load($entityId); - /** @var \Apigee\Edge\Structure\CredentialProduct $product */ + /** @var CredentialProduct $product */ foreach ($credential->getApiProducts() as $product) { if ($product->getApiproduct() === static::$testApiProduct->id()) { $this->assertEquals($product->getStatus(), CredentialProductInterface::STATUS_REVOKED); @@ -224,11 +224,11 @@ public function testApiProductStatusChange(string $entityId): void public function testGenerate(): string { static::markOnlineTestSkipped(__FUNCTION__); - /** @var \Apigee\Edge\Api\Management\Controller\AppCredentialControllerInterface $controller */ + /** @var AppCredentialControllerInterface $controller */ $controller = $this->entityController(); - /** @var \Apigee\Edge\Api\Management\Entity\AppInterface $app */ + /** @var AppInterface $app */ $app = static::appByOwnerController()->load(static::$testApp->id()); - /** @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /** @var AppCredentialInterface $credential */ $credential = $controller->generate( [static::$testApiProduct->id()], $app->getAttributes(), @@ -243,7 +243,7 @@ public function testGenerate(): string // Thanks for the offline tests, we can not expect a concrete value // here. $this->assertNotEquals('-1', $credential->getExpiresAt()); - /** @var \Apigee\Edge\Api\Management\Entity\AppInterface $updatedApp */ + /** @var AppInterface $updatedApp */ $updatedApp = static::appByOwnerController()->load(static::$testApp->id()); // Credential generation should not deleted any previously existing app // credentials. @@ -260,13 +260,13 @@ public function testGenerate(): string public function testDeleteApiProduct(string $entityId): void { static::markOnlineTestSkipped(__FUNCTION__); - /** @var \Apigee\Edge\Api\Management\Controller\AppCredentialControllerInterface $controller */ + /** @var AppCredentialControllerInterface $controller */ $controller = static::entityController(); - /** @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /** @var AppCredentialInterface $credential */ $credential = $controller->load($entityId); $productNames = $this->getCredentialProducts($credential); $this->assertContains(static::$testApiProduct->id(), $productNames); - /* @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /* @var AppCredentialInterface $credential */ $controller->deleteApiProduct( $entityId, static::$testApiProduct->id() @@ -281,15 +281,15 @@ public function testDeleteApiProduct(string $entityId): void */ public function testAddAttributesToEntity(): string { - /** @var \Apigee\Edge\Api\Management\Entity\AppCredentialInterface $credential */ + /** @var AppCredentialInterface $credential */ $credentials = static::$testApp->getCredentials(); $credential = reset($credentials); - /** @var \Apigee\Edge\Structure\AttributesProperty $attributes */ + /** @var AttributesProperty $attributes */ $attributes = $credential->getAttributes(); $originalAttributes = $attributes->values(); $attributes->add('name1', 'value1'); $attributes->add('name2', 'value2'); - /** @var \Apigee\Edge\Structure\AttributesProperty $attributesProperty */ + /** @var AttributesProperty $attributesProperty */ $attributesProperty = static::entityController()->updateAttributes($credential->id(), $attributes); /** @var array $newAttributes */ $newAttributes = $attributesProperty->values(); @@ -334,14 +334,14 @@ abstract protected static function setupTestApp(AppGroup $appOwner): AppInterfac abstract protected static function setupTestAppOwner(): AppGroup; /** - * @return \Apigee\Edge\Tests\Test\Controller\EntityControllerTesterInterface|\Apigee\Edge\Api\Management\Controller\AppByOwnerControllerInterface + * @return EntityControllerTesterInterface|AppByOwnerControllerInterface */ abstract protected static function appByOwnerController(): EntityControllerTesterInterface; private function getCredentialProducts(AppCredentialInterface $credential) { return array_map(function ($product) { - /* @var \Apigee\Edge\Structure\CredentialProduct $product */ + /* @var CredentialProduct $product */ return $product->getApiproduct(); }, $credential->getApiProducts()); } diff --git a/tests/Api/ApigeeX/Controller/AppGroupAppCredentialControllerTest.php b/tests/Api/ApigeeX/Controller/AppGroupAppCredentialControllerTest.php index dd208831..38cc3ac3 100644 --- a/tests/Api/ApigeeX/Controller/AppGroupAppCredentialControllerTest.php +++ b/tests/Api/ApigeeX/Controller/AppGroupAppCredentialControllerTest.php @@ -22,7 +22,6 @@ use Apigee\Edge\Api\ApigeeX\Entity\AppGroup; use Apigee\Edge\Api\ApigeeX\Entity\AppGroupInterface; use Apigee\Edge\Api\Management\Entity\AppInterface; -use Apigee\Edge\Api\Management\Entity\AppOwnerInterface; use Apigee\Edge\ClientInterface; use Apigee\Edge\Entity\EntityInterface; use Apigee\Edge\Tests\Api\ApigeeX\Entity\AppGroupAppTestEntityProviderTrait; @@ -48,7 +47,7 @@ class AppGroupAppCredentialControllerTest extends AppCredentialControllerTestBas /** * {@inheritdoc} */ - protected static function entityController(ClientInterface $client = null): EntityControllerTesterInterface + protected static function entityController(?ClientInterface $client = null): EntityControllerTesterInterface { $client = $client ?? static::defaultAPIClient();