chore: Inline shared test package

Author: keelerm84

.github/actions/ci/action.yml

@@ -10,10 +10,6 @@ inputs:
     type: boolean
     required: false
     default: false
-  shared-test-version:
-    description: 'Which version of the shared test package should we required'
-    required: false
-    default: 4.x-dev
   token:
     description: 'Token used to prevent composer rate limiting'
     required: true
@@ -33,10 +29,6 @@ runs:
       shell: bash
       run: composer install --no-progress
 
-    - name: Require appropriate shared tests package
-      shell: bash
-      run: composer require --dev 'launchdarkly/server-sdk-shared-tests:${{ inputs.shared-test-version }}'
-
     - name: Downgrade to lowest versions
       if: ${{ inputs.use-lowest-dependencies }}
       shell: bash

.github/workflows/ci.yml

@@ -23,29 +23,14 @@ jobs:
       fail-fast: false
       matrix:
         include:
-          # 8.1 configurations
           - php-version: 8.1
             use-lowest-dependencies: true
-            shared-test-version: 5.x-dev
-          - php-version: 8.1
-            use-lowest-dependencies: true
-            shared-test-version: dev-main
 
-          # 8.2 configurations
-          - php-version: 8.2
-            use-lowest-dependencies: false
-            shared-test-version: 5.x-dev
           - php-version: 8.2
             use-lowest-dependencies: false
-            shared-test-version: dev-main
 
-          # 8.3 configurations
-          - php-version: 8.3
-            use-lowest-dependencies: false
-            shared-test-version: 5.x-dev
           - php-version: 8.3
             use-lowest-dependencies: false
-            shared-test-version: dev-main
 
     steps:
       - uses: actions/checkout@v4
@@ -56,5 +41,4 @@ jobs:
         with:
           php-version: ${{ matrix.php-version }}
           use-lowest-dependencies: ${{ matrix.use-lowest-dependencies }}
-          shared-test-version: ${{ matrix.shared-test-version }}
           token: ${{ secrets.GITHUB_TOKEN }}

composer.json

@@ -1,10 +1,4 @@
 {
-    "repositories": [
-        {
-            "type": "vcs",
-            "url": "https://github.com/launchdarkly/php-server-sdk-shared-tests"
-        }
-    ],
     "name": "launchdarkly/server-sdk-redis-phpredis",
     "description": "LaunchDarkly PHP SDK Redis integration using the phpredis extension",
     "keywords": [

src/LaunchDarkly/Impl/Integrations/BigSegmentStore.php renamed to src/LaunchDarkly/Impl/Integrations/BigSegmentsStore.php

@@ -12,9 +12,9 @@
 use Redis;
 
 /**
- * Internal implementation of the php-redis BigSegmentStore interface.
+ * Internal implementation of the php-redis BigSegmentsStore interface.
  */
-class BigSegmentStore implements Subsystems\BigSegmentStore
+class BigSegmentsStore implements Subsystems\BigSegmentsStore
 {
     private const string KEY_LAST_UP_TO_DATE = ':big_segments_synchronized_on';
     private const string KEY_CONTEXT_INCLUDE = ':big_segment_include:';
@@ -35,14 +35,14 @@ public function __construct(
         $this->prefix = $options['prefix'] ?? Integrations\PHPRedis::DEFAULT_PREFIX;
     }
 
-    public function getMetadata(): Types\BigSegmentStoreMetadata
+    public function getMetadata(): Types\BigSegmentsStoreMetadata
     {
         try {
             /** @var string|false */
             $lastUpToDate = $this->connection->get($this->prefix . self::KEY_LAST_UP_TO_DATE);
         } catch (Exception $e) {
             $this->logger->warning('Error getting last-up-to-date time from Redis', ['exception' => $e->getMessage()]);
-            return new Types\BigSegmentStoreMetadata(lastUpToDate: null);
+            return new Types\BigSegmentsStoreMetadata(lastUpToDate: null);
         }
 
         if ($lastUpToDate === false) {
@@ -51,7 +51,7 @@ public function getMetadata(): Types\BigSegmentStoreMetadata
             $lastUpToDate = (int)$lastUpToDate;
         }
 
-        return new Types\BigSegmentStoreMetadata(lastUpToDate: $lastUpToDate);
+        return new Types\BigSegmentsStoreMetadata(lastUpToDate: $lastUpToDate);
     }
 
     public function getMembership(string $contextHash): ?array
@@ -62,7 +62,7 @@ public function getMembership(string $contextHash): ?array
             /** @var array<string> */
             $excludeRefs = $this->connection->sMembers($this->prefix . self::KEY_CONTEXT_EXCLUDE . $contextHash);
         } catch (Exception $e) {
-            $this->logger->warning('Error getting big segment membership from Redis', ['exception' => $e->getMessage()]);
+            $this->logger->warning('Error getting big segments membership from Redis', ['exception' => $e->getMessage()]);
             return null;
         }
 

src/LaunchDarkly/Integrations/PHPRedis.php

@@ -52,16 +52,16 @@ public static function featureRequester($options = [])
     /**
      * @param array<string,mixed> $options
      *   - `prefix`: namespace prefix to add to all hash keys
-     * @return callable(LoggerInterface, array): Subsystems\BigSegmentStore
+     * @return callable(LoggerInterface, array): Subsystems\BigSegmentsStore
      */
-    public static function bigSegmentStore(Redis $client, array $options = []): callable
+    public static function bigSegmentsStore(Redis $client, array $options = []): callable
     {
         if (!extension_loaded('redis')) {
             throw new \RuntimeException("phpredis extension is required to use Integrations\\PHPRedis");
         }
 
-        return function (LoggerInterface $logger, array $baseOptions) use ($client, $options): Subsystems\BigSegmentStore {
-            return new Integrations\BigSegmentStore($client, $logger, array_merge($baseOptions, $options));
+        return function (LoggerInterface $logger, array $baseOptions) use ($client, $options): Subsystems\BigSegmentsStore {
+            return new Integrations\BigSegmentsStore($client, $logger, array_merge($baseOptions, $options));
         };
     }
 }