[#603] Change ClientInterface constant names and HybridOauth2 (#604)

* [#603] Change ClientInterface constant names and HybridOauth2 * Required apigee-client-php library to 2.0.9
apigee · Jul 27, 2021 · 1e16a4c · 1e16a4c
1 parent 389fdda
commit 1e16a4c
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 13 deletions.
diff --git a/composer.json b/composer.json
@@ -6,7 +6,7 @@
     "require": {
         "php": ">=7.1",
         "ext-json": "*",
-        "apigee/apigee-client-php": "^2.0.6",
+        "apigee/apigee-client-php": "^2.0.9",
         "drupal/core": "^8.7.7 || ^9",
         "drupal/entity": "^1.0",
         "drupal/key": "^1.8",

diff --git a/src/Command/Util/ApigeeEdgeManagementCliService.php b/src/Command/Util/ApigeeEdgeManagementCliService.php
@@ -62,7 +62,7 @@ public function createEdgeRoleForDrupal(StyleInterface $io,
 
     // Set default base URL if var is null or empty string.
     if (empty($base_url)) {
-      $base_url = ApigeeClientInterface::DEFAULT_ENDPOINT;
+      $base_url = ApigeeClientInterface::EDGE_ENDPOINT;
     }
     else {
       // Validate it is a valid URL.

diff --git a/src/Connector/HybridAuthentication.php b/src/Connector/HybridAuthentication.php
@@ -21,13 +21,13 @@
 namespace Drupal\apigee_edge\Connector;
 
 use Apigee\Edge\ClientInterface;
-use Apigee\Edge\HttpClient\Plugin\Authentication\HybridOauth2;
+use Apigee\Edge\HttpClient\Plugin\Authentication\ApigeeOnGcpOauth2;
 use Apigee\Edge\HttpClient\Plugin\Authentication\NullAuthentication;
 
 /**
  * Decorator for Hybrid authentication plugin.
  */
-class HybridAuthentication extends HybridOauth2 {
+class HybridAuthentication extends ApigeeOnGcpOauth2 {
 
   /**
    * {@inheritdoc}

diff --git a/src/KeyEntityFormEnhancer.php b/src/KeyEntityFormEnhancer.php
@@ -21,7 +21,7 @@
 namespace Drupal\apigee_edge;
 
 use Apigee\Edge\Exception\ApiRequestException;
-use Apigee\Edge\Exception\HybridOauth2AuthenticationException;
+use Apigee\Edge\Exception\ApigeeOnGcpOauth2AuthenticationException;
 use Apigee\Edge\Exception\OauthAuthenticationException;
 use Apigee\Edge\HttpClient\Plugin\Authentication\Oauth;
 use Drupal\apigee_edge\Exception\AuthenticationKeyException;
@@ -475,7 +475,7 @@ private function createSuggestion(\Exception $exception, KeyInterface $key): Mar
       ]);
     }
 
-    elseif ($exception instanceof HybridOauth2AuthenticationException) {
+    elseif ($exception instanceof ApigeeOnGcpOauth2AuthenticationException) {
       $fail_text = $this->t('Failed to connect to the authorization server.');
       // General error message.
       $suggestion = $this->t('@fail_text Check the debug information below for more details.', [

diff --git a/src/Plugin/EdgeKeyTypeBase.php b/src/Plugin/EdgeKeyTypeBase.php
@@ -69,10 +69,10 @@ public function getAuthenticationType(KeyInterface $key): string {
    */
   public function getEndpoint(KeyInterface $key): string {
     if ($this->getInstanceType($key) === EdgeKeyTypeInterface::INSTANCE_TYPE_HYBRID) {
-      return ClientInterface::HYBRID_ENDPOINT;
+      return ClientInterface::APIGEE_ON_GCP_ENDPOINT;
     }
     elseif ($this->getInstanceType($key) === EdgeKeyTypeInterface::INSTANCE_TYPE_PUBLIC) {
-      return Client::DEFAULT_ENDPOINT;
+      return Client::EDGE_ENDPOINT;
     }
     return $key->getKeyValues()['endpoint'];
   }
@@ -100,7 +100,7 @@ public function getInstanceType(KeyInterface $key): string {
     }
 
     // Backwards compatibility, before Hybrid support.
-    if (empty($key_values['endpoint']) || $key_values['endpoint'] === ClientInterface::DEFAULT_ENDPOINT) {
+    if (empty($key_values['endpoint']) || $key_values['endpoint'] === ClientInterface::EDGE_ENDPOINT) {
       return EdgeKeyTypeInterface::INSTANCE_TYPE_PUBLIC;
     }
 

diff --git a/tests/src/Unit/Command/Util/ApigeeEdgeManagementCliServiceTest.php b/tests/src/Unit/Command/Util/ApigeeEdgeManagementCliServiceTest.php
@@ -153,7 +153,7 @@ public function testCreateEdgeRoleForDrupalDefaultRoleAndBaseUrl() {
       ->shouldBeCalledTimes(1)
       ->willReturn('{ "name": "' . $this->org . '" }');
     $this->httpClient
-      ->get(Argument::exact(ApigeeClientInterface::DEFAULT_ENDPOINT . '/o/' . $this->org), Argument::type('array'))
+      ->get(Argument::exact(ApigeeClientInterface::EDGE_ENDPOINT . '/o/' . $this->org), Argument::type('array'))
       ->shouldBeCalledTimes(1)
       ->willReturn($response_org->reveal());
 
@@ -163,17 +163,17 @@ public function testCreateEdgeRoleForDrupalDefaultRoleAndBaseUrl() {
     $response_role->getStatusCode()->willReturn(404);
     $exception = new ClientException('Forbidden', $request_role->reveal(), $response_role->reveal());
     $this->httpClient
-      ->get(Argument::exact(ApigeeClientInterface::DEFAULT_ENDPOINT . '/o/' . $this->org . '/userroles/' . ApigeeEdgeManagementCliServiceInterface::DEFAULT_ROLE_NAME), Argument::type('array'))
+      ->get(Argument::exact(ApigeeClientInterface::EDGE_ENDPOINT . '/o/' . $this->org . '/userroles/' . ApigeeEdgeManagementCliServiceInterface::DEFAULT_ROLE_NAME), Argument::type('array'))
       ->willThrow($exception);
 
     // The role should be created.
     $this->httpClient
-      ->post(Argument::exact(ApigeeClientInterface::DEFAULT_ENDPOINT . '/o/' . $this->org . '/userroles'), Argument::type('array'))
+      ->post(Argument::exact(ApigeeClientInterface::EDGE_ENDPOINT . '/o/' . $this->org . '/userroles'), Argument::type('array'))
       ->shouldBeCalledTimes(1);
 
     // The permissions should be set.
     $this->httpClient
-      ->post(Argument::exact(ApigeeClientInterface::DEFAULT_ENDPOINT . '/o/' . $this->org . '/userroles/' . ApigeeEdgeManagementCliServiceInterface::DEFAULT_ROLE_NAME . '/permissions'), Argument::type('array'))
+      ->post(Argument::exact(ApigeeClientInterface::EDGE_ENDPOINT . '/o/' . $this->org . '/userroles/' . ApigeeEdgeManagementCliServiceInterface::DEFAULT_ROLE_NAME . '/permissions'), Argument::type('array'))
       ->shouldBeCalledTimes(12);
 
     $apigee_edge_management_cli_service = new ApigeeEdgeManagementCliService($this->httpClient->reveal());