Issue #3310455 by heddn: PHP 8 support & Drupal 10 support

composer.json

@@ -9,7 +9,7 @@
     ],
     "minimum-stability": "dev",
     "require": {
-        "php": "^7.2",
+        "php": "^7.2|^8",
         "league/oauth2-client": "^2"
     },
     "suggest": {

modules/box/box_token_manager.info.yml

 name: 'Box Token Manager'
 type: module
 description: 'OAuth2 Token Manager for Box'
-core: 8.x
-core_version_requirement: ^8 || ^9
+core_version_requirement: ^9.3 || ^10
 package: 'Security'
 dependencies:
   - oauth2_token_manager:oauth2_token_manager

modules/box/composer.json

@@ -15,7 +15,7 @@
     "extra": {
         "drush": {
             "services": {
-                "drush.services.yml": "^9 || ^10"
+                "drush.services.yml": "^10 || ^11 || ^12"
             }
         }
     }

modules/quip/composer.json

@@ -12,7 +12,7 @@
     "extra": {
         "drush": {
             "services": {
-                "drush.services.yml": "^9 || ^10"
+                "drush.services.yml": "^10 || ^11 || ^12"
             }
         }
     }

modules/quip/quip_token_manager.info.yml

 name: 'Quip Token Manager'
 type: module
 description: 'OAuth2 Token Manager for Quip'
-core: 8.x
-core_version_requirement: ^8 || ^9
+core_version_requirement: ^9.3 || ^10
 package: 'Security'
 dependencies:
   - oauth2_token_manager:oauth2_token_manager

modules/slack/composer.json

@@ -15,7 +15,7 @@
     "extra": {
         "drush": {
             "services": {
-                "drush.services.yml": "^9 || ^10"
+                "drush.services.yml": "^10 || ^11 || ^12"
             }
         }
     }

modules/slack/slack_token_manager.info.yml

 name: 'Slack Token Manager'
 type: module
 description: 'OAuth2 Token Manager for Slack'
-core: 8.x
-core_version_requirement: ^8 || ^9
+core_version_requirement: ^9.3 || ^10
 package: 'Security'
 dependencies:
   - oauth2_token_manager:oauth2_token_manager

oauth2_token_manager.info.yml

 name: 'OAuth2 Token Manager'
 type: module
 description: 'OAuth2 Token Manager'
-core: 8.x
-core_version_requirement: ^8 || ^9
+core_version_requirement: ^9.3 || ^10
 package: 'Security'

src/Commands/TokenGeneratorCommandsBase.php

@@ -35,7 +35,7 @@ class TokenGeneratorCommandsBase extends DrushCommands {
    *
    * @throws \Exception
    */
-  public function setRefreshToken($refresh_token) {
+  public function setRefreshToken($refresh_token): void {
     if (!$this->tokenManager->setSystemRefreshToken($refresh_token)) {
       throw new \Exception('Using refresh token failed.');
     }
@@ -46,7 +46,7 @@ class TokenGeneratorCommandsBase extends DrushCommands {
   /**
    * Initializes a new refresh token.
    */
-  public function initRefreshToken() {
+  public function initRefreshToken(): void {
     $authUrl = $this->tokenManager->initSystemAccessToken();
     $this->logger()->success(dt('Please visit the following URL to initialize the system token: {url}', ['url' => $authUrl]));
   }

tests/src/Functional/LoadTest.php

@@ -22,7 +22,7 @@ class LoadTest extends BrowserTestBase {
    *
    * @var array
    */
-  public static $modules = [
+  protected static $modules = [
     'oauth2_token_manager',
     'box_token_manager',
     'quip_token_manager',
@@ -39,7 +39,7 @@ class LoadTest extends BrowserTestBase {
   /**
    * {@inheritdoc}
    */
-  protected function setUp() {
+  protected function setUp(): void {
     parent::setUp();
     $this->user = $this->drupalCreateUser(['administer site configuration']);
     $this->drupalLogin($this->user);
@@ -48,9 +48,9 @@ class LoadTest extends BrowserTestBase {
   /**
    * Tests that the home page loads with a 200 response.
    */
-  public function testLoad() {
+  public function testLoad(): void {
     $this->drupalGet(Url::fromRoute('<front>'));
-    $this->assertResponse(200);
+    $this->assertSession()->statusCodeEquals(200);
   }
 
 }