Fix CI

Author: tvdijen

composer.json

@@ -48,6 +48,9 @@
         "predis/predis": "~3.3",
         "simplesamlphp/simplesamlphp-test-framework": "^1.11"
     },
+    "suggest": {
+        "predis/predis": "Needed if a Redis server is used to store session information"
+    },
     "support": {
         "issues": "https://github.com/simplesamlphp/simplesamlphp-module-consent/issues",
         "source": "https://github.com/simplesamlphp/simplesamlphp-module-consent"

src/Consent/Store/Redis.php

@@ -10,7 +10,6 @@
 use SimpleSAML\Configuration;
 use SimpleSAML\Logger;
 
-use function class_exists;
 use function count;
 
 /**
@@ -270,7 +269,7 @@ public function saveConsent(string $userId, string $destinationId, string $attri
             }
 
             return true;
-        } catch (\Exception $e) {
+        } catch (Exception $e) {
             Logger::error('consent:Redis - Failed to save consent: ' . $e->getMessage());
             return false;
         }

tests/src/Consent/Store/RedisTest.php

@@ -32,7 +32,8 @@ protected function setUp(): void
         /*
          * Create a mock Redis client that simulates the behavior of a Redis server.
          * This allows us to test the Redis store without needing an actual Redis server.
-         * The @method annotations are directly from https://github.com/predis/predis/blob/v3.6.0/src/ClientInterface.php
+         * The @method annotations are directly from
+         *   https://github.com/predis/predis/blob/v3.6.0/src/ClientInterface.php
          */
         $this->client = new class () extends Client {
             /** @var array<string, mixed> */

tools/composer-require-checker.json

@@ -1,4 +1,5 @@
 {
   "symbol-whitelist": [
+    "Predis\\Client"
   ]
 }